[PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings

[PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings

[Frank Li]

Cleanup warning for vfxxx.

After apply binding patch
https://lore.kernel.org/imx/20250819165317.3739366-1-Frank.Li@nxp.com/T/#u

All warning under nxp/vf is eliminated.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Frank Li (4):
      ARM: dts: vfxxx: rename nodename iomux to pinctrl
      ARM: dts: vf610: add grp surfix to pinctrl
      ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings
      ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic

 arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts |  8 ++++----
 arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi      | 14 +++++++-------
 arch/arm/boot/dts/nxp/vf/vf610m4.dtsi            |  4 ++++
 arch/arm/boot/dts/nxp/vf/vfxxx.dtsi              |  4 ++--
 4 files changed, 17 insertions(+), 13 deletions(-)
---
base-commit: 9c15bfe5311638391431166b201d964304ecce89
change-id: 20250819-vfxxx_warning-fe8c49e17267

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

[PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl

[Frank Li]

Rename nodename iomux to pinctrl to fix below warnings:
  arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dtb: iomuxc@40048000 (fsl,vf610-iomuxc): $nodename:0: 'iomuxc@40048000' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
	from schema $id: http://devicetree.org/schemas/pinctrl/fsl,vf610-iomuxc.yaml#

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

diff --git a/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi b/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
index 124003c0be26a0cc3b3036b30411f50363117887..19de9506e0c895ee0ba959b4f69424d811f0f064 100644
--- a/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
+++ b/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
@@ -304,7 +304,7 @@ qspi0: spi@40044000 {
 				status = "disabled";
 			};
 
-			iomuxc: iomuxc@40048000 {
+			iomuxc: pinctrl@40048000 {
 				compatible = "fsl,vf610-iomuxc";
 				reg = <0x40048000 0x1000>;
 			};

-- 
2.34.1

[PATCH 2/4] ARM: dts: vf610: add grp surfix to pinctrl

[Frank Li]

Add grp surfix for pinctrl to fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dtb: pinctrl@40048000 (fsl,vf610-iomuxc): i2c0grp-gpio: {'fsl,pins': [[144, 0, 0, 0, 12738, 148, 0, 0, 0, 12738]], 'phandle': 19} is not of type 'array'
	from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml#

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts |  8 ++++----
 arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi      | 14 +++++++-------
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts
index 029f49be40e373f706f7f67c34358ba9272ea0af..be61472393620e54c2fb83abe9fb232c055628ca 100644
--- a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts
+++ b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dts
@@ -412,13 +412,13 @@ &mdio1 {
 };
 
 &iomuxc {
-	pinctrl_gpio_e6185_eeprom_sel: pinctrl-gpio-e6185-eeprom-spi0 {
+	pinctrl_gpio_e6185_eeprom_sel: pinctrl-gpio-e6185-eeprom-spi0-grp {
 		fsl,pins = <
 			VF610_PAD_PTE27__GPIO_132	0x33e2
 		>;
 	};
 
-	pinctrl_gpio_spi0: pinctrl-gpio-spi0 {
+	pinctrl_gpio_spi0: pinctrl-gpio-spi0-grp {
 		fsl,pins = <
 			VF610_PAD_PTB22__GPIO_44	0x33e2
 			VF610_PAD_PTB21__GPIO_43	0x33e2
@@ -428,7 +428,7 @@ VF610_PAD_PTB18__GPIO_40	0x33e2
 		>;
 	};
 
-	pinctrl_mdio_mux: pinctrl-mdio-mux {
+	pinctrl_mdio_mux: pinctrl-mdio-mux-grp {
 		fsl,pins = <
 			VF610_PAD_PTA18__GPIO_8		0x31c2
 			VF610_PAD_PTA19__GPIO_9		0x31c2
@@ -437,7 +437,7 @@ VF610_PAD_PTB3__GPIO_25		0x31c2
 		>;
 	};
 
-	pinctrl_pca9554_22: pinctrl-pca95540-22 {
+	pinctrl_pca9554_22: pinctrl-pca95540-22-grp {
 		fsl,pins = <
 			VF610_PAD_PTB28__GPIO_98	0x219d
 		>;
diff --git a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi
index ce5e52896b19d1ce80c2f34bef160ab57f12885b..91cc496ffb9065c98f539d7f5b553cb3926031aa 100644
--- a/arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi
+++ b/arch/arm/boot/dts/nxp/vf/vf610-zii-dev.dtsi
@@ -335,7 +335,7 @@ VF610_PAD_PTC17__ENET_RMII1_TXEN	0x30d2
 		>;
 	};
 
-	pinctrl_gpio_spi0: pinctrl-gpio-spi0 {
+	pinctrl_gpio_spi0: pinctrl-gpio-spi0-grp {
 		fsl,pins = <
 			VF610_PAD_PTB22__GPIO_44	0x33e2
 			VF610_PAD_PTB21__GPIO_43	0x33e2
@@ -345,19 +345,19 @@ VF610_PAD_PTB18__GPIO_40	0x33e2
 		>;
 	};
 
-	pinctrl_gpio_switch0: pinctrl-gpio-switch0 {
+	pinctrl_gpio_switch0: pinctrl-gpio-switch0-grp {
 		fsl,pins = <
 			VF610_PAD_PTB5__GPIO_27		0x219d
 		>;
 	};
 
-	pinctrl_gpio_switch1: pinctrl-gpio-switch1 {
+	pinctrl_gpio_switch1: pinctrl-gpio-switch1-grp {
 		fsl,pins = <
 			VF610_PAD_PTB4__GPIO_26		0x219d
 		>;
 	};
 
-	pinctrl_i2c_mux_reset: pinctrl-i2c-mux-reset {
+	pinctrl_i2c_mux_reset: pinctrl-i2c-mux-reset-grp {
 		fsl,pins = <
 			 VF610_PAD_PTE14__GPIO_119	0x31c2
 			 >;
@@ -370,7 +370,7 @@ VF610_PAD_PTB15__I2C0_SDA	0x37ff
 		>;
 	};
 
-	pinctrl_i2c0_gpio: i2c0grp-gpio {
+	pinctrl_i2c0_gpio: i2c0-gpio-grp {
 		fsl,pins = <
 			VF610_PAD_PTB14__GPIO_36	0x31c2
 			VF610_PAD_PTB15__GPIO_37	0x31c2
@@ -392,7 +392,7 @@ VF610_PAD_PTA23__I2C2_SDA	0x37ff
 		>;
 	};
 
-	pinctrl_leds_debug: pinctrl-leds-debug {
+	pinctrl_leds_debug: pinctrl-leds-debug-grp {
 		fsl,pins = <
 			 VF610_PAD_PTD20__GPIO_74	0x31c2
 			 >;
@@ -436,7 +436,7 @@ VF610_PAD_PTD22__UART2_RX	0x21a1
 		>;
 	};
 
-	pinctrl_usb_vbus: pinctrl-usb-vbus {
+	pinctrl_usb_vbus: pinctrl-usb-vbus-grp {
 		fsl,pins = <
 			VF610_PAD_PTA16__GPIO_6	0x31c2
 		>;

-- 
2.34.1

[PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings

[Frank Li]

Rename nand@400e0000 to nand-controller@400e0000 to fix below warning:
nand@400e0000: $nodename:0: 'nand@400e0000' does not match '^nand-controller(@.*)?'

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

diff --git a/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi b/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
index 19de9506e0c895ee0ba959b4f69424d811f0f064..568d81807c81559335cbcaa32ab229dfd6e1c795 100644
--- a/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
+++ b/arch/arm/boot/dts/nxp/vf/vfxxx.dtsi
@@ -682,7 +682,7 @@ can1: can@400d4000 {
 				status = "disabled";
 			};
 
-			nfc: nand@400e0000 {
+			nfc: nand-controller@400e0000 {
 				#address-cells = <1>;
 				#size-cells = <0>;
 				compatible = "fsl,vf610-nfc";

-- 
2.34.1

[PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic

[Frank Li]

Add arm,num-irq-priority-bits(4) for nvic to fix below CHECK_DTB warnings:
  arch/arm/boot/dts/nxp/vf/vf610m4-colibri.dtb: interrupt-controller@e000e100 (arm,armv7m-nvic): 'arm,num-irq-priority-bits' is a required property
	from schema $id: http://devicetree.org/schemas/interrupt-controller/arm,nvic.yaml#

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/vf/vf610m4.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi b/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
index 2bb331a87721000880cebeb81dd3c00febfbed3c..ccb5deb4e9528fe005a5691347d4a3852fa52ca3 100644
--- a/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
+++ b/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
@@ -52,6 +52,10 @@ / {
 	aliases { };
 };
 
+&nvic {
+	arm,num-irq-priority-bits = <4>;
+};
+
 &mscm_ir {
 	interrupt-parent = <&nvic>;
 };

-- 
2.34.1

Re: [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl

[Shawn Guo]

On Tue, Aug 19, 2025 at 03:44:06PM -0400, Frank Li wrote:
> Rename nodename iomux to pinctrl to fix below warnings:
>   arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dtb: iomuxc@40048000 (fsl,vf610-iomuxc): $nodename:0: 'iomuxc@40048000' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
> 	from schema $id: http://devicetree.org/schemas/pinctrl/fsl,vf610-iomuxc.yaml#
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Fabio beat you to this one ;)

Shawn

Re: [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings

[Shawn Guo]

On Tue, Aug 19, 2025 at 03:44:08PM -0400, Frank Li wrote:
> Rename nand@400e0000 to nand-controller@400e0000 to fix below warning:
> nand@400e0000: $nodename:0: 'nand@400e0000' does not match '^nand-controller(@.*)?'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Fabio also beat you to this one.

Shawn

Re: [PATCH 2/4] ARM: dts: vf610: add grp surfix to pinctrl

[Shawn Guo]

On Tue, Aug 19, 2025 at 03:44:07PM -0400, Frank Li wrote:
> Add grp surfix for pinctrl to fix below CHECK_DTBS warnings:
>   arch/arm/boot/dts/nxp/vf/vf610-zii-dev-rev-b.dtb: pinctrl@40048000 (fsl,vf610-iomuxc): i2c0grp-gpio: {'fsl,pins': [[144, 0, 0, 0, 12738, 148, 0, 0, 0, 12738]], 'phandle': 19} is not of type 'array'
> 	from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml#
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Applied, thanks!

Re: [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic

[Shawn Guo]

On Tue, Aug 19, 2025 at 03:44:09PM -0400, Frank Li wrote:
> Add arm,num-irq-priority-bits(4) for nvic to fix below CHECK_DTB warnings:
>   arch/arm/boot/dts/nxp/vf/vf610m4-colibri.dtb: interrupt-controller@e000e100 (arm,armv7m-nvic): 'arm,num-irq-priority-bits' is a required property
> 	from schema $id: http://devicetree.org/schemas/interrupt-controller/arm,nvic.yaml#
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm/boot/dts/nxp/vf/vf610m4.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi b/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
> index 2bb331a87721000880cebeb81dd3c00febfbed3c..ccb5deb4e9528fe005a5691347d4a3852fa52ca3 100644
> --- a/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
> +++ b/arch/arm/boot/dts/nxp/vf/vf610m4.dtsi
> @@ -52,6 +52,10 @@ / {
>  	aliases { };
>  };
>  
> +&nvic {
> +	arm,num-irq-priority-bits = <4>;
> +};
> +
>  &mscm_ir {
>  	interrupt-parent = <&nvic>;
>  };

Should probably add it here in name of order.  Fixed it up and applied.

Shawn

> 
> -- 
> 2.34.1
>