[PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings

[PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings

[Frank Li]

Cleanup some CHECK_DTB warings.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Frank Li (10):
      ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning
      ARM: dts: imx35: rename i2c clock-names to ipg
      ARM: dts: imx35: remove simple-bus 'usbphy'
      ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0
      ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1
      ARM: dts: imx53: drop fallback compatible "dlg,da9052"
      ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency
      ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000
      ARM: dts: imx53-smd:  Add power supply node for fsl,sgtl5000
      ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680

 arch/arm/boot/dts/nxp/imx/imx31.dtsi               |  2 +-
 arch/arm/boot/dts/nxp/imx/imx35.dtsi               | 30 ++++++++--------------
 arch/arm/boot/dts/nxp/imx/imx51-babbage.dts        |  2 +-
 arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts         |  2 +-
 arch/arm/boot/dts/nxp/imx/imx53-qsb.dts            |  2 +-
 arch/arm/boot/dts/nxp/imx/imx53-smd.dts            | 18 +++++++++++++
 .../arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi |  2 +-
 arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi   |  8 +++++-
 arch/arm/boot/dts/nxp/imx/imx7s-warp.dts           |  2 --
 arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi             |  1 -
 10 files changed, 41 insertions(+), 28 deletions(-)
---
base-commit: d56315c1bfc31d25586028cc46cf6bcbb6e21e56
change-id: 20260202-imx31_dts_warning-0f0a0f9bae2b

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

[PATCH 01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning

[Frank Li]

Rename emi to emi-bus to fix below CHECK_DTBS warning:
arch/arm/boot/dts/nxp/imx/imx31-bug.dtb: emi@b8000000 (simple-bus): $nodename:0: 'emi@b8000000' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
        from schema $id: http://devicetree.org/schemas/simple-bus.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx31.dtsi | 2 +-
 arch/arm/boot/dts/nxp/imx/imx35.dtsi | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx31.dtsi b/arch/arm/boot/dts/nxp/imx/imx31.dtsi
index 8541a666747a94f4bc64ca4b48c709ed64375a9d..c58f855ea85143e28ff33b14951c470e14c56a8e 100644
--- a/arch/arm/boot/dts/nxp/imx/imx31.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx31.dtsi
@@ -333,7 +333,7 @@ pwm: pwm@53fe0000 {
 			};
 		};
 
-		emi@b8000000 { /* External Memory Interface */
+		emi-bus@b8000000 { /* External Memory Interface */
 			compatible = "simple-bus";
 			reg = <0xb8000000 0x5000>;
 			ranges;
diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index 111d7c0331f504723ef160235eca1381eab0524f..1c010a83d5df138562cd086c7b10d34431378add 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -356,7 +356,7 @@ usbmisc: usbmisc@53ff4600 {
 			};
 		};
 
-		emi@80000000 { /* External Memory Interface */
+		emi-bus@80000000 { /* External Memory Interface */
 			compatible = "fsl,emi", "simple-bus";
 			#address-cells = <1>;
 			#size-cells = <1>;

-- 
2.34.1

[PATCH 02/10] ARM: dts: imx35: rename i2c clock-names to ipg

[Frank Li]

Rename the i2c clock-names from "ipg_per" to "ipg" to match the binding
documentation.

Fix the following CHECK_DTBS warning:
  i2c@43f80000 (fsl,imx35-i2c): clock-names:0: 'ipg' was expected

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

diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index 1c010a83d5df138562cd086c7b10d34431378add..aed7fe2fd6b93fe70a0abc7213fe0b874afe579a 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -79,7 +79,7 @@ i2c1: i2c@43f80000 {
 				compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
 				reg = <0x43f80000 0x4000>;
 				clocks = <&clks 51>;
-				clock-names = "ipg_per";
+				clock-names = "ipg";
 				interrupts = <10>;
 				status = "disabled";
 			};
@@ -90,7 +90,7 @@ i2c3: i2c@43f84000 {
 				compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
 				reg = <0x43f84000 0x4000>;
 				clocks = <&clks 53>;
-				clock-names = "ipg_per";
+				clock-names = "ipg";
 				interrupts = <3>;
 				status = "disabled";
 			};
@@ -119,7 +119,7 @@ i2c2: i2c@43f98000 {
 				compatible = "fsl,imx35-i2c", "fsl,imx1-i2c";
 				reg = <0x43f98000 0x4000>;
 				clocks = <&clks 52>;
-				clock-names = "ipg_per";
+				clock-names = "ipg";
 				interrupts = <4>;
 				status = "disabled";
 			};

-- 
2.34.1

[PATCH 03/10] ARM: dts: imx35: remove simple-bus 'usbphy'

[Frank Li]

Remove simple bus 'usbphy' and move chip nodes to up layers to fix below
CHECK_DTBS warnings.
arch/arm/boot/dts/nxp/imx/imx35-pdk.dtb: usbphy (simple-bus): usb-phy@1:reg:0: [1] is too short
   from schema $id: http://devicetree.org/schemas/simple-bus.yaml

Remove property 'reg' because it is never used at driver.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx35.dtsi | 22 +++++++---------------
 1 file changed, 7 insertions(+), 15 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx35.dtsi b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
index aed7fe2fd6b93fe70a0abc7213fe0b874afe579a..ab7b6463998944899a5c49b566e3beca5c039a94 100644
--- a/arch/arm/boot/dts/nxp/imx/imx35.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx35.dtsi
@@ -393,21 +393,13 @@ weim: memory-controller@b8002000 {
 		};
 	};
 
-	usbphy {
-		compatible = "simple-bus";
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		usbphy0: usb-phy@0 {
-			reg = <0>;
-			compatible = "usb-nop-xceiv";
-			#phy-cells = <0>;
-		};
+	usbphy0: usb-phy0 {
+		compatible = "usb-nop-xceiv";
+		#phy-cells = <0>;
+	};
 
-		usbphy1: usb-phy@1 {
-			reg = <1>;
-			compatible = "usb-nop-xceiv";
-			#phy-cells = <0>;
-		};
+	usbphy1: usb-phy1 {
+		compatible = "usb-nop-xceiv";
+		#phy-cells = <0>;
 	};
 };

-- 
2.34.1

[PATCH 04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0

[Frank Li]

Change node name fpga@0 to fpga@0,0 to fix below CHECK_DTBS warnings:
memory-controller@83fda000 (fsl,imx51-weim): 'fpga@0' does not match any of the regexes: '^.*@[0-7],[0-9a-f]+$', '^pinctrl-[0-9]+$'
        from schema $id: http://devicetree.org/schemas/memory-controllers/fsl/fsl,imx-weim.yam

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
index 079bd3d14999f96ef141964a10f492cb2920d9b5..5118a68dbbdc320a895decd7efa2ad5db6a0dd99 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-ts4800.dts
@@ -141,7 +141,7 @@ &weim {
 	pinctrl-0 = <&pinctrl_weim>;
 	status = "okay";
 
-	fpga@0 {
+	fpga-bus@0,0 {
 		compatible = "simple-bus";
 		fsl,weim-cs-timing = <0x0061008F 0x00000002 0x1c022000
 				      0x00000000 0x1c092480 0x00000000>;

-- 
2.34.1

[PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1

[Frank Li]

Rename at45db321d@1 to flash@1 to fix below CHECK_DTBS warnings:
at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not match '^(flash|.*sram|nand)(@.*)?$'
        from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx51-babbage.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
index 1b6ec55f906839f3efde267aaaee035e348f494e..b17264e06e6916057c426179e4207c4a28acf1ab 100644
--- a/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx51-babbage.dts
@@ -327,7 +327,7 @@ vgen3_reg: vgen3 {
 		};
 	};
 
-	flash: at45db321d@1 {
+	flash: flash@1 {
 		#address-cells = <1>;
 		#size-cells = <1>;
 		compatible = "atmel,at45db321d", "atmel,at45", "atmel,dataflash";

-- 
2.34.1

[PATCH 06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052"

[Frank Li]

The "dlg,da9052" compatible string is not fully compatible with
"dlg,da9053-aa". Drop the fallback compatible to fix the following
CHECK_DTBS warning:

  dialog@48 (dlg,da9053-aa): compatible: ['dlg,da9053-aa', 'dlg,da9052'] is too long

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx53-qsb.dts             | 2 +-
 arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts b/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
index 6831836bd726cb759d47821d104595e00c3bfb46..e7e15d2b82c4455d07e246eeae6830d905ae9bf1 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-qsb.dts
@@ -13,7 +13,7 @@ / {
 
 &i2c1 {
 	pmic: dialog@48 {
-		compatible = "dlg,da9053-aa", "dlg,da9052";
+		compatible = "dlg,da9053-aa";
 		reg = <0x48>;
 		interrupt-parent = <&gpio7>;
 		interrupts = <11 IRQ_TYPE_LEVEL_LOW>; /* low-level active IRQ at GPIO7_11 */
diff --git a/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi b/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
index 6dc70a92d831b42b2c1ec8d8eba3839fb45f64f4..6eed683d92a83ec1e59943ce31bba5cf315f0246 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx53-voipac-dmm-668.dtsi
@@ -128,7 +128,7 @@ &i2c1 {
 	status = "okay";
 
 	pmic: dialog@48 {
-		compatible = "dlg,da9053-aa", "dlg,da9052";
+		compatible = "dlg,da9053-aa";
 		reg = <0x48>;
 		interrupt-parent = <&gpio7>;
 		interrupts = <11 IRQ_TYPE_LEVEL_LOW>; /* low-level active IRQ at GPIO7_11 */

-- 
2.34.1

[PATCH 07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency

[Frank Li]

Replace the clock-frequency property with a fixed-clock providing the
32kHz input to the gpio-gate-clock.

Fix the following CHECK_DTBS warning:
  ti-wifi-clock (gpio-gate-clock): 'clock-frequency' does not match any of the regexes: '^pinctrl-[0-9]+$'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
index cd1e682f11add204f373b54b7ad90909fa6bfb5d..8192344d529462b7ee0a926e8650cdb316457961 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sr-som-ti.dtsi
@@ -41,6 +41,12 @@
 #include <dt-bindings/gpio/gpio.h>
 
 / {
+	clk32k: clock-32k {
+		compatible = "fixed-clock";
+		clock-frequency = <32768>;
+		#clock-cells = <0>;
+	};
+
 	nvcc_sd1: regulator-nvcc-sd1 {
 		compatible = "regulator-fixed";
 		regulator-always-on;
@@ -58,7 +64,7 @@ clk_ti_wifi: ti-wifi-clock {
 		 */
 		compatible = "gpio-gate-clock";
 		#clock-cells = <0>;
-		clock-frequency = <32768>;
+		clocks = <&clk32k>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_microsom_ti_clk>;
 		enable-gpios = <&gpio5 5 GPIO_ACTIVE_HIGH>;

-- 
2.34.1

[PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000

[Frank Li]

According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
not a clock provider. Remove the #clock-cells property to fix the following
CHECK_DTBS warning:
  clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
 		smc1: clock-controller@40410000 {
 			compatible = "fsl,imx7ulp-smc1";
 			reg = <0x40410000 0x1000>;
-			#clock-cells = <1>;
 			clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
 				 <&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
 			clock-names = "divcore", "hsrun_divcore";

-- 
2.34.1

[PATCH 09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000

[Frank Li]

Add power supply, #sound-dai-cells and clock  nodes for fsl,sgtl5000 to
fix below CHECK_DTB warnings:

arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): '#sound-dai-cells' is a required property
        from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'clocks' is a required property
        from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'VDDA-supply' is a required property
        from schema $id: http://devicetree.org/schemas/sound/fsl,sgtl5000.yaml#
arch/arm/boot/dts/nxp/imx/imx53-smd.dtb: sgtl5000@a (fsl,sgtl5000): 'VDDIO-supply' is a required property

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/imx/imx53-smd.dts | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
index 386371c816f4d1f2566661e5dd7d02c498ed269b..a1e19f9709b2c7c7d02812f2b5c2afd382fb5d86 100644
--- a/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx53-smd.dts
@@ -31,6 +31,20 @@ key-volume-down {
 			linux,code = <KEY_VOLUMEDOWN>;
 		};
 	};
+
+	reg_1v8: regulator-1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "1v8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
+
+	reg_3v3: regulator-3v3 {
+		compatible = "regulator-fixed";
+		regulator-name = "3v3";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+	};
 };
 
 &esdhc1 {
@@ -253,6 +267,10 @@ &i2c2 {
 	codec: sgtl5000@a {
 		compatible = "fsl,sgtl5000";
 		reg = <0x0a>;
+		#sound-dai-cells = <0>;
+		clocks = <&clks IMX5_CLK_DUMMY>;
+		VDDA-supply = <&reg_1v8>;
+		VDDIO-supply = <&reg_3v3>;
 	};
 
 	magnetometer: mag3110@e {

-- 
2.34.1

[PATCH 10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680

[Frank Li]

The ov2680 only support 1 lane. Needn't additional property to descript it.

Remove it to fix below DTB_CHECK warnings:
  camera@36 (ovti,ov2680): port:endpoint: 'clock-lanes', 'data-lanes' do not match any of the regexes: '^pinctrl-[0-9]+$'
        from schema $id: http://devicetree.org/schemas/media/i2c/ovti,ov2680.yaml

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Previous binding update discussion [1]
[1] https://lore.kernel.org/imx/aPiA3G9x_RsTL9Mv@valkosipuli.retiisi.eu/
---
 arch/arm/boot/dts/nxp/imx/imx7s-warp.dts | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
index 92b6258059ee0150803d960ea076f6301ce1b209..25f38acc53501b0f0fd39446e0e016ec6fb6433d 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
@@ -199,8 +199,6 @@ ov2680: camera@36 {
 		port {
 			ov2680_to_mipi: endpoint {
 				remote-endpoint = <&mipi_from_sensor>;
-				clock-lanes = <0>;
-				data-lanes = <1>;
 				link-frequencies = /bits/ 64 <330000000>;
 			};
 		};

-- 
2.34.1

Re: [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000

[Peng Fan]

On Mon, Feb 02, 2026 at 02:43:25PM -0500, Frank Li wrote:
>According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
>not a clock provider. Remove the #clock-cells property to fix the following
>CHECK_DTBS warning:
>  clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
>
>Signed-off-by: Frank Li <Frank.Li@nxp.com>
>---
> arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
>--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
>@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
> 		smc1: clock-controller@40410000 {
> 			compatible = "fsl,imx7ulp-smc1";
> 			reg = <0x40410000 0x1000>;
>-			#clock-cells = <1>;

Need to fix binding, this is a clock controller and needs #clock-cells to work
properly.

There is a driver for this compatible:
drivers/clk/imx/clk-imx7ulp.c

Regards,
Peng

> 			clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
> 				 <&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
> 			clock-names = "divcore", "hsrun_divcore";
>
>-- 
>2.34.1
>

Re: [PATCH 05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1

[Daniel Baluta]

On Mon, Feb 2, 2026 at 9:45 PM Frank Li <Frank.Li@nxp.com> wrote:
>
> Rename at45db321d@1 to flash@1 to fix below CHECK_DTBS warnings:
> at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not match '^(flash|.*sram|nand)(@.*)?$'
>         from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml
>

The commit message should say *why* the commit is needed and not
focus on fixing the warnings from a tool.

e.g:

ARM: dts: imx51-babbage: Use generic name for flash node

Prefer generic node name 'flash' over the existing specific one to reflect
the function of the device.

This fixes below CHECK_DTBS warning:
 at45db321d@1 (atmel,at45db321d): $nodename:0: 'at45db321d@1' does not
match '^(flash|.*sram|nand)(@.*)?$'
         from schema $id: http://devicetree.org/schemas/mtd/atmel,dataflash.yaml

<snip>

> -       flash: at45db321d@1 {
> +       flash: flash@1 {
>                 #address-cells = <1>;
>                 #size-cells = <1>;
>                 compatible = "atmel,at45db321d", "atmel,at45", "atmel,dataflash";
>

Thanks,
Daniel.

Re: [PATCH 08/10] ARM: dts: imx7ulp: remove #clock-cells from clock-controller@40410000

[Frank Li]

On Tue, Feb 03, 2026 at 09:52:39AM +0800, Peng Fan wrote:
> On Mon, Feb 02, 2026 at 02:43:25PM -0500, Frank Li wrote:
> >According to the fsl,imx7ulp-pm.yaml binding, clock-controller@40410000 is
> >not a clock provider. Remove the #clock-cells property to fix the following
> >CHECK_DTBS warning:
> >  clock-controller@40410000 (fsl,imx7ulp-smc1): '#clock-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
> >
> >Signed-off-by: Frank Li <Frank.Li@nxp.com>
> >---
> > arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi | 1 -
> > 1 file changed, 1 deletion(-)
> >
> >diff --git a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >index 880b9a4f32b0846a773dbf9ad30715c84ac2fda6..6d8853d36d930aabb98fb57f49d6153ca6597561 100644
> >--- a/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >+++ b/arch/arm/boot/dts/nxp/imx/imx7ulp.dtsi
> >@@ -291,7 +291,6 @@ pcc2: clock-controller@403f0000 {
> > 		smc1: clock-controller@40410000 {
> > 			compatible = "fsl,imx7ulp-smc1";
> > 			reg = <0x40410000 0x1000>;
> >-			#clock-cells = <1>;
>
> Need to fix binding, this is a clock controller and needs #clock-cells to work
> properly.

Thanks, strange no one reference label "smc1"

Frank
>
> There is a driver for this compatible:
> drivers/clk/imx/clk-imx7ulp.c
>
> Regards,
> Peng
>
> > 			clocks = <&scg1 IMX7ULP_CLK_CORE_DIV>,
> > 				 <&scg1 IMX7ULP_CLK_HSRUN_CORE_DIV>;
> > 			clock-names = "divcore", "hsrun_divcore";
> >
> >--
> >2.34.1
> >

Re: (subset) [PATCH 00/10] ARM: dts: cleanup some CHECK_DTB warings

[Frank Li]

On Mon, 02 Feb 2026 14:43:17 -0500, Frank Li wrote:
> Cleanup some CHECK_DTB warings.
>
>

Applied, thanks!

[01/10] ARM: dts: imx35: rename emi to emi-bus to fix CHECK_DTBS warning
[02/10] ARM: dts: imx35: rename i2c clock-names to ipg
[03/10] ARM: dts: imx35: remove simple-bus 'usbphy'
[04/10] ARM: dts: imx51-ts4800: rename fpga@0 to fpga@0,0
[05/10] ARM: dts: imx51-babbage: rename at45db321d@1 to flash@1
[06/10] ARM: dts: imx53: drop fallback compatible "dlg,da9052"
[07/10] ARM: dts: imx6qdl-sr-som-ti: use fixed-clock instead of clock-frequency
[09/10] ARM: dts: imx53-smd: Add power supply node for fsl,sgtl5000
[10/10] ARM: dts: imx7s-warp: Remove data-lanes and clock-lanes for ov2680

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