Hi Tomi, On 05/15/2014 12:10 PM, Tomi Valkeinen wrote: > Add DT bindings documentation for LG.Philips LB035Q02 LCD panel. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > --- > .../bindings/video/lgphilips,lb035q02.txt | 33 ++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > create mode 100644 Documentation/devicetree/bindings/video/lgphilips,lb035q02.txt > > diff --git a/Documentation/devicetree/bindings/video/lgphilips,lb035q02.txt b/Documentation/devicetree/bindings/video/lgphilips,lb035q02.txt > new file mode 100644 > index 000000000000..1a1e653e5407 > --- /dev/null > +++ b/Documentation/devicetree/bindings/video/lgphilips,lb035q02.txt > @@ -0,0 +1,33 @@ > +LG.Philips LB035Q02 Panel > +========================= > + > +Required properties: > +- compatible: "lgphilips,lb035q02" > +- enable-gpios: panel enable gpio > + > +Optional properties: > +- label: a symbolic name for the panel > + > +Required nodes: > +- Video port for DPI input > + > +Example > +------- > + > +lcd-panel: panel@0 { > + compatible = "lgphilips,lb035q02"; > + reg = <0>; > + spi-max-frequency = <100000>; > + spi-cpol; > + spi-cpha; > + > + label = "lcd"; > + > + enable-gpios = <&gpio7 7 0>; > + > + port { > + lcd_in: endpoint { > + remote-endpoint = <&dpi_out>; > + }; > + }; > +}; > The lcd-panel should be a child node of a SPI controller. Maybe this could be mentioned? Or the example could be more explicit, like: &mcspi1 { lcd-panel: panel@0 { .... }; }; Regards, Florian -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html