From: Marcel Ziswiler marcel.ziswiler@toradex.com
Currently, compiling any i.MX 6 based target results in the following warning:
w+arch/arm/dts/imx6q-dhcom-pdk2.dtb: Warning (pwms_property): /display-bl:pwms: cell 3 is not a phandle reference w+arch/arm/dts/imx6q-dhcom-pdk2.dtb: Warning (pwms_property): /display-bl: Missing property '#pwm-cells' in node /soc/bus@2000000/gpc@20dc000 or bad phandle (referred from pwms[3]) w+arch/arm/dts/imx6dl-dhcom-pdk2.dtb: Warning (pwms_property): /display-bl:pwms: cell 3 is not a phandle reference w+arch/arm/dts/imx6dl-dhcom-pdk2.dtb: Warning (pwms_property): /display-bl: Missing property '#pwm-cells' in node /soc/bus@2000000/gpc@20dc000 or bad phandle (referred from pwms[3])
This is due to the imx6qdl-dhcom-pdk2.dtsi using a later pwm-cells of 3 (e.g. the one with the last cell indicating whether or not it is inverted) then what U-Boot's current imx6qdl.dtsi has. Unfortunately, fixing aka synchronising that one as well would lead to further issues:
w+arch/arm/dts/imx6dl-aristainetos2c_cslb_7.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-brppt2.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-cubox-i.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-aristainetos2c_7.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-cubox-i-emmc-som-v15.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-cubox-i-som-v15.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-icore.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-icore-mipi.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-sabreauto.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6dl-sabresd.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-cubox-i.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-cubox-i-emmc-som-v15.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-cubox-i-som-v15.dtb: Warning (pwms_property): /pwmleds/front:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-icore.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-icore-mipi.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-logicpd.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-novena.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-phytec-mira-rdk-nand.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-sabreauto.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6q-sabresd.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6qp-sabreauto.dtb: Warning (pwms_property): /backlight:pwms: property size (12) too small for cell size 3 w+arch/arm/dts/imx6qp-sabresd.dtb: Warning (pwms_property): /backlight-lvds:pwms: property size (12) too small for cell size 3
Messing with all those so late in the game does not seem like too good of an idea. Therefore, I suggest reverting just that one PWM node for now. We may (and really should) properly synchronise all i.MX 6 DTs again in the next cycle.
Fixes: 8ed378ff78d1 ("ARM: dts: imx: Synchronize iMX6QDL DHCOM PDK2 DTs with Linux 5.15.12") Signed-off-by: Marcel Ziswiler marcel.ziswiler@toradex.com
---
arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi b/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi index bf6b3a5ce0..9dcb37ec7e 100644 --- a/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi +++ b/arch/arm/dts/imx6qdl-dhcom-pdk2.dtsi @@ -25,7 +25,7 @@ compatible = "pwm-backlight"; default-brightness-level = <8>; enable-gpios = <&gpio3 27 GPIO_ACTIVE_HIGH>; /* GPIO G */ - pwms = <&pwm1 0 50000 PWM_POLARITY_INVERTED>; + pwms = <&pwm1 0 50000>; status = "okay"; };