On 05/19/2015 09:04 AM, Philipp Zabel wrote: > Hi Eric, > > Am Dienstag, den 19.05.2015, 08:50 -0700 schrieb Eric Nelson: >> Also add label for backlight_lvds and connect it to the LVDS display. >> >> Signed-off-by: Eric Nelson <eric.nelson@xxxxxxxxxxxxxxxxxxx> >> --- > [...] >> @@ -129,6 +129,17 @@ >> power-supply = <®_3p3v>; >> status = "okay"; >> }; >> + >> + panel { >> + compatible = "hannstar,hsd100pxn1"; >> + backlight = <&backlight_lvds>; >> + >> + port { >> + panel_in: endpoint { >> + remote-endpoint = <&lvds0_out>; >> + }; >> + }; >> + }; >> }; >> >> &audmux { >> @@ -374,18 +385,11 @@ >> fsl,data-width = <18>; > > The fsl,data-width property shouldn't be needed anymore if you > set .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG for the hsd100pxn1 > panel in panel-simple.c. > Thanks again Philipp. That's a much better place to put it, since this is a panel feature, not a feature of the LVDS channel. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html