* [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl
2025-08-19 19:44 [PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings Frank Li
@ 2025-08-19 19:44 ` Frank Li
2025-08-22 9:38 ` Shawn Guo
2025-08-19 19:44 ` [PATCH 2/4] ARM: dts: vf610: add grp surfix " Frank Li
` (2 subsequent siblings)
3 siblings, 1 reply; 9+ messages in thread
From: Frank Li @ 2025-08-19 19:44 UTC (permalink / raw)
To: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, imx, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl
2025-08-19 19:44 ` [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl Frank Li
@ 2025-08-22 9:38 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2025-08-22 9:38 UTC (permalink / raw)
To: Frank Li
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
devicetree, linux-kernel, imx
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/4] ARM: dts: vf610: add grp surfix to pinctrl
2025-08-19 19:44 [PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings Frank Li
2025-08-19 19:44 ` [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl Frank Li
@ 2025-08-19 19:44 ` Frank Li
2025-08-22 9:40 ` Shawn Guo
2025-08-19 19:44 ` [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings Frank Li
2025-08-19 19:44 ` [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic Frank Li
3 siblings, 1 reply; 9+ messages in thread
From: Frank Li @ 2025-08-19 19:44 UTC (permalink / raw)
To: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, imx, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 2/4] ARM: dts: vf610: add grp surfix to pinctrl
2025-08-19 19:44 ` [PATCH 2/4] ARM: dts: vf610: add grp surfix " Frank Li
@ 2025-08-22 9:40 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2025-08-22 9:40 UTC (permalink / raw)
To: Frank Li
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
devicetree, linux-kernel, imx
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!
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings
2025-08-19 19:44 [PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings Frank Li
2025-08-19 19:44 ` [PATCH 1/4] ARM: dts: vfxxx: rename nodename iomux to pinctrl Frank Li
2025-08-19 19:44 ` [PATCH 2/4] ARM: dts: vf610: add grp surfix " Frank Li
@ 2025-08-19 19:44 ` Frank Li
2025-08-22 9:40 ` Shawn Guo
2025-08-19 19:44 ` [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic Frank Li
3 siblings, 1 reply; 9+ messages in thread
From: Frank Li @ 2025-08-19 19:44 UTC (permalink / raw)
To: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, imx, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings
2025-08-19 19:44 ` [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings Frank Li
@ 2025-08-22 9:40 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2025-08-22 9:40 UTC (permalink / raw)
To: Frank Li
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
devicetree, linux-kernel, imx
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic
2025-08-19 19:44 [PATCH 0/4] ARM: dts: vf: fix CHECK_DTB warnings Frank Li
` (2 preceding siblings ...)
2025-08-19 19:44 ` [PATCH 3/4] ARM: dts: vfxxx: rename nand to nand-controller to fix DTB_CHECK warnings Frank Li
@ 2025-08-19 19:44 ` Frank Li
2025-08-22 9:41 ` Shawn Guo
3 siblings, 1 reply; 9+ messages in thread
From: Frank Li @ 2025-08-19 19:44 UTC (permalink / raw)
To: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-kernel, devicetree, linux-kernel, imx, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic
2025-08-19 19:44 ` [PATCH 4/4] ARM: dts: vfxxx: add arm,num-irq-priority-bits for nvic Frank Li
@ 2025-08-22 9:41 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2025-08-22 9:41 UTC (permalink / raw)
To: Frank Li
Cc: Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Stefan Agner,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-kernel,
devicetree, linux-kernel, imx
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
>
^ permalink raw reply [flat|nested] 9+ messages in thread