Re: [PATCH 2/2] ARM: dts: sabrelite: use simple-panel instead of display-timings for LVDS0

Eric Nelson
2015-05-19T16:51:23Z

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-Q5RJGjKts06CY9SHAMCTRUEOCMrvLtNR@public.gmane.org>
>> ---
> [...]
>> @@ -129,6 +129,17 @@
>> power-supply = <&reg_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.