[PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[Frank Li]

clean up most ls1021a CHECK_DTB warning.

Old uboot check esdhc@1560000. The new uboot already switch to check both
esdhc@1560000 and mmc@1560000. So we can rename it now.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v3:
- rebase guoshawn/imx/dt tree. Remove patches that were already merged.
- Link to v2: https://lore.kernel.org/r/20250820-ls1021a_dts_warning-v2-0-2e39648a32b7@nxp.com

Changes in v2:
- squash rename to flash patches
- remove duplicate patches already post in
https://lore.kernel.org/linux-devicetree/20250725061339.266125-1-alexander.stein@ew.tq-group.com/
- Link to v1: https://lore.kernel.org/r/20250818-ls1021a_dts_warning-v1-0-7a79b6b4a0e2@nxp.com

---
Frank Li (4):
      ARM: dts: ls1021a: Rename node name nor to flash
      ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'
      ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000
      ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1

 arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 8 ++++----
 arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts | 2 --
 arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts | 2 +-
 arch/arm/boot/dts/nxp/ls/ls1021a.dtsi    | 2 +-
 4 files changed, 6 insertions(+), 8 deletions(-)
---
base-commit: 75ad5f47c58aee30248d294a58c8ee52e079a8e3
change-id: 20250818-ls1021a_dts_warning-fff933bd83da

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

[PATCH v3 1/4] ARM: dts: ls1021a: Rename node name nor to flash

[Frank Li]

Rename node name nor to flash to fix below CHECK_DTBS warnings:

arch/arm/boot/dts/nxp/ls/ls1021a-twr.dtb: memory-controller@1530000 (fsl,ifc): 'nor@0,0' does not match any of the regexes: '(flash|fpga|board-control|cpld)@[a-f0-9]+(,[a-f0-9]+)+$', '^nand@[a-f0-9]+(,[a-f0-9]+)+$', '^pinctrl-[0-9]+$'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 4 ++--
 arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts b/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
index f1acb97aee69bbcc30c8034e627a0bc6584bf608..8bc8ff2e3b03d6eeb43d79fe0555842536ab735c 100644
--- a/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
+++ b/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
@@ -66,7 +66,7 @@ &dspi0 {
 	bus-num = <0>;
 	status = "okay";
 
-	dspiflash: at45db021d@0 {
+	dspiflash: flash@0 {
 		#address-cells = <1>;
 		#size-cells = <1>;
 		compatible = "atmel,at45db021d", "atmel,at45", "atmel,dataflash";
@@ -187,7 +187,7 @@ &ifc {
 		 <0x3 0x0 0x0 0x7fb00000 0x00000100>;
 	status = "okay";
 
-	nor@0,0 {
+	flash@0,0 {
 		#address-cells = <1>;
 		#size-cells = <1>;
 		compatible = "cfi-flash";
diff --git a/arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts b/arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts
index f5c03871b2050c7cad854afaf59db046640df3c6..38281b904301df6a9d612fdd4384475a68b0cb33 100644
--- a/arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts
+++ b/arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts
@@ -151,7 +151,7 @@ &ifc {
 	ranges = <0x0 0x0 0x0 0x60000000 0x08000000>;
 	status = "okay";
 
-	nor@0,0 {
+	flash@0,0 {
 		#address-cells = <1>;
 		#size-cells = <1>;
 		compatible = "cfi-flash";

-- 
2.34.1

[PATCH v3 2/4] ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'

[Frank Li]

Rename 'mdio-mux-emi1' to 'mdio-mux@54'.

Add fallback compatible string mdio-mux.

Fix below warning:
  arch/arm/boot/dts/nxp/ls/ls1021a-qds.dtb: memory-controller@1530000 (fsl,ifc): board-control@3,0: 'oneOf' conditional failed, one must be fixed:
        'bank-width', 'device-width', 'mdio-mux-emi1' do not match any of the regexes: '^mdio-mux@[a-f0-9,]+$', 'pinctrl-[0-9]+'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts b/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
index 8bc8ff2e3b03d6eeb43d79fe0555842536ab735c..a880875ced83e10c37a4d49d9b9951cd3d458ff0 100644
--- a/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
+++ b/arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts
@@ -211,8 +211,8 @@ fpga: board-control@3,0 {
 		device-width = <1>;
 		ranges = <0 3 0 0x100>;
 
-		mdio-mux-emi1 {
-			compatible = "mdio-mux-mmioreg";
+		mdio-mux@54 {
+			compatible = "mdio-mux-mmioreg", "mdio-mux";
 			mdio-parent-bus = <&mdio0>;
 			#address-cells = <1>;
 			#size-cells = <0>;

-- 
2.34.1

[PATCH v3 3/4] ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000

[Frank Li]

Rename node name esdhc@1560000 to mmc@1560000 to fix below CHECK_DTBS
warnings:
  arch/arm/boot/dts/nxp/ls/ls1021a-tqmls1021a-mbls1021a-rgb-cdtech-fc21.dtb: esdhc@1560000 (fsl,ls1021a-esdhc): $nodename:0: 'esdhc@1560000' does not match '^mmc(@.*)?$'
	from schema $id: http://devicetree.org/schemas/mmc/fsl,esdhc.yaml#

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

diff --git a/arch/arm/boot/dts/nxp/ls/ls1021a.dtsi b/arch/arm/boot/dts/nxp/ls/ls1021a.dtsi
index 29105773add7f4da0ab1c3acee753514eaae4a62..e0b9ea6dd510057875c12037f636672a4e77d035 100644
--- a/arch/arm/boot/dts/nxp/ls/ls1021a.dtsi
+++ b/arch/arm/boot/dts/nxp/ls/ls1021a.dtsi
@@ -154,7 +154,7 @@ qspi: spi@1550000 {
 			status = "disabled";
 		};
 
-		esdhc: esdhc@1560000 {
+		esdhc: mmc@1560000 {
 			compatible = "fsl,ls1021a-esdhc", "fsl,esdhc";
 			reg = <0x0 0x1560000 0x0 0x10000>;
 			interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;

-- 
2.34.1

[PATCH v3 4/4] ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1

[Frank Li]

Remove redundant #address-cells and #size-cells for ethernet-switch@1
because children node have not address.

Fix below CHECK_DTBS warnings:
  arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dtb: ethernet-switch@1 (nxp,sja1105t): Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/dsa/nxp,sja1105.yaml#

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts b/arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts
index 1ea32fff41201b7a787dd116eef675318861ad19..da76566f3510ce866586a214b4df6992e5fc4843 100644
--- a/arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts
+++ b/arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts
@@ -40,8 +40,6 @@ &dspi0 {
 	/* ADG704BRMZ 1:4 SPI mux/demux */
 	sja1105: ethernet-switch@1 {
 		reg = <0x1>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 		compatible = "nxp,sja1105t";
 		/* 12 MHz */
 		spi-max-frequency = <12000000>;

-- 
2.34.1

Re: [PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[Alexander Stein]

Hi Frank,
Am Freitag, 22. August 2025, 16:49:57 CEST schrieb Frank Li:
> clean up most ls1021a CHECK_DTB warning.
> 
> Old uboot check esdhc@1560000. The new uboot already switch to check both
> esdhc@1560000 and mmc@1560000. So we can rename it now.

Please be aware you are not the only vendor. Do you have a link for
the corresponding change, so other can easily keep up?

Best regards,
Alexander

> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Changes in v3:
> - rebase guoshawn/imx/dt tree. Remove patches that were already merged.
> - Link to v2: https://lore.kernel.org/r/20250820-ls1021a_dts_warning-v2-0-2e39648a32b7@nxp.com
> 
> Changes in v2:
> - squash rename to flash patches
> - remove duplicate patches already post in
> https://lore.kernel.org/linux-devicetree/20250725061339.266125-1-alexander.stein@ew.tq-group.com/
> - Link to v1: https://lore.kernel.org/r/20250818-ls1021a_dts_warning-v1-0-7a79b6b4a0e2@nxp.com
> 
> ---
> Frank Li (4):
>       ARM: dts: ls1021a: Rename node name nor to flash
>       ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'
>       ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000
>       ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1
> 
>  arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 8 ++++----
>  arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts | 2 --
>  arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts | 2 +-
>  arch/arm/boot/dts/nxp/ls/ls1021a.dtsi    | 2 +-
>  4 files changed, 6 insertions(+), 8 deletions(-)
> ---
> base-commit: 75ad5f47c58aee30248d294a58c8ee52e079a8e3
> change-id: 20250818-ls1021a_dts_warning-fff933bd83da
> 
> Best regards,
> --
> Frank Li <Frank.Li@nxp.com>
> 
> 
> 


-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/

Re: [PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[Frank Li]

On Mon, Aug 25, 2025 at 08:30:17AM +0200, Alexander Stein wrote:
> Hi Frank,
> Am Freitag, 22. August 2025, 16:49:57 CEST schrieb Frank Li:
> > clean up most ls1021a CHECK_DTB warning.
> >
> > Old uboot check esdhc@1560000. The new uboot already switch to check both
> > esdhc@1560000 and mmc@1560000. So we can rename it now.
>
> Please be aware you are not the only vendor. Do you have a link for
> the corresponding change, so other can easily keep up?

https://github.com/nxp-imx/uboot-imx/commit/83862cd15951489d0eb75a1fa6d0e757142e3ce9

ARM64 platfrom already change to mmc@* for a whole.

Frank


>
> Best regards,
> Alexander
>
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> > Changes in v3:
> > - rebase guoshawn/imx/dt tree. Remove patches that were already merged.
> > - Link to v2: https://lore.kernel.org/r/20250820-ls1021a_dts_warning-v2-0-2e39648a32b7@nxp.com
> >
> > Changes in v2:
> > - squash rename to flash patches
> > - remove duplicate patches already post in
> > https://lore.kernel.org/linux-devicetree/20250725061339.266125-1-alexander.stein@ew.tq-group.com/
> > - Link to v1: https://lore.kernel.org/r/20250818-ls1021a_dts_warning-v1-0-7a79b6b4a0e2@nxp.com
> >
> > ---
> > Frank Li (4):
> >       ARM: dts: ls1021a: Rename node name nor to flash
> >       ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'
> >       ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000
> >       ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1
> >
> >  arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 8 ++++----
> >  arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts | 2 --
> >  arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts | 2 +-
> >  arch/arm/boot/dts/nxp/ls/ls1021a.dtsi    | 2 +-
> >  4 files changed, 6 insertions(+), 8 deletions(-)
> > ---
> > base-commit: 75ad5f47c58aee30248d294a58c8ee52e079a8e3
> > change-id: 20250818-ls1021a_dts_warning-fff933bd83da
> >
> > Best regards,
> > --
> > Frank Li <Frank.Li@nxp.com>
> >
> >
> >
>
>
> --
> TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
> Amtsgericht München, HRB 105018
> Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
> http://www.tq-group.com/
>
>

Re: [PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[Alexander Stein]

Hi,

Am Freitag, 22. August 2025, 16:49:57 CEST schrieb Frank Li:
> clean up most ls1021a CHECK_DTB warning.
> 
> Old uboot check esdhc@1560000. The new uboot already switch to check both
> esdhc@1560000 and mmc@1560000. So we can rename it now.
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

For the whole series:
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>

> ---
> Changes in v3:
> - rebase guoshawn/imx/dt tree. Remove patches that were already merged.
> - Link to v2: https://lore.kernel.org/r/20250820-ls1021a_dts_warning-v2-0-2e39648a32b7@nxp.com
> 
> Changes in v2:
> - squash rename to flash patches
> - remove duplicate patches already post in
> https://lore.kernel.org/linux-devicetree/20250725061339.266125-1-alexander.stein@ew.tq-group.com/
> - Link to v1: https://lore.kernel.org/r/20250818-ls1021a_dts_warning-v1-0-7a79b6b4a0e2@nxp.com
> 
> ---
> Frank Li (4):
>       ARM: dts: ls1021a: Rename node name nor to flash
>       ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'
>       ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000
>       ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1
> 
>  arch/arm/boot/dts/nxp/ls/ls1021a-qds.dts | 8 ++++----
>  arch/arm/boot/dts/nxp/ls/ls1021a-tsn.dts | 2 --
>  arch/arm/boot/dts/nxp/ls/ls1021a-twr.dts | 2 +-
>  arch/arm/boot/dts/nxp/ls/ls1021a.dtsi    | 2 +-
>  4 files changed, 6 insertions(+), 8 deletions(-)
> ---
> base-commit: 75ad5f47c58aee30248d294a58c8ee52e079a8e3
> change-id: 20250818-ls1021a_dts_warning-fff933bd83da
> 
> Best regards,
> --
> Frank Li <Frank.Li@nxp.com>
> 
> 
> 


-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/

Re: [PATCH v3 0/4] ARM: dts: clean up most ls1021a CHECK_DTB warning

[Peng Fan]

On Fri, Aug 22, 2025 at 10:49:57AM -0400, Frank Li wrote:
>clean up most ls1021a CHECK_DTB warning.
>
>Old uboot check esdhc@1560000. The new uboot already switch to check both
>esdhc@1560000 and mmc@1560000. So we can rename it now.
>
>Signed-off-by: Frank Li <Frank.Li@nxp.com>
>---
>Changes in v3:
>- rebase guoshawn/imx/dt tree. Remove patches that were already merged.
>- Link to v2: https://lore.kernel.org/r/20250820-ls1021a_dts_warning-v2-0-2e39648a32b7@nxp.com
>
>Changes in v2:
>- squash rename to flash patches
>- remove duplicate patches already post in
>https://lore.kernel.org/linux-devicetree/20250725061339.266125-1-alexander.stein@ew.tq-group.com/
>- Link to v1: https://lore.kernel.org/r/20250818-ls1021a_dts_warning-v1-0-7a79b6b4a0e2@nxp.com
>
>---
>Frank Li (4):
>      ARM: dts: ls1021a: Rename node name nor to flash
>      ARM: dts: ls1021a: Rename 'mdio-mux-emi1' to 'mdio-mux@54'
>      ARM: dts: ls1021a: Rename esdhc@1560000 to mmc@1560000
>      ARM: dts: ls1021a-tsn: Remove redundant #address-cells for ethernet-switch@1
>


Reviewed-by: Peng Fan <peng.fan@nxp.com>