Hi YoungJun,
Thank you for the patch.
On Wednesday 16 April 2014 13:38:57 YoungJun Cho wrote:
> This patch adds DT bindings for s6e3fa0 panel.
> The bindings describes panel resources, display timings, delays
> and physical size.
>
> Changelog v2:
> - Adds unit address (commented by Sachin Kamat)
>
> Signed-off-by: YoungJun Cho <yj44.cho@xxxxxxxxxxx>
> Signed-off-by: Inki Dae <inki.dae@xxxxxxxxxxx>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> ---
> .../devicetree/bindings/panel/samsung,s6e3fa0.txt | 52 +++++++++++++++++
> 1 file changed, 52 insertions(+)
> create mode 100644
> Documentation/devicetree/bindings/panel/samsung,s6e3fa0.txt
>
> diff --git a/Documentation/devicetree/bindings/panel/samsung,s6e3fa0.txt
> b/Documentation/devicetree/bindings/panel/samsung,s6e3fa0.txt new file mode
> 100644
> index 0000000..889a74d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/panel/samsung,s6e3fa0.txt
> @@ -0,0 +1,52 @@
> +Samsung S6E3FA0 AMOLED LCD 5.7 inch panel
> +
> +Required properties:
> + - compatible: "samsung,s6e3fa0"
> + - reg: the virtual channel number of a DSI peripheral
> + - vdd3-supply: core voltage supply
> + - vci-supply: voltage supply for analog circuits
> + - reset-gpio: a GPIO spec for the reset pin
> + - display-timings: timings for the connected panel as described by [1]
> +
> +Optional properties:
> + - power-on-delay: delay after turning regulators on [ms]
> + - reset-delay: delay after reset sequence [ms]
> + - init-delay: delay after initialization sequence [ms]
> + - panel-width-mm: physical panel width [mm]
> + - panel-height-mm: physical panel height [mm]
Based on the next patch the panel seems to require quite a lot of device-
specific handling, although I might be mistaken as I'm not too familiar with
DSI. If my understanding is correct, do we really need to specify all those
properties in DT, or could they be hardcoded in the driver ? I would usually
push for generic drivers that read device information from DT, but I'm
wondering whether that is the right choice here.
If you decide to keep the information in DT I think you should document what
happens when optional properties are not specified. I suppose the driver uses
a default value, I would document it in the DT bindings.
> +The device node can contain one 'port' child node with one child
> +'endpoint' node, according to the bindings defined in [2]. This
> +node should describe panel's video bus.
> +
> +[1]: Documentation/devicetree/bindings/video/display-timing.txt
> +[2]: Documentation/devicetree/bindings/media/video-interfaces.txt
> +
> +Example:
> +
> + panel@0 {
> + compatible = "samsung,s6e3fa0";
> + reg = <0>;
> + vdd3-supply = <&vcclcd_reg>;
> + vci-supply = <&vlcd_reg>;
> + reset-gpio = <&gpy7 4 0>;
> + power-on-delay= <10>;
> + reset-delay = <5>;
> + init-delay = <120>;
> + panel-width-mm = <71>;
> + panel-height-mm = <126>;
> +
> + display-timings {
> + timing0: timing-0 {
> + clock-frequency = <57153600>;
> + hactive = <1080>;
> + vactive = <1920>;
> + hfront-porch = <2>;
> + hback-porch = <2>;
> + hsync-len = <1>;
> + vfront-porch = <1>;
> + vback-porch = <4>;
> + vsync-len = <1>;
> + };
> + };
> + };
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
