On 2/20/2020 1:43 PM, Fabio Estevam wrote: > Hi Troy, > > On Thu, Feb 20, 2020 at 6:37 PM Troy Kisky > <[email protected]> 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 <[email protected]> >> >> 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 = <®_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
