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: devicet...@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.wall...@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@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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
