[PATCH 1/2] drm/panel: Add DT bindings for Samsung S6D16D0

[Sam Ravnborg]

On Mon, Oct 08, 2018 at 01:00:05PM +0200, Linus Walleij wrote:
> This adds the device tree bindings for the Samsung S6D16D0
> panel. This is a command mode only panel using DSI.
> 
> Cc: devicetree at vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
> ---
>  .../display/panel/samsung,s6d16d0.txt         | 30 +++++++++++++++++++
>  1 file changed, 30 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.txt
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.txt b/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.txt
> new file mode 100644
> index 000000000000..b94e366f451b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6d16d0.txt
> @@ -0,0 +1,30 @@
> +Samsung S6D16D0 4" 864x480 AMOLED panel
> +
> +Required properties:
> +  - compatible: should be:
> +    "samsung,s6d16d0",
> +  - reg: the virtual channel number of a DSI peripheral
> +  - vdd1-supply: I/O voltage supply
> +  - reset-gpios: a GPIO spec for the reset pin (active low)
That it is active low is part of the GPIO part and should not
be mandated as part of the binding?

> +
> +The device node can contain one 'port' child node with one child
> +'endpoint' node, according to the bindings defined in
> +media/video-interfaces.txt. This node should describe panel's video bus.
> +
> +Example:
> +&dsi {
> +	...
> +
> +	panel at 0 {
> +		compatible = "samsung,s6d16d0";
> +		reg = <0>;
> +		vdd1-supply = <&foo>;
> +		reset-gpios = <&foo_gpio 0 GPIO_ACTIVE_LOW>;
This line specifies if ACTIVE HIGH or LOW.

	Sam