[PATCH] board: nitrogen6x: migrate to CONFIG_DM_VIDEO

[Troy Kisky]

On 2/20/2020 1:43 PM, Fabio Estevam wrote:
> Hi Troy,
> 
> On Thu, Feb 20, 2020 at 6:37 PM Troy Kisky
> <troy.kisky at boundarydevices.com> wrote:
>>
>> Enable driver model for Video to remove compiler warning.
>> To enable display, issue
>>
>> setenv stdout serial,vidconsole;
>>
>> Add CONFIG_DM_VIDEO to all defconfigs whose board is
>> nitrogen6x.
>>         mx6qsabrelite_defconfig
>>         nitrogen6dl2g_defconfig
>>         nitrogen6dl_defconfig
>>         nitrogen6q2g_defconfig
>>         nitrogen6q_defconfig
>>         nitrogen6s1g_defconfig
>>         nitrogen6s_defconfig
>>
>> Signed-off-by: Troy Kisky <troy.kisky at boundarydevices.com>
>>
>> diff --git a/arch/arm/dts/imx6qdl-sabrelite.dtsi b/arch/arm/dts/imx6qdl-sabrelite.dtsi
>> index 673a19c3df..86375156b9 100644
>> --- a/arch/arm/dts/imx6qdl-sabrelite.dtsi
>> +++ b/arch/arm/dts/imx6qdl-sabrelite.dtsi
>> @@ -195,6 +195,15 @@
>>                 pwm_lvds = &pwm4;
>>         };
>>
>> +       backlight_lvds: backlight-lvds {
>> +               brightness-levels = <0 1 2 3 4 5 6 7 8 9 10>;
>> +               compatible = "pwm-backlight";
>> +               default-brightness-level = <8>;
>> +               power-supply = <&reg_3p3v>;
>> +               pwms = <&pwm4 0 5000000>;
>> +               status = "okay";
>> +       };
>> +
>>         chosen {
>>                 stdout-path = &uart2;
>>         };
>> @@ -219,6 +228,17 @@
>>                 gpio = GP_REG_USBOTG;
>>                 enable-active-high;
>>         };
>> +
>> +       panel-lvds0 {
>> +               backlight = <&backlight_lvds>;
>> +               compatible = "hannstar,hsd100pxn1";
> 
> Instead of adding only the LVDS pieces here, what about resyncing the
> whole dts with the upstream kernel on a separate patch ?
> 

Or I can just skip the dts changes. I don't think they are needed.


Do we have a policy for dead dts code ?

Thanks
Troy