[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