[PATCH 00/17] arm64: dts: imx: 3rd dtb warning clearn up

[PATCH 00/17] arm64: dts: imx: 3rd dtb warning clearn up

[Frank Li]

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Frank Li (17):
      arm64: dts: imx8-ss-img: remove undocument slot for jpeg
      arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma"
      arm64: dts: fsl-ls1088a: rename gpio admin_led_lower to admin-led-lower-hog
      arm64: dts: imx8mm-venice: rename rs485_en as rs485-en-hog
      arm64: dts: imx8mm-phygate: rename uart4_rs485_en to uart4-rs485-en-hog
      arm64: dts: layerscape: remove undocument big-endian in watchdog
      arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node
      arm64: dts: fsl-ls1046a-qds: rename mdio-mux-emi to mdio-mux-emi@54
      arm64: dts: fsl-ls1028: add missed supply for wm8904
      arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20
      arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub*
      arm64: dts: imx8mp-venice: add vddl and vana for sensor@10
      arm64: dts: fsl-ls1088a-ten64: change to low case hex value
      arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic
      arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb
      arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply
      arm64: dts: imx8mm-beacon-kit: reorder reg-names cec and edid of hdmi@3d

 arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts       | 12 ++++++------
 arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi           |  1 -
 .../dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts | 12 ++++++++++++
 arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi           |  3 +--
 arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts        |  2 +-
 arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts      |  6 +++---
 .../dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts   |  4 ++--
 arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi           |  2 --
 .../boot/dts/freescale/imx8mm-beacon-baseboard.dtsi      | 16 ++++++++++++++++
 arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts      |  4 ++--
 .../freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtso    |  2 +-
 .../freescale/imx8mm-phygate-tauri-l-rs232-rs485.dtso    |  2 +-
 .../dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso    | 16 ++++++++++++++++
 .../dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso |  2 +-
 .../dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso     |  4 ++--
 .../dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     |  4 ++--
 .../dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso    | 16 ++++++++++++++++
 .../dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso |  2 +-
 .../dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     |  4 ++--
 .../dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     |  4 ++--
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts   |  2 ++
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts   |  2 ++
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts   |  2 ++
 .../boot/dts/freescale/imx8mn-beacon-baseboard.dtsi      | 16 ++++++++++++++++
 arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts      |  4 ++--
 arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts   |  2 ++
 .../boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso  | 16 ++++++++++++++++
 .../boot/dts/freescale/imx93-tqma9352-mba93xxca.dts      |  2 +-
 .../boot/dts/freescale/imx93-tqma9352-mba93xxla.dts      |  2 +-
 arch/arm64/boot/dts/freescale/s32v234.dtsi               |  2 +-
 30 files changed, 132 insertions(+), 36 deletions(-)
---
base-commit: 6dd6ca44177f8413f10dbf63b17acbca99beae7d
change-id: 20240812-imx_warning-7bad8223a128

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

[PATCH 01/17] arm64: dts: imx8-ss-img: remove undocument slot for jpeg

[Frank Li]

Remove undocument 'slot' property. It is default 0 at driver. We can remove
it safely.

Fix below warning:
arch/arm64/boot/dts/freescale/imx8dx-colibri-iris.dtb: jpegdec@58400000:
	'slot' does not match any of the regexes: 'pinctrl-[0-9]+'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Previous discuss at thread
- https://lore.kernel.org/imx/e3d64b07-98fc-4d43-a1ef-561cbac58552@nxp.com/
- https://lore.kernel.org/all/bbb1875b-7980-46aa-80b4-dbaf2a2d5755@linaro.org/

Slot should be managed by driver self. So remove "slot" in dts file.
---
 arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
index 77d2928997b4b..d39242c1b9f79 100644
--- a/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi
@@ -26,7 +26,6 @@ jpegdec: jpegdec@58400000 {
 		assigned-clock-rates = <200000000>, <200000000>;
 		power-domains = <&pd IMX_SC_R_MJPEG_DEC_MP>,
 				<&pd IMX_SC_R_MJPEG_DEC_S0>;
-		slot = <0>;
 	};
 
 	jpegenc: jpegenc@58450000 {
@@ -39,7 +38,6 @@ jpegenc: jpegenc@58450000 {
 		assigned-clock-rates = <200000000>, <200000000>;
 		power-domains = <&pd IMX_SC_R_MJPEG_ENC_MP>,
 				<&pd IMX_SC_R_MJPEG_ENC_S0>;
-		slot = <0>;
 	};
 
 	img_jpeg_dec_lpcg: clock-controller@585d0000 {

-- 
2.34.1

[PATCH 02/17] arm64: dts: fsl-ls1043a: move "fsl,ls1043a-qdma" ahead "fsl,ls1021a-qdma"

[Frank Li]

Move fsl,ls1043a-qdma ahead fallback string fsl,ls1021a-qdma.

Fix warning:
dma-controller@8380000: compatible: 'oneOf' conditional failed, one must be fixed:
        ['fsl,ls1021a-qdma', 'fsl,ls1043a-qdma'] is too long

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

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index fc38a0e543ab8..c0e3e8fa1e794 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -955,7 +955,7 @@ pcie3: pcie@3600000 {
 		};
 
 		qdma: dma-controller@8380000 {
-			compatible = "fsl,ls1021a-qdma", "fsl,ls1043a-qdma";
+			compatible = "fsl,ls1043a-qdma", "fsl,ls1021a-qdma";
 			reg = <0x0 0x8380000 0x0 0x1000>, /* Controller regs */
 			      <0x0 0x8390000 0x0 0x10000>, /* Status regs */
 			      <0x0 0x83a0000 0x0 0x40000>; /* Block regs */

-- 
2.34.1

[PATCH 03/17] arm64: dts: fsl-ls1088a: rename gpio admin_led_lower to admin-led-lower-hog

[Frank Li]

Rename node name admin_led_lower to admin-led-lower-hog to fix warning:
arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: gpio@76: 'admin_led_lower' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
index d4867d6cf47cd..57f2082143c8a 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
@@ -220,7 +220,7 @@ sfpgpio: gpio@76 {
 		#gpio-cells = <2>;
 		gpio-controller;
 
-		admin_led_lower {
+		admin-led-lower-hog {
 			gpio-hog;
 			gpios = <13 GPIO_ACTIVE_HIGH>;
 			output-low;

-- 
2.34.1

[PATCH 04/17] arm64: dts: imx8mm-venice: rename rs485_en as rs485-en-hog

[Frank Li]

Rename gpio rs485_en as rs485-en-hog to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb:
  gpio@30230000: 'rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'

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     | 4 ++--
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     | 4 ++--
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     | 4 ++--
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     | 4 ++--
 6 files changed, 10 insertions(+), 10 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 47d3c0c49e8a0..bb2056746f8c9 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/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-low;
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 7fcd8c851159a..45ac8bdce8699 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,14 +19,14 @@
 /plugin/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-high;
 		line-name = "rs485_en";
 	};
 
-	rs485_hd {
+	rs485-hd-hog {
 		gpio-hog;
 		gpios = <2 GPIO_ACTIVE_HIGH>;
 		output-low;
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 b19e38fc27bae..30aa620d7004d 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,14 +19,14 @@
 /plugin/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-high;
 		line-name = "rs485_en";
 	};
 
-	rs485_hd {
+	rs485-hd-hog {
 		gpio-hog;
 		gpios = <2 GPIO_ACTIVE_HIGH>;
 		output-high;
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 c0f6aece844a0..b1a9f35e1dfa7 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/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-low;
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 e909cbd8db4bc..44ebc0a58c51a 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,14 +19,14 @@
 /plugin/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-high;
 		line-name = "rs485_en";
 	};
 
-	rs485_hd {
+	rs485-hd-hog {
 		gpio-hog;
 		gpios = <2 GPIO_ACTIVE_HIGH>;
 		output-low;
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 1508cb5175b3e..2f8a7ac408739 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,14 +19,14 @@
 /plugin/;
 
 &gpio4 {
-	rs485_en {
+	rs485-en-hog {
 		gpio-hog;
 		gpios = <0 GPIO_ACTIVE_HIGH>;
 		output-high;
 		line-name = "rs485_en";
 	};
 
-	rs485_hd {
+	rs485-hd-hog {
 		gpio-hog;
 		gpios = <2 GPIO_ACTIVE_HIGH>;
 		output-high;

-- 
2.34.1

[PATCH 05/17] arm64: dts: imx8mm-phygate: rename uart4_rs485_en to uart4-rs485-en-hog

[Frank Li]

Rename gpio uart4_rs485_en to uart4-rs485-en-hog to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtb:
  gpio@30220000: 'uart4_rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[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 f1f38b739ef76..78f4e8d5814da 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
@@ -18,7 +18,7 @@ &gpio3 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_gpio3_hog>;
 
-	uart4_rs485_en {
+	uart4-rs485-en-hog {
 		gpio-hog;
 		gpios = <20 GPIO_ACTIVE_HIGH>;
 		output-low;
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 1d8951e1a47e8..66288948bdd39 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
@@ -19,7 +19,7 @@ &gpio3 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_gpio3_hog>;
 
-	uart4_rs485_en {
+	uart4-rs485-en-hog {
 		gpio-hog;
 		gpios = <20 GPIO_ACTIVE_HIGH>;
 		output-high;

-- 
2.34.1

[PATCH 06/17] arm64: dts: layerscape: remove undocument big-endian in watchdog

[Frank Li]

Remove undocument property big-endian in watchdog.
drivers/watchdog/imx2_wdt.c also never parser it.

Fix below wanring:
arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb: watchdog@2ad0000:
	Unevaluated properties are not allowed ('big-endian' was unexpected)

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

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
index a3c57da63a01b..b84512bc4c881 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
@@ -430,7 +430,6 @@ wdog0: watchdog@2ad0000 {
 			reg = <0x0 0x2ad0000 0x0 0x10000>;
 			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL QORIQ_CLK_PLL_DIV(1)>;
-			big-endian;
 		};
 
 		sai1: sai@2b50000 {
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
index c0e3e8fa1e794..5837f1b15dccb 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
@@ -776,7 +776,6 @@ wdog0: watchdog@2ad0000 {
 			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
 					    QORIQ_CLK_PLL_DIV(1)>;
-			big-endian;
 		};
 
 		edma0: dma-controller@2c00000 {

-- 
2.34.1

[PATCH 07/17] arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node

[Frank Li]

Move fixed clock "clock-sc16is7x" from dspi to root node to fix below
warning:
arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb:
    serial@0: Unevaluated properties are not allowed ('clock-sc16is7xx' was unexpected)

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

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
index 2517528f684fe..75081ce3e9a6f 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
@@ -20,6 +20,12 @@ sys_mclk: clock-mclk {
 		clock-frequency = <25000000>;
 	};
 
+	sc16is7xx_clk: clock-sc16is7xx {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <24000000>;
+	};
+
 	reg_1p8v: regulator-1p8v {
 		compatible = "regulator-fixed";
 		regulator-name = "1P8V";
@@ -69,12 +75,6 @@ serial@0 {
 		clocks = <&sc16is7xx_clk>;
 		interrupt-parent = <&gpio1>;
 		interrupts = <13 IRQ_TYPE_EDGE_FALLING>;
-
-		sc16is7xx_clk: clock-sc16is7xx {
-			compatible = "fixed-clock";
-			#clock-cells = <0>;
-			clock-frequency = <24000000>;
-		};
 	};
 };
 

-- 
2.34.1

[PATCH 08/17] arm64: dts: fsl-ls1046a-qds: rename mdio-mux-emi to mdio-mux-emi@54

[Frank Li]

mdio-mux-emi have reg property. Rename it as mdio-mux-emi@54.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
index 25eb1d5a69381..5c504ff47f2ca 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
@@ -237,7 +237,7 @@ &fpga {
 	#address-cells = <1>;
 	#size-cells = <1>;
 
-	mdio-mux-emi1 {
+	mdio-mux-emi1@54 {
 		compatible = "mdio-mux-mmioreg", "mdio-mux";
 		mdio-parent-bus = <&mdio0>;
 		#address-cells = <1>;

-- 
2.34.1

[PATCH 09/17] arm64: dts: fsl-ls1028: add missed supply for wm8904

[Frank Li]

Add A(CP, DB, DC, MIC)VDD-supply for wm8904 to fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dtb:
	audio-codec@1a: 'AVDD-supply' is a required property

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 .../dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts     | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
index 195bdbafdf7c9..d9fac647f4327 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts
@@ -26,6 +26,13 @@ pwm-fan {
 		cooling-levels = <1 128 192 255>;
 	};
 
+	reg_3p3v: regulator-3p3v {
+		compatible = "regulator-fixed";
+		regulator-name = "3P3V";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+	};
+
 	sound {
 		#address-cells = <1>;
 		#size-cells = <0>;
@@ -107,6 +114,11 @@ wm8904: audio-codec@1a {
 		clock-names = "mclk";
 		assigned-clocks = <&mclk>;
 		assigned-clock-rates = <1250000>;
+		AVDD-supply = <&reg_3p3v>;
+		CPVDD-supply = <&reg_3p3v>;
+		DBVDD-supply = <&reg_3p3v>;
+		DCVDD-supply = <&reg_3p3v>;
+		MICVDD-supply = <&reg_3p3v>;
 	};
 };
 

-- 
2.34.1

[PATCH 10/17] arm64: dts: imx8mm-venice-gw7902(3): add #address-cells for gsc@20

[Frank Li]

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

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts | 2 ++
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts | 2 ++
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts | 2 ++
 arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts | 2 ++
 4 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
index 1d56f2a6c06a5..c11260c26d0b4 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
@@ -314,6 +314,8 @@ gsc: gsc@20 {
 		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
+		#address-cells = <1>;
+		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
index 45470160f98f5..db1737bf637df 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
@@ -280,6 +280,8 @@ gsc: gsc@20 {
 		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
+		#address-cells = <1>;
+		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
index ef951bc9f0dd4..05489a31e7fd8 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
@@ -330,6 +330,8 @@ gsc: gsc@20 {
 		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
+		#address-cells = <1>;
+		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
index 72004ab6bda55..0b1fa04f1d678 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
@@ -312,6 +312,8 @@ gsc: gsc@20 {
 		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
 		interrupt-controller;
 		#interrupt-cells = <1>;
+		#address-cells = <1>;
+		#size-cells = <0>;
 
 		adc {
 			compatible = "gw,gsc-adc";

-- 
2.34.1

[PATCH 11/17] arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub*

[Frank Li]

According to binging doc usb/ti,usb8041.yaml and in
drivers/usb/misc/onboard_usb_dev.h

ti_tusb8041_data = { .supply_names = { "vdd" },};

It should vdd-supply instead vcc-supply.

Fixes: 04b77e0124ef ("arm64: dts: freescale: add fsl-lx2160a-mblx2160a board")
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
index da0f58e26b9aa..f6a4f8d543015 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
@@ -320,7 +320,7 @@ hub_2_0: hub@1 {
 		reg = <1>;
 		peer-hub = <&hub_3_0>;
 		reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
-		vcc-supply = <&reg_vcc3v3>;
+		vdd-supply = <&reg_vcc3v3>;
 	};
 
 	hub_3_0: hub@2 {
@@ -328,7 +328,7 @@ hub_3_0: hub@2 {
 		reg = <2>;
 		peer-hub = <&hub_2_0>;
 		reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
-		vcc-supply = <&reg_vcc3v3>;
+		vdd-supply = <&reg_vcc3v3>;
 	};
 };
 

-- 
2.34.1

[PATCH 12/17] arm64: dts: imx8mp-venice: add vddl and vana for sensor@10

[Frank Li]

Add vddl and vana for sensor@10 to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtb:
	sensor@10: 'VANA-supply' is a required property

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 .../dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso    | 16 ++++++++++++++++
 .../dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso    | 16 ++++++++++++++++
 .../boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso  | 16 ++++++++++++++++
 3 files changed, 48 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
index 4eaf8aabcbfff..c09aa80d2ba23 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-imx219.dtso
@@ -13,6 +13,20 @@
 &{/} {
 	compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
 
+	reg_vana: regulator-2p8v {
+		compatible = "regulator-fixed";
+		regulator-name = "2P8V";
+		regulator-min-microvolt = <2800000>;
+		regulator-max-microvolt = <2800000>;
+	};
+
+	reg_vddl: regulator-1p2v {
+		compatible = "regulator-fixed";
+		regulator-name = "1P2V";
+		regulator-min-microvolt = <1200000>;
+		regulator-max-microvolt = <1200000>;
+	};
+
 	reg_cam: regulator-cam {
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_reg_cam>;
@@ -45,6 +59,8 @@ imx219: sensor@10 {
 		reg = <0x10>;
 		clocks = <&cam24m>;
 		VDIG-supply = <&reg_cam>;
+		VANA-supply = <&reg_vana>;
+		VDDL-supply = <&reg_vddl>;
 
 		port {
 			/* MIPI CSI-2 bus endpoint */
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
index f3ece4b7fbbde..cfc014eb038d6 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-imx219.dtso
@@ -13,6 +13,20 @@
 &{/} {
 	compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
 
+	reg_vana: regulator-2p8v {
+		compatible = "regulator-fixed";
+		regulator-name = "2P8V";
+		regulator-min-microvolt = <2800000>;
+		regulator-max-microvolt = <2800000>;
+	};
+
+	reg_vddl: regulator-1p2v {
+		compatible = "regulator-fixed";
+		regulator-name = "1P2V";
+		regulator-min-microvolt = <1200000>;
+		regulator-max-microvolt = <1200000>;
+	};
+
 	reg_cam: regulator-cam {
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_reg_cam>;
@@ -45,6 +59,8 @@ imx219: sensor@10 {
 		reg = <0x10>;
 		clocks = <&cam24m>;
 		VDIG-supply = <&reg_cam>;
+		VANA-supply = <&reg_vana>;
+		VDDL-supply = <&reg_vddl>;
 
 		port {
 			/* MIPI CSI-2 bus endpoint */
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 eb673a947484a..7d9fcdee58a7f 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx-imx219.dtso
@@ -11,6 +11,20 @@
 /plugin/;
 
 &{/} {
+	reg_vana: regulator-2p8v {
+		compatible = "regulator-fixed";
+		regulator-name = "2P8V";
+		regulator-min-microvolt = <2800000>;
+		regulator-max-microvolt = <2800000>;
+	};
+
+	reg_vddl: regulator-1p2v {
+		compatible = "regulator-fixed";
+		regulator-name = "1P2V";
+		regulator-min-microvolt = <1200000>;
+		regulator-max-microvolt = <1200000>;
+	};
+
 	reg_cam: regulator-cam {
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_reg_cam>;
@@ -39,6 +53,8 @@ imx219: sensor@10 {
 		reg = <0x10>;
 		clocks = <&cam24m>;
 		VDIG-supply = <&reg_cam>;
+		VANA-supply = <&reg_vana>;
+		VDDL-supply = <&reg_vddl>;
 
 		port {
 			/* MIPI CSI-2 bus endpoint */

-- 
2.34.1

[PATCH 13/17] arm64: dts: fsl-ls1088a-ten64: change to low case hex value

[Frank Li]

Change to low case for hex value to fix below warning:

arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: flash@0:
    partitions: Unevaluated properties are not allowed ('partition@5C0000' was unexpected)

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
index 57f2082143c8a..bc0d89427fbe5 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
@@ -323,9 +323,9 @@ partition@580000 {
 				reg = <0x580000 0x40000>;
 			};
 
-			partition@5C0000 {
+			partition@5c0000 {
 				label = "dpc";
-				reg = <0x5C0000 0x40000>;
+				reg = <0x5c0000 0x40000>;
 			};
 
 			partition@600000 {

-- 
2.34.1

[PATCH 14/17] arm64: dts: s32v234: remove fallback compatible string arm,cortex-a9-gic

[Frank Li]

Remove fallback comaptible string arm,cortex-a9-gic to fix below warning:

/home/lizhi/source/linux-upstream-dts/arch/arm64/boot/dts/freescale/s32v234-evb.dtb: interrupt-controller@7d001000: compatible: 'oneOf' conditional failed, one must be fixed:
            ['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
            /home/lizhi/source/linux-upstream-dts/arch/arm64/boot/dts/freescale/s32v234-evb.dtb: interrupt-controller@7d001000: compatible: 'oneOf' conditional failed, one must be fixed:
                    ['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
                    'nvidia,tegra210-agic' was expected
                    'arm,cortex-a15-gic' is not one of ['nvidia,tegra186-agic', 'nvidia,tegra194-agic', 'nvidia,tegra234-agic']
            'arm,gic-400' was expected

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

diff --git a/arch/arm64/boot/dts/freescale/s32v234.dtsi b/arch/arm64/boot/dts/freescale/s32v234.dtsi
index 42409ec567925..bf608ded5dda6 100644
--- a/arch/arm64/boot/dts/freescale/s32v234.dtsi
+++ b/arch/arm64/boot/dts/freescale/s32v234.dtsi
@@ -89,7 +89,7 @@ IRQ_TYPE_LEVEL_LOW)>,
 	};
 
 	gic: interrupt-controller@7d001000 {
-		compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic";
+		compatible = "arm,cortex-a15-gic";
 		#interrupt-cells = <3>;
 		#address-cells = <0>;
 		interrupt-controller;

-- 
2.34.1

[PATCH 15/17] arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb

[Frank Li]

Rename hub to usb to fix below warning:
arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb: hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
 arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
index 852dd3d2eac74..325d8097bf7fc 100644
--- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
@@ -533,7 +533,7 @@ &usbotg2 {
 	samsung,picophy-dc-vol-level-adjust = <7>;
 	status = "okay";
 
-	hub_2_0: hub@1 {
+	hub_2_0: usb@1 {
 		compatible = "usb424,2517";
 		reg = <1>;
 		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
index da8f19a646a98..b1fcbc12bd882 100644
--- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
@@ -486,7 +486,7 @@ &usbotg2 {
 	samsung,picophy-dc-vol-level-adjust = <7>;
 	status = "okay";
 
-	hub_2_0: hub@1 {
+	hub_2_0: usb@1 {
 		compatible = "usb424,2517";
 		reg = <1>;
 		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;

-- 
2.34.1

[PATCH 16/17] arm64: dts: imx8mm-beacon-kit: add DVDD-supply and DOVDD-supply

[Frank Li]

According to binding doc, DVDD-supply and DOVDD-supply is required
properties. Add these to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: camera@10: 'DVDD-supply' is a required proper

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 .../boot/dts/freescale/imx8mm-beacon-baseboard.dtsi      | 16 ++++++++++++++++
 .../boot/dts/freescale/imx8mn-beacon-baseboard.dtsi      | 16 ++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
index 6086dae2e5fbe..ea1d5b9c6bae0 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi
@@ -56,6 +56,20 @@ pcie0_refclk_gated:  pcie0-refclk-gated {
 		enable-gpios = <&pca6416_1 2 GPIO_ACTIVE_LOW>;
 	};
 
+	reg_1v5: regulator-1v5 {
+		compatible = "regulator-fixed";
+		regulator-name = "1V5";
+		regulator-min-microvolt = <1500000>;
+		regulator-max-microvolt = <1500000>;
+	};
+
+	reg_1v8: regulator-1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "1V8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
+
 	reg_audio: regulator-audio {
 		compatible = "regulator-fixed";
 		regulator-name = "3v3_aud";
@@ -187,6 +201,8 @@ camera@10 {
 		assigned-clock-parents = <&clk IMX8MM_CLK_24M>;
 		assigned-clock-rates = <24000000>;
 		AVDD-supply = <&reg_camera>;  /* 2.8v */
+		DVDD-supply = <&reg_1v5>;
+		DOVDD-supply = <&reg_1v8>;
 		powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
 		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
 
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
index 20018ee2c803e..77d14ea459e57 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-baseboard.dtsi
@@ -40,6 +40,20 @@ led-3 {
 		};
 	};
 
+	reg_1v5: regulator-1v5 {
+		compatible = "regulator-fixed";
+		regulator-name = "1V5";
+		regulator-min-microvolt = <1500000>;
+		regulator-max-microvolt = <1500000>;
+	};
+
+	reg_1v8: regulator-1v8 {
+		compatible = "regulator-fixed";
+		regulator-name = "1V8";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
+
 	reg_audio: regulator-audio {
 		compatible = "regulator-fixed";
 		regulator-name = "3v3_aud";
@@ -158,6 +172,8 @@ camera@10 {
 		assigned-clock-parents = <&clk IMX8MN_CLK_24M>;
 		assigned-clock-rates = <24000000>;
 		AVDD-supply = <&reg_camera>;  /* 2.8v */
+		DVDD-supply = <&reg_1v5>;
+		DOVDD-supply = <&reg_1v8>;
 		powerdown-gpios = <&gpio1 7 GPIO_ACTIVE_HIGH>;
 		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
 

-- 
2.34.1

[PATCH 17/17] arm64: dts: imx8mm-beacon-kit: reorder reg-names cec and edid of hdmi@3d

[Frank Li]

Change reg-names order to fix below warning:
arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: hdmi@3d: reg-names:1: 'edid' was expected
        from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7533.yaml#
arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dtb: hdmi@3d: reg-names:2: 'cec' was expected
        from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7533.yaml#

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts | 4 ++--
 arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
index 905c98cb080d2..97ff1ddd63188 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
@@ -62,8 +62,8 @@ adv_bridge: hdmi@3d {
 		compatible = "adi,adv7535";
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_hdmi_bridge>;
-		reg = <0x3d>, <0x3c>, <0x3e>, <0x3f>;
-		reg-names = "main", "cec", "edid", "packet";
+		reg = <0x3d>, <0x3e>, <0x3c>, <0x3f>;
+		reg-names = "main", "edid", "cec", "packet";
 		adi,dsi-lanes = <4>;
 		avdd-supply = <&reg_hdmi>;
 		a2vdd-supply = <&reg_hdmi>;
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
index bbd80896db964..1df5ceb113879 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
@@ -62,8 +62,8 @@ adv_bridge: hdmi@3d {
 		compatible = "adi,adv7535";
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_hdmi_bridge>;
-		reg = <0x3d>, <0x3c>, <0x3e>, <0x3f>;
-		reg-names = "main", "cec", "edid", "packet";
+		reg = <0x3d>, <0x3e>, <0x3c>, <0x3f>;
+		reg-names = "main", "edid", "cec", "packet";
 		adi,dsi-lanes = <4>;
 		avdd-supply = <&reg_hdmi>;
 		a2vdd-supply = <&reg_hdmi>;

-- 
2.34.1

Re: [PATCH 02/17] arm64: dts: fsl-ls1043a: move "fsl, ls1043a-qdma" ahead "fsl, ls1021a-qdma"

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:34:57 CEST schrieb Frank Li:
> Move fsl,ls1043a-qdma ahead fallback string fsl,ls1021a-qdma.
> 
> Fix warning:
> dma-controller@8380000: compatible: 'oneOf' conditional failed, one must be fixed:
>         ['fsl,ls1021a-qdma', 'fsl,ls1043a-qdma'] is too long
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

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

> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> index fc38a0e543ab8..c0e3e8fa1e794 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> @@ -955,7 +955,7 @@ pcie3: pcie@3600000 {
>  		};
>  
>  		qdma: dma-controller@8380000 {
> -			compatible = "fsl,ls1021a-qdma", "fsl,ls1043a-qdma";
> +			compatible = "fsl,ls1043a-qdma", "fsl,ls1021a-qdma";
>  			reg = <0x0 0x8380000 0x0 0x1000>, /* Controller regs */
>  			      <0x0 0x8390000 0x0 0x10000>, /* Status regs */
>  			      <0x0 0x83a0000 0x0 0x40000>; /* Block regs */
> 
> 


-- 
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 03/17] arm64: dts: fsl-ls1088a: rename gpio admin_led_lower to admin-led-lower-hog

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:34:58 CEST schrieb Frank Li:
> Rename node name admin_led_lower to admin-led-lower-hog to fix warning:
> arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dtb: gpio@76: 'admin_led_lower' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

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

> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> index d4867d6cf47cd..57f2082143c8a 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> @@ -220,7 +220,7 @@ sfpgpio: gpio@76 {
>  		#gpio-cells = <2>;
>  		gpio-controller;
>  
> -		admin_led_lower {
> +		admin-led-lower-hog {
>  			gpio-hog;
>  			gpios = <13 GPIO_ACTIVE_HIGH>;
>  			output-low;
> 
> 


-- 
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 04/17] arm64: dts: imx8mm-venice: rename rs485_en as rs485-en-hog

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:34:59 CEST schrieb Frank Li:
> Rename gpio rs485_en as rs485-en-hog to fix below warning:
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb:
>   gpio@30230000: 'rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.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     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     | 4 ++--
>  6 files changed, 10 insertions(+), 10 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 47d3c0c49e8a0..bb2056746f8c9 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/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-low;
> 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 7fcd8c851159a..45ac8bdce8699 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,14 +19,14 @@
>  /plugin/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-high;
>  		line-name = "rs485_en";
>  	};
>  
> -	rs485_hd {
> +	rs485-hd-hog {
>  		gpio-hog;
>  		gpios = <2 GPIO_ACTIVE_HIGH>;
>  		output-low;
> 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 b19e38fc27bae..30aa620d7004d 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,14 +19,14 @@
>  /plugin/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-high;
>  		line-name = "rs485_en";
>  	};
>  
> -	rs485_hd {
> +	rs485-hd-hog {
>  		gpio-hog;
>  		gpios = <2 GPIO_ACTIVE_HIGH>;
>  		output-high;
> 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 c0f6aece844a0..b1a9f35e1dfa7 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/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-low;
> 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 e909cbd8db4bc..44ebc0a58c51a 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,14 +19,14 @@
>  /plugin/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-high;
>  		line-name = "rs485_en";
>  	};
>  
> -	rs485_hd {
> +	rs485-hd-hog {
>  		gpio-hog;
>  		gpios = <2 GPIO_ACTIVE_HIGH>;
>  		output-low;
> 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 1508cb5175b3e..2f8a7ac408739 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,14 +19,14 @@
>  /plugin/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {
>  		gpio-hog;
>  		gpios = <0 GPIO_ACTIVE_HIGH>;
>  		output-high;
>  		line-name = "rs485_en";
>  	};
>  
> -	rs485_hd {
> +	rs485-hd-hog {
>  		gpio-hog;
>  		gpios = <2 GPIO_ACTIVE_HIGH>;
>  		output-high;
> 
> 


-- 
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 05/17] arm64: dts: imx8mm-phygate: rename uart4_rs485_en to uart4-rs485-en-hog

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:35:00 CEST schrieb Frank Li:
> Rename gpio uart4_rs485_en to uart4-rs485-en-hog to fix below warning:
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtb:
>   gpio@30220000: 'uart4_rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.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 f1f38b739ef76..78f4e8d5814da 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
> @@ -18,7 +18,7 @@ &gpio3 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_gpio3_hog>;
>  
> -	uart4_rs485_en {
> +	uart4-rs485-en-hog {
>  		gpio-hog;
>  		gpios = <20 GPIO_ACTIVE_HIGH>;
>  		output-low;
> 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 1d8951e1a47e8..66288948bdd39 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
> @@ -19,7 +19,7 @@ &gpio3 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_gpio3_hog>;
>  
> -	uart4_rs485_en {
> +	uart4-rs485-en-hog {
>  		gpio-hog;
>  		gpios = <20 GPIO_ACTIVE_HIGH>;
>  		output-high;
> 
> 


-- 
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 06/17] arm64: dts: layerscape: remove undocument big-endian in watchdog

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:35:01 CEST schrieb Frank Li:
> Remove undocument property big-endian in watchdog.
> drivers/watchdog/imx2_wdt.c also never parser it.
> 
> Fix below wanring:
> arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb: watchdog@2ad0000:
> 	Unevaluated properties are not allowed ('big-endian' was unexpected)
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi | 1 -
>  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> index a3c57da63a01b..b84512bc4c881 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> @@ -430,7 +430,6 @@ wdog0: watchdog@2ad0000 {
>  			reg = <0x0 0x2ad0000 0x0 0x10000>;
>  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL QORIQ_CLK_PLL_DIV(1)>;
> -			big-endian;

RM LS1012A Rev3 10/2020 lists WDOG1 (2ad_0000) as big-endian.

>  		};
>  
>  		sai1: sai@2b50000 {
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> index c0e3e8fa1e794..5837f1b15dccb 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> @@ -776,7 +776,6 @@ wdog0: watchdog@2ad0000 {
>  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
>  					    QORIQ_CLK_PLL_DIV(1)>;
> -			big-endian;

RM LS1043A Rev6 07/2020 lists WDOG1 (2ad_0000) as big-endian.

What about ls1021a? According to RM this is also big-endian.
regmap_get_val_endian() parses this property and regmap is used
by drivers/watchdog/imx2_wdt.c

Best regards,
Alexander


>  		};
>  
>  		edma0: dma-controller@2c00000 {
> 
> 


-- 
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 07/17] arm64: dts: fsl-ls1012a-frdm: move clock-sc16is7xx under root node

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:35:02 CEST schrieb Frank Li:
> Move fixed clock "clock-sc16is7x" from dspi to root node to fix below
> warning:
> arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb:
>     serial@0: Unevaluated properties are not allowed ('clock-sc16is7xx' was unexpected)
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>

> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> index 2517528f684fe..75081ce3e9a6f 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts
> @@ -20,6 +20,12 @@ sys_mclk: clock-mclk {
>  		clock-frequency = <25000000>;
>  	};
>  
> +	sc16is7xx_clk: clock-sc16is7xx {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <24000000>;
> +	};
> +
>  	reg_1p8v: regulator-1p8v {
>  		compatible = "regulator-fixed";
>  		regulator-name = "1P8V";
> @@ -69,12 +75,6 @@ serial@0 {
>  		clocks = <&sc16is7xx_clk>;
>  		interrupt-parent = <&gpio1>;
>  		interrupts = <13 IRQ_TYPE_EDGE_FALLING>;
> -
> -		sc16is7xx_clk: clock-sc16is7xx {
> -			compatible = "fixed-clock";
> -			#clock-cells = <0>;
> -			clock-frequency = <24000000>;
> -		};
>  	};
>  };
>  
> 
> 


-- 
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 11/17] arm64: dts: fsl-lx2160a-tqmlx2160a: change "vcc" to "vdd" for hub*

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:35:06 CEST schrieb Frank Li:
> According to binging doc usb/ti,usb8041.yaml and in
> drivers/usb/misc/onboard_usb_dev.h
> 
> ti_tusb8041_data = { .supply_names = { "vdd" },};
> 
> It should vdd-supply instead vcc-supply.
> 
> Fixes: 04b77e0124ef ("arm64: dts: freescale: add fsl-lx2160a-mblx2160a board")
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Thanks.

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
>  arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
> index da0f58e26b9aa..f6a4f8d543015 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a-mblx2160a.dts
> @@ -320,7 +320,7 @@ hub_2_0: hub@1 {
>  		reg = <1>;
>  		peer-hub = <&hub_3_0>;
>  		reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
> -		vcc-supply = <&reg_vcc3v3>;
> +		vdd-supply = <&reg_vcc3v3>;
>  	};
>  
>  	hub_3_0: hub@2 {
> @@ -328,7 +328,7 @@ hub_3_0: hub@2 {
>  		reg = <2>;
>  		peer-hub = <&hub_2_0>;
>  		reset-gpios = <&gpioex1 0 GPIO_ACTIVE_LOW>;
> -		vcc-supply = <&reg_vcc3v3>;
> +		vdd-supply = <&reg_vcc3v3>;
>  	};
>  };
>  
> 
> 


-- 
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 15/17] arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb

[Alexander Stein]

Am Dienstag, 13. August 2024, 06:35:10 CEST schrieb Frank Li:
> Rename hub to usb to fix below warning:
> arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb: hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Thanks.

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

> ---
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> index 852dd3d2eac74..325d8097bf7fc 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> @@ -533,7 +533,7 @@ &usbotg2 {
>  	samsung,picophy-dc-vol-level-adjust = <7>;
>  	status = "okay";
>  
> -	hub_2_0: hub@1 {
> +	hub_2_0: usb@1 {
>  		compatible = "usb424,2517";
>  		reg = <1>;
>  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
> diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> index da8f19a646a98..b1fcbc12bd882 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> @@ -486,7 +486,7 @@ &usbotg2 {
>  	samsung,picophy-dc-vol-level-adjust = <7>;
>  	status = "okay";
>  
> -	hub_2_0: hub@1 {
> +	hub_2_0: usb@1 {
>  		compatible = "usb424,2517";
>  		reg = <1>;
>  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
> 
> 


-- 
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 15/17] arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb

[Alexander Stein]

Am Dienstag, 13. August 2024, 10:40:04 CEST schrieb Alexander Stein:
> Am Dienstag, 13. August 2024, 06:35:10 CEST schrieb Frank Li:
> > Rename hub to usb to fix below warning:
> > arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb: hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
> > 
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> 
> Thanks.
> 
> Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>

Still valid, but there is already https://lore.kernel.org/all/20240808123206.192906-1-festevam@gmail.com/

> > ---
> >  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
> >  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> > index 852dd3d2eac74..325d8097bf7fc 100644
> > --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts
> > @@ -533,7 +533,7 @@ &usbotg2 {
> >  	samsung,picophy-dc-vol-level-adjust = <7>;
> >  	status = "okay";
> >  
> > -	hub_2_0: hub@1 {
> > +	hub_2_0: usb@1 {
> >  		compatible = "usb424,2517";
> >  		reg = <1>;
> >  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
> > diff --git a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> > index da8f19a646a98..b1fcbc12bd882 100644
> > --- a/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
> > @@ -486,7 +486,7 @@ &usbotg2 {
> >  	samsung,picophy-dc-vol-level-adjust = <7>;
> >  	status = "okay";
> >  
> > -	hub_2_0: hub@1 {
> > +	hub_2_0: usb@1 {
> >  		compatible = "usb424,2517";
> >  		reg = <1>;
> >  		reset-gpios = <&expander1 2 GPIO_ACTIVE_LOW>;
> > 
> > 
> 
> 
> 


-- 
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 05/17] arm64: dts: imx8mm-phygate: rename uart4_rs485_en to uart4-rs485-en-hog

[Krzysztof Kozlowski]

On 13/08/2024 06:35, Frank Li wrote:
> Rename gpio uart4_rs485_en to uart4-rs485-en-hog to fix below warning:
> arch/arm64/boot/dts/freescale/imx8mm-phygate-tauri-l-rs232-rs232.dtb:
>   gpio@30220000: 'uart4_rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[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 +-

These are simple node renames. Squash them all, not one board by one.

Best regards,
Krzysztof

Re: [PATCH 04/17] arm64: dts: imx8mm-venice: rename rs485_en as rs485-en-hog

[Krzysztof Kozlowski]

On 13/08/2024 06:34, Frank Li wrote:
> Rename gpio rs485_en as rs485-en-hog to fix below warning:
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb:
>   gpio@30230000: 'rs485_en' does not match any of the regexes: '^(hog-[0-9]+|.+-hog(-[0-9]+)?)$', 'pinctrl-[0-9]+'
> 
> 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     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso     | 4 ++--
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso     | 4 ++--
>  6 files changed, 10 insertions(+), 10 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 47d3c0c49e8a0..bb2056746f8c9 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/;
>  
>  &gpio4 {
> -	rs485_en {
> +	rs485-en-hog {

These are simple node renames. Squash them all, not one board by one.


Best regards,
Krzysztof

Re: [PATCH 08/17] arm64: dts: fsl-ls1046a-qds: rename mdio-mux-emi to mdio-mux-emi@54

[Krzysztof Kozlowski]

On 13/08/2024 06:35, Frank Li wrote:
> mdio-mux-emi have reg property. Rename it as mdio-mux-emi@54.

That's correct name.
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> index 25eb1d5a69381..5c504ff47f2ca 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> @@ -237,7 +237,7 @@ &fpga {
>  	#address-cells = <1>;
>  	#size-cells = <1>;
>  
> -	mdio-mux-emi1 {
> +	mdio-mux-emi1@54 {

and that's not correct.

>  		compatible = "mdio-mux-mmioreg", "mdio-mux";
>  		mdio-parent-bus = <&mdio0>;
>  		#address-cells = <1>;
> 

Best regards,
Krzysztof

Re: [PATCH 15/17] arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb

[Krzysztof Kozlowski]

On 13/08/2024 06:35, Frank Li wrote:
> Rename hub to usb to fix below warning:
> arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb: hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxca.dts | 2 +-
>  arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts | 2 +-

No, synchronize your work with other TQ, so we won't have to do the same
review.

That's not really correct. See other thread.

Best regards,
Krzysztof

Re: [PATCH 15/17] arm64: dts: imx93-tqma9352-mba93xxla: rename hub to usb

[Krzysztof Kozlowski]

On 13/08/2024 10:49, Alexander Stein wrote:
> Am Dienstag, 13. August 2024, 10:40:04 CEST schrieb Alexander Stein:
>> Am Dienstag, 13. August 2024, 06:35:10 CEST schrieb Frank Li:
>>> Rename hub to usb to fix below warning:
>>> arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb: hub@1: $nodename:0: 'hub@1' does not match '^usb(@.*)?'
>>>
>>> Signed-off-by: Frank Li <Frank.Li@nxp.com>
>>
>> Thanks.
>>
>> Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> 
> Still valid, but there is already https://lore.kernel.org/all/20240808123206.192906-1-festevam@gmail.com/
> 

Fix the schema instead...

Best regards,
Krzysztof

Re: [PATCH 06/17] arm64: dts: layerscape: remove undocument big-endian in watchdog

[Frank Li]

On Tue, Aug 13, 2024 at 10:31:23AM +0200, Alexander Stein wrote:
> Am Dienstag, 13. August 2024, 06:35:01 CEST schrieb Frank Li:
> > Remove undocument property big-endian in watchdog.
> > drivers/watchdog/imx2_wdt.c also never parser it.
> >
> > Fix below wanring:
> > arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb: watchdog@2ad0000:
> > 	Unevaluated properties are not allowed ('big-endian' was unexpected)
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi | 1 -
> >  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 -
> >  2 files changed, 2 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > index a3c57da63a01b..b84512bc4c881 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > @@ -430,7 +430,6 @@ wdog0: watchdog@2ad0000 {
> >  			reg = <0x0 0x2ad0000 0x0 0x10000>;
> >  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
> >  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL QORIQ_CLK_PLL_DIV(1)>;
> > -			big-endian;
>
> RM LS1012A Rev3 10/2020 lists WDOG1 (2ad_0000) as big-endian.

Thank you point this. I remember I run this last month, but not sure if
1012 platform. let me double check again.

Frank
>
> >  		};
> >
> >  		sai1: sai@2b50000 {
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > index c0e3e8fa1e794..5837f1b15dccb 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > @@ -776,7 +776,6 @@ wdog0: watchdog@2ad0000 {
> >  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
> >  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
> >  					    QORIQ_CLK_PLL_DIV(1)>;
> > -			big-endian;
>
> RM LS1043A Rev6 07/2020 lists WDOG1 (2ad_0000) as big-endian.
>
> What about ls1021a? According to RM this is also big-endian.
> regmap_get_val_endian() parses this property and regmap is used
> by drivers/watchdog/imx2_wdt.c
>
> Best regards,
> Alexander
>
>
> >  		};
> >
> >  		edma0: dma-controller@2c00000 {
> >
> >
>
>
> --
> 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 06/17] arm64: dts: layerscape: remove undocument big-endian in watchdog

[Frank Li]

On Tue, Aug 13, 2024 at 01:00:42PM -0400, Frank Li wrote:
> On Tue, Aug 13, 2024 at 10:31:23AM +0200, Alexander Stein wrote:
> > Am Dienstag, 13. August 2024, 06:35:01 CEST schrieb Frank Li:
> > > Remove undocument property big-endian in watchdog.
> > > drivers/watchdog/imx2_wdt.c also never parser it.
> > >
> > > Fix below wanring:
> > > arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb: watchdog@2ad0000:
> > > 	Unevaluated properties are not allowed ('big-endian' was unexpected)
> > >
> > > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > > ---
> > >  arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi | 1 -
> > >  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 1 -
> > >  2 files changed, 2 deletions(-)
> > >
> > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > > index a3c57da63a01b..b84512bc4c881 100644
> > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi
> > > @@ -430,7 +430,6 @@ wdog0: watchdog@2ad0000 {
> > >  			reg = <0x0 0x2ad0000 0x0 0x10000>;
> > >  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
> > >  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL QORIQ_CLK_PLL_DIV(1)>;
> > > -			big-endian;
> >
> > RM LS1012A Rev3 10/2020 lists WDOG1 (2ad_0000) as big-endian.
>
> Thank you point this. I remember I run this last month, but not sure if
> 1012 platform. let me double check again.

I really thanks you for your careful review. I dump reset value of one
watch dog at 1043 platform.

0x02A80000:  00 30 00 00 00 10 00 04 00 01 00 00 00 00 00 00

It is big endian.

Frank

>
> Frank
> >
> > >  		};
> > >
> > >  		sai1: sai@2b50000 {
> > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > > index c0e3e8fa1e794..5837f1b15dccb 100644
> > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
> > > @@ -776,7 +776,6 @@ wdog0: watchdog@2ad0000 {
> > >  			interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
> > >  			clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
> > >  					    QORIQ_CLK_PLL_DIV(1)>;
> > > -			big-endian;
> >
> > RM LS1043A Rev6 07/2020 lists WDOG1 (2ad_0000) as big-endian.
> >
> > What about ls1021a? According to RM this is also big-endian.
> > regmap_get_val_endian() parses this property and regmap is used
> > by drivers/watchdog/imx2_wdt.c
> >
> > Best regards,
> > Alexander
> >
> >
> > >  		};
> > >
> > >  		edma0: dma-controller@2c00000 {
> > >
> > >
> >
> >
> > --
> > 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/
> >
> >