[PATCH 0/9] arm64: dts: freescale: 2nrd clean up dtb warning under freescale

[PATCH 0/9] arm64: dts: freescale: 2nrd clean up dtb warning under freescale

[Frank Li]

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Frank Li (9):
      arm64: dts: layerscape: use short name about thermal zone
      arm64: dts: fsl-ls1028a: remove undocumented 'little-endian' for dspi node
      arm64: dts: fsl-ls208xa: move reboot node under syscon
      arm64: dts: imx: add fallback compatible string fsl,imx8mm
      arm64: dts: imx8mm-venice-gw7901: add #address(size)-cells for gsc@20
      arm64: dts: imx8mp-data-modul-edm-sbc: remove #clock-cells for sai3
      arm64: dts: imx8mp-venice-gw74xx-imx219: use gateworks,imx8mp-gw74xx
      arm64: dts: imx8mm: Add fallback compatible string fsl,imx8mm
      arm64: dts: imx8mm-phygate: fix typo pinctrcl-0

 arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi     |  5 +---
 arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi     |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi     |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi     |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi     | 27 +++++++++++-----------
 arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi     |  2 +-
 .../imx8mm-phygate-tauri-l-rs232-rs232.dtso        |  5 ++--
 .../imx8mm-phygate-tauri-l-rs232-rs485.dtso        |  5 ++--
 .../imx8mm-phygate-tauri-l-rs232-rts-cts.dtso      |  3 ++-
 .../imx8mm-venice-gw72xx-0x-rs232-rts.dtso         |  2 +-
 .../freescale/imx8mm-venice-gw72xx-0x-rs422.dtso   |  2 +-
 .../freescale/imx8mm-venice-gw72xx-0x-rs485.dtso   |  2 +-
 .../imx8mm-venice-gw73xx-0x-rs232-rts.dtso         |  2 +-
 .../freescale/imx8mm-venice-gw73xx-0x-rs422.dtso   |  2 +-
 .../freescale/imx8mm-venice-gw73xx-0x-rs485.dtso   |  2 +-
 .../boot/dts/freescale/imx8mm-venice-gw7901.dts    |  2 ++
 .../dts/freescale/imx8mp-data-modul-edm-sbc.dts    |  1 -
 .../dts/freescale/imx8mp-venice-gw74xx-imx219.dtso |  2 +-
 18 files changed, 35 insertions(+), 35 deletions(-)
---
base-commit: 8400291e289ee6b2bf9779ff1c83a291501f017b
change-id: 20240805-fsl_dts_warning-8ea01aca79dd

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

[PATCH 1/9] arm64: dts: layerscape: use short name about thermal zone

[Frank Li]

Cut thermal node name shorter to fixed below warning:
arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-kbox-a-230-ls.dtb:
  thermal-zones: 'core-cluster-thermal' does not match any of the regexes: '^[a-zA-Z][a-zA-Z0-9\\-]{1,10}-thermal$', 'pinctrl-[0-9]+

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 14 +++++++-------
 arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi |  2 +-
 6 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
index 6b6e3ee950e53..82b0bef07034f 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
@@ -175,7 +175,7 @@ ddr-ctrler-crit {
 			};
 		};
 
-		core-cluster-thermal {
+		core-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 1>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index 17f4e31711209..402499f68f47a 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -214,7 +214,7 @@ fman-crit {
 			};
 		};
 
-		core-cluster-thermal {
+		core-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 3>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
index 200e52622f998..f8f45c99507db 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
@@ -182,7 +182,7 @@ fman-crit {
 			};
 		};
 
-		core-cluster-thermal {
+		core-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 3>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
index 8ce4b6aae79d4..1a696f54c0be5 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
@@ -131,7 +131,7 @@ its: msi-controller@6020000 {
 	};
 
 	thermal-zones {
-		core-cluster-thermal {
+		core-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 0>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index bde89de2576e1..2e4dfad637bd9 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -80,7 +80,7 @@ reboot {
 	};
 
 	thermal-zones {
-		ddr-ctrl1-thermal {
+		ddr1-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 1>;
@@ -94,7 +94,7 @@ ddr-ctrler1-crit {
 			};
 		};
 
-		ddr-ctrl2-thermal {
+		ddr2-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 2>;
@@ -108,7 +108,7 @@ ddr-ctrler2-crit {
 			};
 		};
 
-		ddr-ctrl3-thermal {
+		ddr3-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 3>;
@@ -122,7 +122,7 @@ ddr-ctrler3-crit {
 			};
 		};
 
-		core-cluster1-thermal {
+		cluster1-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 4>;
@@ -151,7 +151,7 @@ map0 {
 			};
 		};
 
-		core-cluster2-thermal {
+		cluster2-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 5>;
@@ -180,7 +180,7 @@ map0 {
 			};
 		};
 
-		core-cluster3-thermal {
+		cluster3-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 6>;
@@ -209,7 +209,7 @@ map0 {
 			};
 		};
 
-		core-cluster4-thermal {
+		cluster4-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 7>;
diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
index 26c7ca31e22e7..65d05e5fc010e 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
@@ -492,7 +492,7 @@ map0 {
 			};
 		};
 
-		ddr-cluster5-thermal {
+		ddr5-thermal {
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 			thermal-sensors = <&tmu 1>;

-- 
2.34.1

Re: [PATCH 1/9] arm64: dts: layerscape: use short name about thermal zone

[Krzysztof Kozlowski]

On 05/08/2024 17:49, Frank Li wrote:
> Cut thermal node name shorter to fixed below warning:
> arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-kbox-a-230-ls.dtb:
>   thermal-zones: 'core-cluster-thermal' does not match any of the regexes: '^[a-zA-Z][a-zA-Z0-9\\-]{1,10}-thermal$', 'pinctrl-[0-9]+
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi |  2 +-
>  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi |  2 +-
>  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi |  2 +-
>  arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi |  2 +-
>  arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 14 +++++++-------
>  arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi |  2 +-
>  6 files changed, 12 insertions(+), 12 deletions(-)

I already sent patches for these.

Best regards,
Krzysztof

[PATCH 2/9] arm64: dts: fsl-ls1028a: remove undocumented 'little-endian' for dspi node

[Frank Li]

According to binding doc http://devicetree.org/schemas/spi/fsl,dspi.yaml
and driver drivers/spi/spi-fsl-dspi.c, default is little-endian, should
use big-endian for big-endian system. Remove 'little-endian' to fix below
warning:

arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-85bb.dtb: spi@2100000: Unevaluated properties are not allowed ('little-endian' was unexpected)

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

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
index 82b0bef07034f..1e7d4c0d76a20 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
@@ -381,7 +381,6 @@ dspi0: spi@2100000 {
 			dmas = <&edma0 0 62>, <&edma0 0 60>;
 			dma-names = "tx", "rx";
 			spi-num-chipselects = <4>;
-			little-endian;
 			status = "disabled";
 		};
 
@@ -397,7 +396,6 @@ dspi1: spi@2110000 {
 			dmas = <&edma0 0 58>, <&edma0 0 56>;
 			dma-names = "tx", "rx";
 			spi-num-chipselects = <4>;
-			little-endian;
 			status = "disabled";
 		};
 
@@ -413,7 +411,6 @@ dspi2: spi@2120000 {
 			dmas = <&edma0 0 54>, <&edma0 0 2>;
 			dma-names = "tx", "rx";
 			spi-num-chipselects = <3>;
-			little-endian;
 			status = "disabled";
 		};
 

-- 
2.34.1

[PATCH 3/9] arm64: dts: fsl-ls208xa: move reboot node under syscon

[Frank Li]

Move reboot node under syscon and change compatible string to
fsl,ls1028a-rstcr because it is exactly same as ls1028a.

Fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls2088a-qds.dtb:
 /syscon@1e60000: failed to match any schema with compatible: ['fsl,ls2080a-rstcr', 'syscon']

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

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index 2e4dfad637bd9..d7d519a4e5ead 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -68,15 +68,14 @@ its: msi-controller@6020000 {
 	};
 
 	rstcr: syscon@1e60000 {
-		compatible = "fsl,ls2080a-rstcr", "syscon";
+		compatible = "fsl,ls1028a-reset", "syscon", "simple-mfd";
 		reg = <0x0 0x1e60000 0x0 0x4>;
-	};
 
-	reboot {
-		compatible = "syscon-reboot";
-		regmap = <&rstcr>;
-		offset = <0x0>;
-		mask = <0x2>;
+		reboot {
+			compatible = "syscon-reboot";
+			offset = <0x0>;
+			mask = <0x2>;
+		};
 	};
 
 	thermal-zones {

-- 
2.34.1

[PATCH 4/9] arm64: dts: imx: add fallback compatible string fsl,imx8mm

[Frank Li]

Add fallback compatible string fsl,imx8mm to fix below warning
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
        ['gw,imx8mm-gw72xx-0x'] is too short

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso     | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
index f6ad1a4b8b665..6930b2786afc2 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
@@ -16,7 +16,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw72xx-0x";
+	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
index c3cd9f2b0db34..a2d338ec714c3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
@@ -19,7 +19,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw72xx-0x";
+	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
index cc0a287226ab8..19fff8473970c 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
@@ -19,7 +19,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw72xx-0x";
+	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
index 1f8ea20dfafcb..6092d37e0069e 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
@@ -16,7 +16,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw73xx-0x";
+	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
index 3e6404340d529..5e7b2d639bff3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
@@ -19,7 +19,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw73xx-0x";
+	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
index 2c71ab9854cb3..a1f0e4e85fd7d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
@@ -19,7 +19,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mm-gw73xx-0x";
+	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
 };
 
 &gpio4 {

-- 
2.34.1

Re: [PATCH 4/9] arm64: dts: imx: add fallback compatible string fsl,imx8mm

[Frank Li]

On Mon, Aug 05, 2024 at 11:49:46AM -0400, Frank Li wrote:
> Add fallback compatible string fsl,imx8mm to fix below warning
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
>         ['gw,imx8mm-gw72xx-0x'] is too short
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

I just saw Fabio's patch merge into shawn's dt64.

f384d2828f0d5 arm64: dts: imx8mm-venice-gw72xx-0x: Remove compatible from dtso

This is not needed at all. Please skip this one.

Frank Li

> ---
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso     | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     | 2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> index f6ad1a4b8b665..6930b2786afc2 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> @@ -16,7 +16,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw72xx-0x";
> +	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> index c3cd9f2b0db34..a2d338ec714c3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> @@ -19,7 +19,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw72xx-0x";
> +	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> index cc0a287226ab8..19fff8473970c 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> @@ -19,7 +19,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw72xx-0x";
> +	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> index 1f8ea20dfafcb..6092d37e0069e 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> @@ -16,7 +16,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw73xx-0x";
> +	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> index 3e6404340d529..5e7b2d639bff3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> @@ -19,7 +19,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw73xx-0x";
> +	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> index 2c71ab9854cb3..a1f0e4e85fd7d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> @@ -19,7 +19,7 @@
>  /plugin/;
>
>  &{/} {
> -	compatible = "gw,imx8mm-gw73xx-0x";
> +	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
>  };
>
>  &gpio4 {
>
> --
> 2.34.1
>

[PATCH 5/9] arm64: dts: imx8mm-venice-gw7901: add #address(size)-cells for gsc@20

[Frank Li]

Add #address-cells and #size-cells for gsc@20 to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dtb: gsc@20: '#address-cells' is a required propert

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
index 136cb30df03a6..35ae0faa815bc 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
@@ -364,6 +364,8 @@ 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";

-- 
2.34.1

[PATCH 6/9] arm64: dts: imx8mp-data-modul-edm-sbc: remove #clock-cells for sai3

[Frank Li]

Remove #clock-cells for sai3 because sai3 is not clock controller to fix
below warning:
/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dtb: sai@30c30000: Unevaluated properties are not allowed ('#clock-cells' was unexpected)

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts b/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts
index 7e1b58dbe23a7..837ea79741e8d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dts
@@ -499,7 +499,6 @@ &pwm4 {
 };
 
 &sai3 {
-	#clock-cells = <0>;
 	#sound-dai-cells = <0>;
 	assigned-clocks = <&clk IMX8MP_CLK_SAI3>;
 	assigned-clock-parents = <&clk IMX8MP_AUDIO_PLL1_OUT>;

-- 
2.34.1

[PATCH 7/9] arm64: dts: imx8mp-venice-gw74xx-imx219: use gateworks,imx8mp-gw74xx

[Frank Li]

Replace compatible string gw,imx8mp-gw74xx with documented string
gateworks,imx8mp-gw74xx to fix below warning:
gw,imx8mp-gw74xx' is not one of ['fsl,ls1043a-rdb', 'fsl,ls1043a-qds']

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
index edf22ff549a47..3d2841a1504eb 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
@@ -11,7 +11,7 @@
 /plugin/;
 
 &{/} {
-	compatible = "gw,imx8mp-gw74xx", "fsl,imx8mp";
+	compatible = "gateworks,imx8mp-gw74xx", "fsl,imx8mp";
 
 	reg_cam: regulator-cam {
 		pinctrl-names = "default";

-- 
2.34.1

[PATCH 8/9] arm64: dts: imx8mm: Add fallback compatible string fsl,imx8mm

[Frank Li]

Add fallback compatible string imx8mm-phygate to fix below warning:
'phytec,imx8mm-phygate-tauri-l' is not one of ['armadeus,imx1-apf9328', 'fsl,imx1ads']

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso  | 3 ++-
 arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso  | 3 ++-
 .../arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-cts.dtso | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
index bf3e04651ba00..f246b0ba6af29 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
@@ -15,7 +15,8 @@
 /plugin/;
 
 &{/} {
-	compatible = "phytec,imx8mm-phygate-tauri-l";
+	compatible = "phytec,imx8mm-phygate-tauri-l",
+		     "phytec,imx8mm-phycore-som", "fsl,imx8mm";
 
 };
 
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
index f4448cde0407c..67508ca14276f 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
@@ -16,7 +16,8 @@
 /plugin/;
 
 &{/} {
-	compatible = "phytec,imx8mm-phygate-tauri-l";
+	compatible = "phytec,imx8mm-phygate-tauri-l",
+		     "phytec,imx8mm-phycore-som", "fsl,imx8mm";
 
 };
 
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-cts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-cts.dtso
index 107f743fbb1c3..714e22ccb9c64 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-cts.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-cts.dtso
@@ -16,7 +16,8 @@
 
 
 &{/} {
-	compatible = "phytec,imx8mm-phygate-tauri-l";
+	compatible = "phytec,imx8mm-phygate-tauri-l",
+		     "phytec,imx8mm-phycore-som", "fsl,imx8mm";
 
 };
 

-- 
2.34.1

Re: [PATCH 8/9] arm64: dts: imx8mm: Add fallback compatible string fsl,imx8mm

[Teresa Remmet]

Hello Frank,

Am Montag, dem 05.08.2024 um 11:49 -0400 schrieb Frank Li:
> Add fallback compatible string imx8mm-phygate to fix below warning:
> 'phytec,imx8mm-phygate-tauri-l' is not one of ['armadeus,imx1-
> apf9328', 'fsl,imx1ads']
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso  | 3 ++-
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso  | 3 ++-
>  .../arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-
> cts.dtso | 3 ++-
>  3 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-
> rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-
> tauri-l-rs232-rs232.dtso
> index bf3e04651ba00..f246b0ba6af29 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso
> @@ -15,7 +15,8 @@
>  /plugin/;
>  
>  &{/} {
> -       compatible = "phytec,imx8mm-phygate-tauri-l";
> +       compatible = "phytec,imx8mm-phygate-tauri-l",
> +                    "phytec,imx8mm-phycore-som", "fsl,imx8mm";

should the compatible not better be removed for overlays?

Regards,
Teresa

>  
>  };
>  
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-
> rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-
> tauri-l-rs232-rs485.dtso
> index f4448cde0407c..67508ca14276f 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso
> @@ -16,7 +16,8 @@
>  /plugin/;
>  
>  &{/} {
> -       compatible = "phytec,imx8mm-phygate-tauri-l";
> +       compatible = "phytec,imx8mm-phygate-tauri-l",
> +                    "phytec,imx8mm-phycore-som", "fsl,imx8mm";
>  
>  };
>  
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-
> rs232-rts-cts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-
> tauri-l-rs232-rts-cts.dtso
> index 107f743fbb1c3..714e22ccb9c64 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-
> cts.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rts-
> cts.dtso
> @@ -16,7 +16,8 @@
>  
>  
>  &{/} {
> -       compatible = "phytec,imx8mm-phygate-tauri-l";
> +       compatible = "phytec,imx8mm-phygate-tauri-l",
> +                    "phytec,imx8mm-phycore-som", "fsl,imx8mm";
>  
>  };
>  
> 

-- 
PHYTEC Messtechnik GmbH | Barcelona-Allee 1 | 55129 Mainz, Germany

Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber,
Dipl.-Ing. (FH) Markus Lickes | Handelsregister Mainz HRB 4656 |
Finanzamt Mainz | St.Nr. 266500608, DE 149059855

[PATCH 9/9] arm64: dts: imx8mm-phygate: fix typo pinctrcl-0

[Frank Li]

Fix typo pinctrcl-0 with pinctrl-0.
Fix below warning:

arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb: gpio@30220000: 'pinctrl-0' is a dependency of 'pinctrl-names'
        from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml#
arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb: uart4_rs485_en: $nodename:0: 'uart4_rs485_en' does not match '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
index f246b0ba6af29..ce197266262a5 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
@@ -22,7 +22,7 @@
 
 &gpio3 {
 	pinctrl-names = "default";
-	pinctrcl-0 = <&pinctrl_gpio3_hog>;
+	pinctrl-0 = <&pinctrl_gpio3_hog>;
 
 	uart4_rs485_en {
 		gpio-hog;
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
index 67508ca14276f..f2a7811f1b9f2 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
@@ -23,7 +23,7 @@
 
 &gpio3 {
 	pinctrl-names = "default";
-	pinctrcl-0 = <&pinctrl_gpio3_hog>;
+	pinctrl-0 = <&pinctrl_gpio3_hog>;
 
 	uart4_rs485_en {
 		gpio-hog;

-- 
2.34.1

Re: [PATCH 9/9] arm64: dts: imx8mm-phygate: fix typo pinctrcl-0

[Teresa Remmet]

Am Montag, dem 05.08.2024 um 11:49 -0400 schrieb Frank Li:
> Fix typo pinctrcl-0 with pinctrl-0.
> Fix below warning:
> 
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb:
> gpio@30220000: 'pinctrl-0' is a dependency of 'pinctrl-names'
>         from schema $id:
> http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml#
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb:
> uart4_rs485_en: $nodename:0: 'uart4_rs485_en' does not match '^(hog-
> [0-9]+|.+-hog(-[0-9]+)?)$
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Reviewed-by: Teresa Remmet <t.remmet@phytec.de>

Thanks for fixing this.

Teresa

> ---
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-
> rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-
> tauri-l-rs232-rs232.dtso
> index f246b0ba6af29..ce197266262a5 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs232.dtso
> @@ -22,7 +22,7 @@
>  
>  &gpio3 {
>         pinctrl-names = "default";
> -       pinctrcl-0 = <&pinctrl_gpio3_hog>;
> +       pinctrl-0 = <&pinctrl_gpio3_hog>;
>  
>         uart4_rs485_en {
>                 gpio-hog;
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-
> rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-
> tauri-l-rs232-rs485.dtso
> index 67508ca14276f..f2a7811f1b9f2 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-
> rs485.dtso
> @@ -23,7 +23,7 @@
>  
>  &gpio3 {
>         pinctrl-names = "default";
> -       pinctrcl-0 = <&pinctrl_gpio3_hog>;
> +       pinctrl-0 = <&pinctrl_gpio3_hog>;
>  
>         uart4_rs485_en {
>                 gpio-hog;
> 

-- 
PHYTEC Messtechnik GmbH | Barcelona-Allee 1 | 55129 Mainz, Germany

Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber,
Dipl.-Ing. (FH) Markus Lickes | Handelsregister Mainz HRB 4656 |
Finanzamt Mainz | St.Nr. 266500608, DE 149059855

Re: [PATCH 9/9] arm64: dts: imx8mm-phygate: fix typo pinctrcl-0

[Alexander Stein]

Hi,

Am Montag, 5. August 2024, 17:49:51 CEST schrieb Frank Li:
> Fix typo pinctrcl-0 with pinctrl-0.
> Fix below warning:
> 
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb: gpio@30220000: 'pinctrl-0' is a dependency of 'pinctrl-names'
>         from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml#
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtb: uart4_rs485_en: $nodename:0: 'uart4_rs485_en' does not match '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
> index f246b0ba6af29..ce197266262a5 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso
> @@ -22,7 +22,7 @@
>  
>  &gpio3 {
>  	pinctrl-names = "default";
> -	pinctrcl-0 = <&pinctrl_gpio3_hog>;
> +	pinctrl-0 = <&pinctrl_gpio3_hog>;
>  

I think this commit should contain a Fixes tag.

best regards,
Alexander

>  	uart4_rs485_en {
>  		gpio-hog;
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
> index 67508ca14276f..f2a7811f1b9f2 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso
> @@ -23,7 +23,7 @@
>  
>  &gpio3 {
>  	pinctrl-names = "default";
> -	pinctrcl-0 = <&pinctrl_gpio3_hog>;
> +	pinctrl-0 = <&pinctrl_gpio3_hog>;
>  
>  	uart4_rs485_en {
>  		gpio-hog;
> 
> 


-- 
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/