[PATCH v1] ARM: dts: imx: Fix iMX6QDL DHCOM PDK2 DTC Warnings

Marcel Ziswiler marcel at ziswiler.com
Wed Apr 27 00:29:38 CEST 2022


From: Marcel Ziswiler <marcel.ziswiler at 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 at 2000000/gpc at 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 at 2000000/gpc at 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 at 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";
 	};
 
-- 
2.34.1



More information about the U-Boot mailing list