[PATCH v2 0/8] arm64: dts: imx: cleanup some warning with W=1

[PATCH v2 0/8] arm64: dts: imx: cleanup some warning with W=1

[Frank.Li]

Cleanup some warning when W=1

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v2:
- collect review-by tags
- add description about fan-controller optional descrition at commit msg.
- Link to v1: https://patch.msgid.link/20260925-cleanup_dtb_warning-v1-0-f999fafbca60@nxp.com

---
Frank Li (8):
      dt-bindings: display: remove #address-cells and #size-cells off requires
      dt-bindings: embedded-controller: remove #address-cells and #size-cells from required
      arm64: dts: s32n79: add @addr for *-bus node
      arm64: dts: imx8mq: remove #address-cells and #size-cells in dsi@30a00000
      arm64: dts: fsl-lx216x: remove #address-cells and #size-cells from pinctrl node
      arm64: dts: imx: remove unnecessary #address-cells and #size-cells
      arm64: dts: imx95: rename bus@4c100000 to hsio-bus
      arm64: dts: imx952: rename soc to soc@0

 Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml     | 2 --
 Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml | 2 --
 arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts               | 4 ----
 arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi         | 4 ----
 arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi                     | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts            | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts            | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts            | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts            | 2 --
 arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts            | 2 --
 arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts                 | 2 --
 arch/arm64/boot/dts/freescale/imx8mq.dtsi                         | 2 --
 arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts              | 2 --
 arch/arm64/boot/dts/freescale/imx95.dtsi                          | 2 +-
 arch/arm64/boot/dts/freescale/imx952.dtsi                         | 2 +-
 arch/arm64/boot/dts/freescale/s32n79.dtsi                         | 6 +++---
 16 files changed, 5 insertions(+), 35 deletions(-)
---
base-commit: 3d7783543c2646af69ad65825e810060494bea21
change-id: 20260924-cleanup_dtb_warning-d350d5b29681

Best regards,
--  
Frank Li <Frank.Li@nxp.com>

[PATCH v2 1/8] dt-bindings: display: remove #address-cells and #size-cells off requires

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The display panel node is not necessary located in dsi node. So remove
these from requires list.

Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
index 5952e6448ed47..a0c3ba9275684 100644
--- a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
@@ -137,8 +137,6 @@ properties:
       - port@1
 
 required:
-  - '#address-cells'
-  - '#size-cells'
   - clock-names
   - clocks
   - compatible

-- 
2.43.0

[PATCH v2 2/8] dt-bindings: embedded-controller: remove #address-cells and #size-cells from required

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The "^fan-controller@[0-9a-f]+$" child node is optional, so it is not
necessary to always have address-based child nodes.

"refer to the board hardware user manuals to see what features are present"
Means fan-controller is optional, which may not present at boards.

Remove '#address-cells' and '#size-cells' from the required list.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
change at v2
- add descriptions about why fan-controller is optional.
---
 Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml b/Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml
index 82d4b2dadbae4..99f685938e20a 100644
--- a/Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml
+++ b/Documentation/devicetree/bindings/embedded-controller/gw,gsc.yaml
@@ -136,8 +136,6 @@ required:
   - interrupts
   - interrupt-controller
   - "#interrupt-cells"
-  - "#address-cells"
-  - "#size-cells"
 
 additionalProperties: false
 

-- 
2.43.0

[PATCH v2 3/8] arm64: dts: s32n79: add @addr for *-bus node

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Add address suffix for bus node to fix warning (W=1)
  arch/arm64/boot/dts/freescale/s32n79.dtsi:15.10-54.4: Warning (unit_address_vs_reg): /cis-bus: node has a reg or ranges property, but no unit name

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/s32n79.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/s32n79.dtsi b/arch/arm64/boot/dts/freescale/s32n79.dtsi
index 94ab58783fdc8..cbb43df8e1bc5 100644
--- a/arch/arm64/boot/dts/freescale/s32n79.dtsi
+++ b/arch/arm64/boot/dts/freescale/s32n79.dtsi
@@ -12,7 +12,7 @@ / {
 	#address-cells = <2>;
 	#size-cells = <2>;
 
-	cis-bus {
+	cis-bus@4f200000 {
 		compatible = "simple-bus";
 		ranges = <0x4f200000 0x0 0x4f200000 0xc00000>;
 		#address-cells = <1>;
@@ -53,7 +53,7 @@ smmu: iommu@4fc00000 {
 		};
 	};
 
-	coss-bus {
+	coss-bus@4a000000 {
 		compatible = "simple-bus";
 		ranges = <0x4a000000 0x0 0x4a000000 0xff0000>,
 			 <0x4e000000 0x0 0x4e000000 0x1000000>;
@@ -330,7 +330,7 @@ clks: protocol@14 {
 		};
 	};
 
-	fss-bus {
+	fss-bus@5b490000 {
 		compatible = "simple-bus";
 		ranges = <0x5b490000 0x0 0x5b490000 0x1000>;
 		#address-cells = <1>;

-- 
2.43.0

[PATCH v2 4/8] arm64: dts: imx8mq: remove #address-cells and #size-cells in dsi@30a00000

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

No children nodes under dsi@30a00000, so remove '#address-cells' and
'#size-cells' to fix warning with (W=1):

arch/arm64/boot/dts/freescale/imx8mq.dtsi:1247.27-1297.6: Warning (avoid_unnecessary_addr_size): /soc@0/bus@30800000/dsi@30a00000: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" or "ranges" property
  DTC [C] arch/arm64/boot/dts/freescale/imx8mq-kontron-pitx-imx8m.dtb

Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 5545b5b20ab2e..8d53df2be2280 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -1247,8 +1247,6 @@ sec_jr2: jr@3000 {
 			mipi_dsi: dsi@30a00000 {
 				compatible = "fsl,imx8mq-nwl-dsi";
 				reg = <0x30a00000 0x300>;
-				#address-cells = <1>;
-				#size-cells = <0>;
 				clocks = <&clk IMX8MQ_CLK_DSI_CORE>,
 					 <&clk IMX8MQ_CLK_DSI_AHB>,
 					 <&clk IMX8MQ_CLK_DSI_IPG_DIV>,

-- 
2.43.0

[PATCH v2 5/8] arm64: dts: fsl-lx216x: remove #address-cells and #size-cells from pinctrl node

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

The pinmux node has no address-based child nodes, so '#address-cells' and
'#size-cells' are unnecessary. Remove them to fix the following W=1 warning:

  arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi:1534.34-1704.5: Warning (avoid_unnecessary_addr_size): /soc/pinmux@70010012c: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" or "ranges" property

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
index bccf2d6730230..6d2667051674b 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
@@ -1534,8 +1534,6 @@ pcs18: ethernet-phy@0 {
 		pinmux_i2crv: pinmux@70010012c {
 			compatible = "pinctrl-single";
 			reg = <0x00000007 0x0010012c 0x0 0xc>;
-			#address-cells = <1>;
-			#size-cells = <0>;
 			pinctrl-single,bit-per-mux;
 			pinctrl-single,register-width = <32>;
 			pinctrl-single,function-mask = <0x7>;

-- 
2.43.0

[PATCH v2 6/8] arm64: dts: imx: remove unnecessary #address-cells and #size-cells

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Remove unnecessary #address-cells and #size-cells to fix the dtb W=1 warning:
  Warning (avoid_unnecessary_addr_size): /soc@0/bus@30800000/i2c@30a20000/gsc@20: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" or "ranges" property

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com> # TQMLX2160A
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts       | 4 ----
 arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi | 4 ----
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts    | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts    | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts    | 2 --
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts    | 2 --
 arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts    | 2 --
 arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts         | 2 --
 arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts      | 2 --
 9 files changed, 22 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
index f51508952d51c..b762ad555ad9e 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
@@ -312,8 +312,6 @@ &qspi {
 
 	en25s64: flash@0 {
 		compatible = "jedec,spi-nor";
-		#address-cells = <1>;
-		#size-cells = <1>;
 		reg = <0>;
 		spi-max-frequency = <20000000>;
 		spi-rx-bus-width = <4>;
@@ -363,8 +361,6 @@ partition@600000 {
 
 	nand: flash@1 {
 		compatible = "spi-nand";
-		#address-cells = <1>;
-		#size-cells = <1>;
 		reg = <1>;
 		spi-max-frequency = <20000000>;
 		spi-rx-bus-width = <4>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
index cc6b6427ec8a6..aac0ae0ad40c7 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
@@ -41,8 +41,6 @@ &fspi {
 	flash0: flash@0 {
 		compatible = "jedec,spi-nor";
 		reg = <0>;
-		#address-cells = <1>;
-		#size-cells = <1>;
 		spi-max-frequency = <10000000>;
 		spi-rx-bus-width = <1>;
 		spi-tx-bus-width = <1>;
@@ -57,8 +55,6 @@ partitions {
 	flash1: flash@1 {
 		compatible = "jedec,spi-nor";
 		reg = <1>;
-		#address-cells = <1>;
-		#size-cells = <1>;
 		spi-max-frequency = <10000000>;
 		spi-rx-bus-width = <1>;
 		spi-tx-bus-width = <1>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
index 272c2b223d167..1bb42116fc0b8 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
@@ -366,8 +366,6 @@ gsc: gsc@20 {
 		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
index 468c7e993c52b..f53718221ad5b 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
@@ -317,8 +317,6 @@ gsc: gsc@20 {
 		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
index 636daa3d6ca20..3eafff879e8d5 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
@@ -282,8 +282,6 @@ gsc: gsc@20 {
 		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
index 99572961d9e16..fcbb40597b12c 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
@@ -335,8 +335,6 @@ gsc: gsc@20 {
 		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
index 5aa0e2cd155ec..ecc12645d7691 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
@@ -315,8 +315,6 @@ gsc: gsc@20 {
 		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts b/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
index b91509bfc9c00..1043bf074f8e4 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
@@ -287,8 +287,6 @@ &lcdif {
 };
 
 &mipi_dsi {
-	#address-cells = <1>;
-	#size-cells = <0>;
 	status = "okay";
 
 	ports {
diff --git a/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts b/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
index c6019d76b9981..f3b647267c300 100644
--- a/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
@@ -202,8 +202,6 @@ &flexspi1 {
 	flash@0 {
 		compatible = "spi-nand";
 		reg = <0>;
-		#address-cells = <1>;
-		#size-cells = <1>;
 		spi-max-frequency = <104000000>;
 		spi-rx-bus-width = <4>;
 		spi-tx-bus-width = <4>;

-- 
2.43.0

[PATCH v2 7/8] arm64: dts: imx95: rename bus@4c100000 to hsio-bus

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Rename bus@4c100000 to hsio-bus to warning (W=1).
  arch/arm64/boot/dts/freescale/imx95.dtsi:1878.16-1907.5: Warning (unit_address_vs_reg): /soc/bus@4c100000: node has a unit name, but no reg or ranges property

Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx95.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx95.dtsi b/arch/arm64/boot/dts/freescale/imx95.dtsi
index d20c2498b9aa2..b8711f9f846ce 100644
--- a/arch/arm64/boot/dts/freescale/imx95.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx95.dtsi
@@ -1875,7 +1875,7 @@ hsio_blk_ctl: syscon@4c0100c0 {
 			power-domains = <&scmi_devpd IMX95_PD_HSIO_TOP>;
 		};
 
-		bus@4c100000 {
+		hsio-bus {
 			compatible = "simple-bus";
 			#address-cells = <2>;
 			#size-cells = <2>;

-- 
2.43.0

[PATCH v2 8/8] arm64: dts: imx952: rename soc to soc@0

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Rename soc to soc@0 to fix warning (W=1)
  arch/arm64/boot/dts/freescale/imx952.dtsi:348.6-1539.4: Warning (unit_address_vs_reg): /soc: node has a reg or ranges property, but no unit name

Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx952.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx952.dtsi b/arch/arm64/boot/dts/freescale/imx952.dtsi
index 1154b66dadb2d..3c51e61d1fc36 100644
--- a/arch/arm64/boot/dts/freescale/imx952.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx952.dtsi
@@ -345,7 +345,7 @@ opp-1000000000 {
 		};
 	};
 
-	soc {
+	soc@0 {
 		compatible = "simple-bus";
 		#address-cells = <2>;
 		#size-cells = <2>;

-- 
2.43.0