[PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier

devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier
@ 2024-10-28  8:23 Heiko Schocher
  2024-10-28  8:23 ` [PATCH v1 1/2] dt-bindings: arm: fsl: Add " Heiko Schocher
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28  8:23 UTC (permalink / raw)
  To: linux-kernel
  Cc: Heiko Schocher, Alexander Stein, Conor Dooley, Fabio Estevam,
	Gregor Herburger, Hiago De Franco, Hugo Villeneuve,
	Joao Paulo Goncalves, Krzysztof Kozlowski, Mathieu Othacehe,
	Max Merchel, Michael Walle, Peng Fan, Pengutronix Kernel Team,
	Rob Herring, Sascha Hauer, Shawn Guo, Tim Harvey, devicetree, imx,
	linux-arm-kernel

The board uses a ABB specific SoM from ADLink, based on NXP
i.MX8MP SoC. The SoM is used on 3 different carrier boards,
with small differences, which are all catched up in
devicetree overlays. The kernel image, the basic dtb
and all dtbos are collected in a fitimage. As bootloader
is used U-Boot which detects in his SPL stage the carrier
board by probing some i2c devices. When the correct
carrier is probed, the SPL applies all needed dtbos to
the dtb with which U-Boot gets loaded. Same principle
later before linux image boot, U-Boot applies the dtbos
needed for the carrier board before booting Linux.

So intentionally, there are no full dtb images for all
carrier combinations, only dtbos which applied as needed
on boottime.

Patches to U-Boot mailinglist not posted yet, as U-Boot
uses the same dtb as linux, so hope to get the linux
files first in, and use them when posting U-Boot board
support.

series is based on:
819837584309 - (tag: v6.12-rc5, origin/master, origin/HEAD) Linux 6.12-rc5



Heiko Schocher (2):
  dt-bindings: arm: fsl: Add ABB SoM and carrier
  arm64: dts: imx8mp: add aristainetos3 board support

 .../devicetree/bindings/arm/fsl.yaml          |    2 +
 arch/arm64/boot/dts/freescale/Makefile        |    5 +
 .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
 .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
 .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
 .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
 .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
 .../imx8mp-aristainetos3-helios.dtso          |   13 +
 .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
 .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
 .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
 .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
 12 files changed, 1720 insertions(+)
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi

-- 
2.20.1


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH v1 1/2] dt-bindings: arm: fsl: Add ABB SoM and carrier
  2024-10-28  8:23 [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Heiko Schocher
@ 2024-10-28  8:23 ` Heiko Schocher
  2024-10-28 10:20   ` Krzysztof Kozlowski
  2024-10-28  8:23 ` [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support Heiko Schocher
  2024-10-29 12:44 ` [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Rob Herring (Arm)
  2 siblings, 1 reply; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28  8:23 UTC (permalink / raw)
  To: linux-kernel
  Cc: Heiko Schocher, Alexander Stein, Conor Dooley, Gregor Herburger,
	Hiago De Franco, Hugo Villeneuve, Joao Paulo Goncalves,
	Krzysztof Kozlowski, Mathieu Othacehe, Max Merchel, Michael Walle,
	Peng Fan, Rob Herring, Shawn Guo, Tim Harvey, devicetree

add support for the i.MX8MP based SoM and carrier from ABB.

Signed-off-by: Heiko Schocher <hs@denx.de>
---

 Documentation/devicetree/bindings/arm/fsl.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index b39a7e031177..96b0eaa3b80f 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -1073,6 +1073,8 @@ properties:
       - description: i.MX8MP based Boards
         items:
           - enum:
+              - abb,imx8mp-aristianetos3 # i.MX8MP ABB Board
+              - abb,imx8mp-aristianetos3-som # i.MX8MP ABB Board SoM
               - beacon,imx8mp-beacon-kit  # i.MX8MP Beacon Development Kit
               - dmo,imx8mp-data-modul-edm-sbc # i.MX8MP eDM SBC
               - emcraft,imx8mp-navqp      # i.MX8MP Emcraft Systems NavQ+ Kit
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 1/2] dt-bindings: arm: fsl: Add ABB SoM and carrier
  2024-10-28  8:23 ` [PATCH v1 1/2] dt-bindings: arm: fsl: Add " Heiko Schocher
@ 2024-10-28 10:20   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 12+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-28 10:20 UTC (permalink / raw)
  To: Heiko Schocher, linux-kernel
  Cc: Alexander Stein, Conor Dooley, Gregor Herburger, Hiago De Franco,
	Hugo Villeneuve, Joao Paulo Goncalves, Krzysztof Kozlowski,
	Mathieu Othacehe, Max Merchel, Michael Walle, Peng Fan,
	Rob Herring, Shawn Guo, Tim Harvey, devicetree

On 28/10/2024 09:23, Heiko Schocher wrote:
> add support for the i.MX8MP based SoM and carrier from ABB.
> 
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> 
>  Documentation/devicetree/bindings/arm/fsl.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index b39a7e031177..96b0eaa3b80f 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -1073,6 +1073,8 @@ properties:
>        - description: i.MX8MP based Boards
>          items:
>            - enum:
> +              - abb,imx8mp-aristianetos3 # i.MX8MP ABB Board
> +              - abb,imx8mp-aristianetos3-som # i.MX8MP ABB Board SoM

SoM usually is not used alone. If here is one of the few exceptions, you
have entire commit msg to explain that.

If this is not exception, properly mark compatibility, like in other cases.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28  8:23 [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Heiko Schocher
  2024-10-28  8:23 ` [PATCH v1 1/2] dt-bindings: arm: fsl: Add " Heiko Schocher
@ 2024-10-28  8:23 ` Heiko Schocher
  2024-10-28 10:24   ` Krzysztof Kozlowski
  2024-10-29 12:44 ` [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Rob Herring (Arm)
  2 siblings, 1 reply; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28  8:23 UTC (permalink / raw)
  To: linux-kernel
  Cc: Heiko Schocher, Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

Add support for the i.MX8MP based aristainetos3 boards from ABB.

The board uses a ABB specific SoM from ADLink, based on NXP
i.MX8MP SoC. The SoM is used on 3 different carrier boards,
with small differences, which are all catched up in
devicetree overlays. The kernel image, the basic dtb
and all dtbos are collected in a fitimage. As bootloader
is used U-Boot which detects in his SPL stage the carrier
board by probing some i2c devices. When the correct
carrier is probed, the SPL applies all needed dtbos to
the dtb with which U-Boot gets loaded. Same principle
later before linux image boot, U-Boot applies the dtbos
needed for the carrier board before booting Linux.

Signed-off-by: Heiko Schocher <hs@denx.de>
---
checkpatch dropped the following warnings:
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/

ignored, as this compatible string is usedin other dts too, for example in

arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi

 arch/arm64/boot/dts/freescale/Makefile        |    5 +
 .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
 .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
 .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
 .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
 .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
 .../imx8mp-aristainetos3-helios.dtso          |   13 +
 .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
 .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
 .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
 .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
 11 files changed, 1718 insertions(+)
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi

diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
index 9d3df8b218a2..7c3586509b8b 100644
--- a/arch/arm64/boot/dts/freescale/Makefile
+++ b/arch/arm64/boot/dts/freescale/Makefile
@@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
 dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
 dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
 
+dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
+			  imx8mp-aristainetos3-adpismarc.dtbo \
+			  imx8mp-aristainetos3-proton2s.dtbo \
+			  imx8mp-aristainetos3-helios.dtbo \
+			  imx8mp-aristainetos3-helios-lvds.dtbo
 dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
 dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
 dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
new file mode 100644
index 000000000000..cc0cddaa33ea
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
@@ -0,0 +1,64 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+
+&ecspi1 {
+	spidev0: spi@0 {
+		reg = <0>;
+		compatible = "rohm,dh2228fv";
+		spi-max-frequency = <500000>;
+	};
+};
+
+&ecspi2 {
+	spidev1: spi@0 {
+		reg = <0>;
+		compatible = "rohm,dh2228fv";
+		spi-max-frequency = <500000>;
+	};
+};
+
+&i2c2 {
+	/* SX1509(2) U1001@IPi SMARC Plus */
+	gpio8: i2c2_gpioext0@3e {
+		/* GPIO Expander 2 Mapping :
+		 * - 0: E_GPIO1_0	<=>	IPi SMARC Plus CN101_PIN29: E_GPIO1_0
+		 * - 1: E_GPIO1_1	<=>	IPi SMARC Plus CN101_PIN31: E_GPIO1_1
+		 * - 2: E_GPIO1_2	<=>	IPi SMARC Plus CN101_PIN32: E_GPIO1_2
+		 * - 3: E_GPIO1_3	<=>	IPi SMARC Plus CN101_PIN33: E_GPIO1_3
+		 * - 4: E_GPIO1_4	<=>	IPi SMARC Plus CN101_PIN35: E_GPIO1_4
+		 * - 5: E_GPIO1_5	<=>	IPi SMARC Plus CN101_PIN36: E_GPIO1_5
+		 * - 6: E_GPIO1_6	<=>	IPi SMARC Plus CN101_PIN37: E_GPIO1_6
+		 * - 7: E_GPIO1_7	<=>	IPi SMARC Plus CN101_PIN38: E_GPIO1_7
+		 * - 8: E_GPIO2_8	<=>	IPi SMARC Plus CN101_PIN40: E_GPIO2_8
+		 * - 9: TP1002		<=>	IPi SMARC Plus TP1002 (won't use)
+		 * - 10: TP1003		<=>	IPi SMARC Plus TP1003 (won't use)
+		 * - 11: TP1004		<=>	IPi SMARC Plus TP1004 (won't use)
+		 * - 12: TP1005		<=>	IPi SMARC Plus TP1005 (won't use)
+		 * - 13: TP1006		<=>	IPi SMARC Plus TP1006 (won't use)
+		 * - 14: TP1007		<=>	IPi SMARC Plus TP1007 (won't use)
+		 * - 15: TP1008		<=>	IPi SMARC Plus TP1008 (won't use)
+		 * - 16: OSCIO		<=>	IPi SMARC Plus TP1001 (won't use)
+		 */
+		#gpio-cells = <2>;
+		#interrupt-cells = <2>;
+		compatible = "semtech,sx1509q";
+		reg = <0x3e>;
+
+		semtech,probe-reset;
+		gpio-controller;
+		interrupt-controller;
+
+		interrupt-parent = <&gpio6>;
+		interrupts = <1 IRQ_TYPE_EDGE_FALLING>;
+	};
+
+};
+
+&flexcan1 {
+	status = "okay";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
new file mode 100644
index 000000000000..5a9adccbf7cf
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+/dts-v1/;
+/plugin/;
+
+#include "imx8mp-aristainetos3-adpismarc.dtsi"
+
+&{/} {
+	model = "Aristainetos3 ADLink PI SMARC carrier";
+	compatible = "abb,aristainetos3-adpismarc", "imx8mp-aristianetos3",
+		     "abb,aristianetos3-som", "fsl,imx8mp";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
new file mode 100644
index 000000000000..55aabd6fc1f7
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
@@ -0,0 +1,89 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/pwm/pwm.h>
+
+&{/} {
+	panel: panel {
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_lcd0_vdd_en>;
+		compatible = "lg,lb070wv8";
+		backlight = <&lvds_backlight>;
+		enable-gpios = <&gpio1 13 GPIO_ACTIVE_HIGH>;
+
+		port {
+			panel_in: endpoint {
+				remote-endpoint = <&ldb_lvds_ch0>;
+			};
+		};
+	};
+};
+
+&gpio3 {
+	mipi_lvds_select {
+		gpio-hog;
+		gpios = <23 GPIO_ACTIVE_HIGH>;
+		output-low;
+		line-name = "mipi_lvds_select";
+	};
+};
+
+&hdmi_blk_ctrl {
+	status = "disabled";
+};
+
+&hdmi_pvi {
+	status = "disabled";
+};
+
+&hdmi_tx {
+	status = "disabled";
+};
+
+&hdmi_tx_phy {
+	status = "disabled";
+};
+
+&irqsteer_hdmi {
+	status = "disabled";
+};
+
+&ldb_lvds_ch0 {
+	fsl,data-mapping = "jeida";
+	fsl,data-width = <24>;
+	remote-endpoint = <&panel_in>;
+};
+
+&lcdif1 {
+	status = "disabled";
+};
+
+&lcdif2 {
+	status = "okay";
+};
+
+&lcdif3 {
+	status = "disabled";
+};
+
+&lvds_backlight {
+	status = "okay";
+};
+
+&lvds_bridge {
+	status = "okay";
+};
+
+&media_blk_ctrl {
+	/*
+	 * The internal divider will always divide the output LVDS clock by 7
+	 * so our display needs 33246000 Hz, so set VIDEO_PLL1 to
+	 * 33246000 * 7 = 232722000 Hz
+	 */
+	assigned-clock-rates = <500000000>, <200000000>, <0>, <0>, <232722000>;
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
new file mode 100644
index 000000000000..06d1883b962a
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
@@ -0,0 +1,13 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+/dts-v1/;
+/plugin/;
+
+#include "imx8mp-aristainetos3-helios-lvds.dtsi"
+
+&{/} {
+	model = "Aristainetos3 helios LVDS carrier";
+	compatible = "abb,aristainetos3-helios-lvds", "abb,aristainetos3-helios", "abb,aristianetos3-som", "fsl,imx8mp";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
new file mode 100644
index 000000000000..b4b1cb3b0cb3
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
@@ -0,0 +1,103 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+
+#include <dt-bindings/gpio/gpio.h>
+
+&{/} {
+	helios_gpio_leds {
+		compatible = "gpio-leds";
+
+		helios_blue {
+			label = "helios:blue";
+			gpios = <&helios_pca6416 15 GPIO_ACTIVE_LOW>;
+			default-state = "off";
+		};
+
+		helios_green {
+			label = "helios:green";
+			gpios = <&helios_pca6416 14 GPIO_ACTIVE_LOW>;
+			default-state = "off";
+		};
+
+		helios_red {
+			label = "helios:red";
+			gpios = <&helios_pca6416 12 GPIO_ACTIVE_LOW>;
+			default-state = "off";
+		};
+
+		helios_yellow {
+			label = "helios:yellow";
+			gpios = <&helios_pca6416 13 GPIO_ACTIVE_LOW>;
+			default-state = "off";
+		};
+	};
+};
+
+&dp83867_0 {
+	status = "disabled";
+};
+
+&ecspi1 {
+	spidev1_0: spi@0 {
+		compatible = "rohm,dh2228fv";
+		reg = <0>;
+		spi-max-frequency = <54000000>;
+	};
+
+	spidev1_1: spi@1 {
+		compatible = "rohm,dh2228fv";
+		reg = <1>;
+		spi-max-frequency = <54000000>;
+	};
+};
+
+&ecspi2 {
+	spidev2_0: spi@0 {
+		compatible = "rohm,dh2228fv";
+		reg = <0>;
+		spi-max-frequency = <54000000>;
+	};
+};
+
+&fec {
+	status = "disabled";
+};
+
+&i2c1 {
+	eeprom@57 {
+		compatible = "atmel,24c64";
+		reg = <0x57>;
+	};
+};
+
+&i2c3 {
+	helios_pca6416: gpio@20 {
+		compatible = "ti,tca6416";
+		reg = <0x20>;
+		gpio-controller;
+		#gpio-cells = <2>;
+		gpio-line-names = "DIN0_CON",
+			"DIN1_CON",
+			"DIN2_CON",
+			"DIN3_CON",
+			"DIN4_CON",
+			"DIN5_CON",
+			"DIN6_CON",
+			"DIN7_CON",
+			"PM102_RES",
+			"COMx_RES",
+			"BPL_RES",
+			"PC_RES",
+			"LED_RED",
+			"LED_YELLOW",
+			"LED_GREEN",
+			"LED_BLUE";
+	};
+
+	rtc@68 {
+		compatible = "st,m41t00";
+		reg = <0x68>;
+	};
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
new file mode 100644
index 000000000000..e4f6cefe0d7e
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
@@ -0,0 +1,13 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+/dts-v1/;
+/plugin/;
+
+#include "imx8mp-aristainetos3-helios.dtsi"
+
+&{/} {
+	model = "Aristainetos3 helios carrier";
+	compatible = "abb,aristainetos3-helios", "abb,aristianetos3-som", "fsl,imx8mp";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
new file mode 100644
index 000000000000..31f244981580
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
@@ -0,0 +1,176 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+
+#include <dt-bindings/gpio/gpio.h>
+
+&{/} {
+	gpio-leds {
+		compatible = "gpio-leds";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_gpio_led>;
+
+		leda0 {
+			label = "leda0";
+			gpios = <&gpio2 6 GPIO_ACTIVE_HIGH>;
+			default-state = "off";
+		};
+		leda1 {
+			label = "leda1";
+			gpios = <&gpio2 7 GPIO_ACTIVE_HIGH>;
+			default-state = "off";
+		};
+	};
+
+	watchdog {
+		/* MAX6371KA */
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_watchdog_gpio>;
+		compatible = "linux,wdt-gpio";
+		always-running;
+		gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
+		hw_algo = "level";
+		/* Reset triggers in 3..9 seconds */
+		hw_margin_ms = <1500>;
+		status = "okay";
+	};
+};
+
+&dp83867_0 {
+	status = "disabled";
+};
+
+&eqos {
+	max-speed = <100>;
+};
+
+&fec {
+	status = "disabled";
+};
+
+&ecspi1{
+	fsl,spi-num-chipselects = <1>;
+	pinctrl-0 = <&pinctrl_ecspi1>;
+	cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW>;
+
+	spidev1_0: spi@0 {
+		compatible = "rohm,dh2228fv";
+		reg = <0>;
+		spi-max-frequency = <54000000>;
+	};
+
+};
+
+&ecspi2 {
+	spidev2_0: spi@0 {
+		compatible = "rohm,dh2228fv";
+		reg = <0>;
+		spi-max-frequency = <54000000>;
+	};
+};
+
+&gpio1 {
+	gpio-line-names =
+		"", "", "", "", "", "", "", "power",
+		"", "", "", "", "", "", "", "",
+		"", "", "", "", "", "", "", "",
+		"", "", "", "", "", "", "", "";
+};
+
+&gpio6 {
+	gpio-line-names =
+		"relay0", "relay1", "relay2",
+		"heater", "fan", "spare",
+		"clear", "fault";
+};
+
+&i2c2 {
+	tlc59108@40 {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "ti,tlc59108";
+		reg = <0x40>;
+
+		system_red@0 {
+			label = "system:red";
+			reg = <0x0>;
+		};
+
+		system_green@1 {
+			label = "system:green";
+			reg = <0x1>;
+		};
+
+		hydrogen_green@2 {
+			label = "hydrogen:green";
+			reg = <0x2>;
+		};
+
+		hydrogen_red@3 {
+			label = "hydrogen:red";
+			reg = <0x3>;
+		};
+
+		hydrogen_blue@4 {
+			label = "hydrogen:blue";
+			reg = <0x4>;
+		};
+
+		moisture_red@5 {
+			label = "moisture:red";
+			reg = <0x5>;
+		};
+
+		moisture_green@6 {
+			label = "moisture:green";
+			reg = <0x6>;
+		};
+
+		moisture_blue@7 {
+			label = "moisture:blue";
+			reg = <0x7>;
+		};
+	};
+
+	rtc1: rtc@68 {
+		compatible = "dallas,ds1339";
+		reg = <0x68>;
+	};
+};
+
+// SER3
+&uart1 {
+	pinctrl-0 = <&pinctrl_uart1>;
+};
+
+// SER0
+&uart2 {
+	pinctrl-0 = <&pinctrl_uart2>;
+};
+
+// SER1
+&uart3 {
+	pinctrl-0 = <&pinctrl_uart3>;
+};
+
+// SER2
+&uart4 {
+	linux,rs485-enabled-at-boot-time;
+	uart-has-rtscts;
+	rs485-rts-active-low;
+	rs485-rts-delay = <0 0>;
+	rts-gpios = <&gpio3 9 GPIO_ACTIVE_HIGH>;
+};
+
+&usdhc1 {
+	status = "disabled";
+};
+
+&wdog1 {
+	status = "okay";
+};
+
+&iomuxc {
+	pinctrl-0 = <&pinctrl_gpio_proton2s>;
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
new file mode 100644
index 000000000000..6956059553d7
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
@@ -0,0 +1,13 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+/dts-v1/;
+/plugin/;
+
+#include "imx8mp-aristainetos3-proton2s.dtsi"
+
+&{/} {
+	model = "Aristainetos3 proton2s carrier";
+	compatible = "abb,aristainetos3-proton2s", "abb,aristianetos3-som", "fsl,imx8mp";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
new file mode 100644
index 000000000000..b53a2f4eb0ff
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
@@ -0,0 +1,18 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ *
+ * IMX8MP ADLINK raw SoM carrier variant
+ */
+
+/dts-v1/;
+
+#include <dt-bindings/leds/common.h>
+#include <dt-bindings/phy/phy-imx8-pcie.h>
+#include "imx8mp-aristainetos3a-som-v1.dtsi"
+
+/ {
+	model = "Aristainetos3 raw SoM carrier";
+	compatible = "abb,imx8mp-aristianetos3", "abb,imx8mp-aristianetos3-som",
+		     "fsl,imx8mp";
+};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
new file mode 100644
index 000000000000..903917bf0061
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
@@ -0,0 +1,1210 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
+ */
+
+#include "imx8mp.dtsi"
+#include <dt-bindings/net/ti-dp83867.h>
+
+/ {
+	model = "ADLINK LEC-iMX8MP-Q-N-4G-32G";
+	compatible = "abb,imx8mp-aristianetos3-som", "fsl,imx8mp";
+
+	aliases {
+		mmc0 = &usdhc3;	/* eMMC */
+		mmc1 = &usdhc2;	/* MicroSD */
+		ethernet0 = &eqos;
+		ethernet1 = &fec;
+	};
+
+	chosen {
+		bootargs = "console=ttymxc1,115200 earlycon=ec_imx6q,0x30890000,115200";
+		stdout-path = &uart2;
+	};
+
+	connector {
+		compatible = "usb-c-connector";
+		label = "USB-C";
+		data-role = "dual";
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+
+				usb_c_0_hs_ep: endpoint {
+					remote-endpoint = <&dwc3_0_hs_ep>;
+				};
+			};
+		};
+	};
+
+	fixed-regulators {
+		compatible = "simple-bus";
+		device_type = "fixed-regulators";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		dp83867_2v5: regulator-enet {
+			compatible = "regulator-fixed";
+			regulator-name = "enet-2v5";
+			regulator-min-microvolt = <1800000>;
+			regulator-max-microvolt = <1800000>;
+			gpio = <&gpio7 15 GPIO_ACTIVE_HIGH>;
+			regulator-boot-on;
+			regulator-always-on;
+			enable-active-high;
+		};
+	};
+
+	gpio-leds {
+		compatible = "gpio-leds";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_gpio_led>;
+
+		status {
+			label = "yellow:status";
+			gpios = <&gpio3 16 GPIO_ACTIVE_HIGH>;
+			default-state = "on";
+		};
+	};
+
+	lvds_backlight: lvds_backlight {
+		compatible = "pwm-backlight";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_lvds_pwr_en>;
+		pwms = <&pwm2 0 100000>;
+		enable-gpios = <&gpio1 10 GPIO_ACTIVE_HIGH>;
+		status = "disabled";
+
+		brightness-levels = < 0  1  2  3  4  5  6  7  8  9
+				     10 11 12 13 14 15 16 17 18 19
+				     20 21 22 23 24 25 26 27 28 29
+				     30 31 32 33 34 35 36 37 38 39
+				     40 41 42 43 44 45 46 47 48 49
+				     50 51 52 53 54 55 56 57 58 59
+				     60 61 62 63 64 65 66 67 68 69
+				     70 71 72 73 74 75 76 77 78 79
+				     80 81 82 83 84 85 86 87 88 89
+				     90 91 92 93 94 95 96 97 98 99
+				    100>;
+		default-brightness-level = <80>;
+	};
+
+	memory@40000000 {
+		device_type = "memory";
+		/* Memory size 512 MiB..8 GiB will be filled by U-Boot */
+		reg = <0x0 0x40000000 0 0x08000000>;
+	};
+
+	pcie0_refclk: pcie0-refclk {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <100000000>;
+	};
+
+	reg_can1_stby: regulator-can1-stby {
+		compatible = "regulator-fixed";
+		regulator-name = "can1-stby";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_flexcan1_reg>;
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		gpio = <&gpio5 5 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
+	reg_can2_stby: regulator-can2-stby {
+		compatible = "regulator-fixed";
+		regulator-name = "can2-stby";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_flexcan2_reg>;
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		gpio = <&gpio4 27 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
+	reg_usb1_host_vbus: regulator-usb1-vbus {
+		compatible = "regulator-fixed";
+		regulator-name = "usb1_host_vbus";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_usb1_vbus>;
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		gpio = <&gpio1 14 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+		regulator-always-on;
+	};
+
+	reg_usdhc2_vmmc: regulator-usdhc2-vmmc {
+		compatible = "regulator-fixed";
+		enable-active-high;
+		gpio = <&gpio2 19 0>; /* SD2_RESET */
+		off-on-delay-us = <12000>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_usdhc2_vmmc>;
+		regulator-max-microvolt = <3300000>;
+		regulator-min-microvolt = <3300000>;
+		regulator-name = "VDD_3V3_SD";
+		startup-delay-us = <100>;
+		vin-supply = <&buck4>;
+	};
+
+	reg_vdd_3p3v_awo: regulator-vdd-3p3v-awo {
+		compatible = "regulator-fixed";
+		regulator-always-on;
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		regulator-name = "VDD_3P3V_AWO";
+	};
+
+	reserved-memory {
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		rpmsg_reserved: rpmsg@55800000 {
+			no-map;
+			reg = <0 0x55800000 0 0x800000>;
+		};
+	};
+};
+
+&A53_0 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_1 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_2 {
+	cpu-supply = <&buck2>;
+};
+
+&A53_3 {
+	cpu-supply = <&buck2>;
+};
+
+&clk {
+	init-on-array = <IMX8MP_CLK_HSIO_ROOT>;
+	clocks = <&osc_32k>, <&osc_24m>, <&clk_ext1>, <&clk_ext2>,
+		 <&clk_ext3>, <&clk_ext4>;
+	clock-names = "osc_32k", "osc_24m", "clk_ext1", "clk_ext2",
+		      "clk_ext3", "clk_ext4";
+	assigned-clocks = <&clk IMX8MP_CLK_A53_SRC>,
+			  <&clk IMX8MP_CLK_A53_CORE>,
+			  <&clk IMX8MP_CLK_NOC>,
+			  <&clk IMX8MP_CLK_NOC_IO>,
+			  <&clk IMX8MP_CLK_GIC>,
+			  <&clk IMX8MP_CLK_AUDIO_AHB>,
+			  <&clk IMX8MP_CLK_AUDIO_AXI_SRC>,
+			  <&clk IMX8MP_AUDIO_PLL1>,
+			  <&clk IMX8MP_AUDIO_PLL2>,
+			  <&clk IMX8MP_VIDEO_PLL1>;
+};
+
+&ecspi1{
+	#address-cells = <1>;
+	#size-cells = <0>;
+
+	fsl,spi-num-chipselects = <2>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_ecspi1 &pinctrl_ecspi1_cs2>;
+	cs-gpios = <&gpio5 9 GPIO_ACTIVE_LOW &gpio1 6 GPIO_ACTIVE_LOW>;
+	status = "okay";
+};
+
+&ecspi2 {
+	#address-cells = <1>;
+	#size-cells = <0>;
+	fsl,spi-num-chipselects = <1>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_ecspi2>;
+	cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>;
+	status = "okay";
+};
+
+/* eth0 */
+&eqos {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_eqos_rgmii>;
+	phy-handle = <&dp83867_1>;
+	phy-mode = "rgmii-id";
+	snps,force_thresh_dma_mode;
+	snps,mtl-tx-config = <&mtl_tx_setup>;
+	snps,mtl-rx-config = <&mtl_rx_setup>;
+	status = "okay";
+
+	mdio {
+		compatible = "snps,dwmac-mdio";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		dp83867_1: eqos-ethernet-phy@0 {
+			compatible = "ethernet-phy-id2000.a231", "ethernet-phy-ieee802.3-c22";
+			reg = <0>;
+			ti,rx-internal-delay = <DP83867_RGMIIDCTL_1_75_NS>;
+			ti,tx-internal-delay = <DP83867_RGMIIDCTL_1_75_NS>;
+			ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
+			ti,min-output-impedance;
+			ti,dp83867-rxctrl-strap-quirk;
+			interrupt-parent = <&gpio4>;
+			interrupts = <21 IRQ_TYPE_EDGE_FALLING>;
+			reset-gpios = <&gpio4 22 GPIO_ACTIVE_LOW>;
+		};
+	};
+
+	mtl_tx_setup: tx-queues-config {
+		snps,tx-queues-to-use = <5>;
+		snps,tx-sched-sp;
+		queue0 {
+			snps,dcb-algorithm;
+			snps,priority = <0x1>;
+		};
+		queue1 {
+			snps,dcb-algorithm;
+			snps,priority = <0x2>;
+		};
+		queue2 {
+			snps,dcb-algorithm;
+			snps,priority = <0x4>;
+		};
+		queue3 {
+			snps,dcb-algorithm;
+			snps,priority = <0x8>;
+		};
+		queue4 {
+			snps,dcb-algorithm;
+			snps,priority = <0xf0>;
+		};
+	};
+
+	mtl_rx_setup: rx-queues-config {
+		snps,rx-queues-to-use = <5>;
+		snps,rx-sched-sp;
+		queue0 {
+			snps,dcb-algorithm;
+			snps,priority = <0x1>;
+			snps,map-to-dma-channel = <0>;
+		};
+		queue1 {
+			snps,dcb-algorithm;
+			snps,priority = <0x2>;
+			snps,map-to-dma-channel = <1>;
+		};
+		queue2 {
+			snps,dcb-algorithm;
+			snps,priority = <0x4>;
+			snps,map-to-dma-channel = <2>;
+		};
+		queue3 {
+			snps,dcb-algorithm;
+			snps,priority = <0x8>;
+			snps,map-to-dma-channel = <3>;
+		};
+		queue4 {
+			snps,dcb-algorithm;
+			snps,priority = <0xf0>;
+			snps,map-to-dma-channel = <4>;
+		};
+	};
+
+};
+
+/* eth1 */
+&fec {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_fec_rgmii>;
+	phy-handle = <&dp83867_0>;
+	phy-mode = "rgmii-id";
+	fsl,magic-packet;
+	status = "okay";
+
+	mdio {
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		dp83867_0: ethernet-phy@1 {
+			compatible = "ethernet-phy-id2000.a231", "ethernet-phy-ieee802.3-c22";
+			reg = <1>;
+			interrupt-parent = <&gpio4>;
+			interrupts = <3 IRQ_TYPE_EDGE_FALLING>;
+			reset-gpio = <&gpio4 2 GPIO_ACTIVE_LOW>;
+
+			ti,rx-internal-delay = <DP83867_RGMIIDCTL_1_75_NS>;
+			ti,tx-internal-delay = <DP83867_RGMIIDCTL_1_75_NS>;
+			ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>;
+
+			ti,min-output-impedance;
+			ti,dp83867-rxctrl-strap-quirk;
+			eee-broken-1000t;
+		};
+	};
+};
+
+&flexcan1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_flexcan1>;
+	xceiver-supply = <&reg_can1_stby>;
+	status = "disabled";
+};
+
+&flexcan2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_flexcan2>;
+	xceiver-supply = <&reg_can1_stby>;
+	status = "disabled";
+};
+
+&hdmi_blk_ctrl {
+	status = "okay";
+};
+
+&hdmi_pvi {
+	status = "okay";
+};
+
+&hdmi_tx {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_hdmi>;
+	status = "okay";
+};
+
+&hdmi_tx_phy {
+	status = "okay";
+};
+
+&i2c1 {
+	clock-frequency = <100000>;
+	pinctrl-names = "default", "gpio";
+	pinctrl-0 = <&pinctrl_i2c1>;
+	pinctrl-1 = <&pinctrl_i2c1_gpio>;
+	scl-gpios = <&gpio5 14 GPIO_ACTIVE_HIGH>;
+	sda-gpios = <&gpio5 15 GPIO_ACTIVE_HIGH>;
+	status = "okay";
+
+	pmic: pmic@25 {
+		compatible = "nxp,pca9450c";
+		reg = <0x25>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_pmic>;
+		interrupt-parent = <&gpio1>;
+		interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
+
+		/*
+		 * i.MX 8M Plus Data Sheet for Consumer Products
+		 * 3.1.4 Operating ranges
+		 * MIMX8ML8CVNKZAB
+		 */
+		regulators {
+			buck1: BUCK1 {	/* VDD_SOC (dual-phase with BUCK3) */
+				regulator-compatible = "BUCK1";
+				regulator-min-microvolt = <600000>;
+				regulator-max-microvolt = <2187500>;
+				regulator-ramp-delay = <3125>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			buck2: BUCK2 {	/* VDD_ARM */
+				nxp,dvs-run-voltage = <950000>;
+				nxp,dvs-standby-voltage = <850000>;
+				regulator-compatible = "BUCK2";
+				regulator-min-microvolt = <600000>;
+				regulator-max-microvolt = <2187500>;
+				regulator-ramp-delay = <3125>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			buck4: BUCK4 {	/* VDD_3V3 */
+				regulator-compatible = "BUCK4";
+				regulator-min-microvolt = <600000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			buck5: BUCK5 {	/* VDD_1V8 */
+				regulator-compatible = "BUCK5";
+				regulator-min-microvolt = <600000>;
+				regulator-max-microvolt = <3400000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			buck6: BUCK6 {	/* NVCC_DRAM_1V1 */
+				regulator-compatible = "BUCK6";
+				regulator-min-microvolt = <600000>;
+				regulator-max-microvolt = <3400000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			ldo1: LDO1 {	/* NVCC_SNVS_1V8 */
+				regulator-compatible = "LDO1";
+				regulator-min-microvolt = <1600000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			ldo2: LDO2 {	/* VDDA_1V8 */
+				regulator-compatible = "LDO2";
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <1150000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			ldo3: LDO3 {	/* VDDA_1V8 */
+				regulator-compatible = "LDO3";
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			ldo4: LDO4 {	/* PMIC_LDO4 */
+				regulator-compatible = "LDO4";
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			ldo5: LDO5 {	/* NVCC_SD2 */
+				regulator-compatible = "LDO5";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+			};
+		};
+	};
+};
+
+&i2c2 {
+	clock-frequency = <400000>;
+	pinctrl-names = "default", "gpio";
+	pinctrl-0 = <&pinctrl_i2c2>;
+	pinctrl-1 = <&pinctrl_i2c2_gpio>;
+	scl-gpios = <&gpio5 16 GPIO_ACTIVE_HIGH>;
+	sda-gpios = <&gpio5 17 GPIO_ACTIVE_HIGH>;
+	status = "okay";
+};
+
+&i2c3 {
+	clock-frequency = <100000>;
+	pinctrl-names = "default", "gpio";
+	pinctrl-0 = <&pinctrl_i2c3>;
+	pinctrl-1 = <&pinctrl_i2c3_gpio>;
+	scl-gpios = <&gpio5 18 GPIO_ACTIVE_HIGH>;
+	sda-gpios = <&gpio5 19 GPIO_ACTIVE_HIGH>;
+	status = "okay";
+};
+
+&i2c5 {
+	#address-cells = <1>;
+	clock-frequency = <100000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_i2c5>;
+	status = "okay";
+};
+
+&i2c6 {
+	clock-frequency = <100000>;
+	pinctrl-names = "default", "gpio";
+	pinctrl-0 = <&pinctrl_i2c6>;
+	pinctrl-1 = <&pinctrl_i2c6_gpio>;
+	scl-gpios = <&gpio3 19 GPIO_ACTIVE_HIGH>;
+	sda-gpios = <&gpio3 20 GPIO_ACTIVE_HIGH>;
+	status = "okay";
+
+	/* TPM - ST33TPHF2XI2C U2301 */
+	tpm: tpm@2e {
+		#gpio-cells = <2>;
+		#interrupt-cells = <2>;
+		compatible = "st,st33ktpm2xi2c";
+		reg = <0x2e>;
+
+		label = "tpm";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_i2c6_tpm_irq &pinctrl_tpm>;
+		interrupt-parent = <&gpio3>;
+		interrupts = <14 IRQ_TYPE_LEVEL_LOW>;
+		reset-gpio = <&gpio6 11 GPIO_ACTIVE_LOW>;
+		status = "okay";
+	};
+
+	/* SX1509(0) U2605 */
+	gpio6: i2c6_gpioext0@3f {
+		#gpio-cells = <2>;
+		#interrupt-cells = <2>;
+		compatible = "semtech,sx1509q";
+		reg = <0x3f>;
+
+		semtech,probe-reset;
+		gpio-controller;
+		interrupt-controller;
+
+		interrupt-parent = <&gpio1>;
+		interrupts = <12 IRQ_TYPE_EDGE_FALLING>;
+	};
+
+	/* SX1509(1) U2606 */
+	gpio7: i2c6_gpioext1@70 {
+		#gpio-cells = <2>;
+		#interrupt-cells = <2>;
+		compatible = "semtech,sx1509q";
+		reg = <0x70>;
+
+		semtech,probe-reset;
+		gpio-controller;
+		interrupt-controller;
+
+		interrupt-parent = <&gpio4>;
+		interrupts = <19 IRQ_TYPE_EDGE_FALLING>;
+
+		pinctrl_tpm: aristainetos3-tpm-grp {
+			pins = "gpio6","gpio7"; // TPM_PP, TPM_LP
+			output-high;
+		};
+	};
+
+	/* RTC U2607 */
+	rtc0: rtc@51 {
+		compatible = "nxp,pcf8563";
+		reg = <0x51>;
+		#clock-cells = <0>;
+	};
+};
+
+&irqsteer_hdmi {
+	status = "okay";
+};
+
+&lcdif1 {
+	status = "disabled";
+};
+
+&lcdif2 {
+	status = "disabled";
+};
+
+/* HDMI */
+&lcdif3 {
+	status = "okay";
+
+};
+
+&lvds_bridge {
+	status = "disabled";
+};
+
+&mipi_dsi {
+	status = "disabled";
+};
+
+&pcie{
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pcie>;
+	reset-gpio = <&gpio4 20 GPIO_ACTIVE_LOW>;
+	ext_osc = <0>;
+	clocks = <&clk IMX8MP_CLK_HSIO_ROOT>,
+		 <&clk IMX8MP_CLK_PCIE_AUX>,
+		 <&clk IMX8MP_CLK_HSIO_AXI>,
+		 <&clk IMX8MP_CLK_PCIE_ROOT>;
+	clock-names = "pcie", "pcie_aux", "pcie_phy", "pcie_bus";
+	assigned-clocks = <&clk IMX8MP_CLK_HSIO_AXI>,
+			  <&clk IMX8MP_CLK_PCIE_AUX>;
+	assigned-clock-rates = <500000000>, <10000000>;
+	assigned-clock-parents = <&clk IMX8MP_SYS_PLL2_500M>,
+				 <&clk IMX8MP_SYS_PLL2_50M>;
+	reserved-region = <&rpmsg_reserved>;
+	fsl,tx-deemph-gen1 = <0x1f>;
+	fsl,max-link-speed = <3>;
+	status = "okay";
+};
+
+&pcie_phy{
+	fsl,refclk-pad-mode = <IMX8_PCIE_REFCLK_PAD_INPUT>;
+	clocks = <&pcie0_refclk>;
+	clock-names = "ref";
+	status = "okay";
+};
+
+&pwm1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm1>;
+	status = "okay";
+};
+
+&pwm2 {
+	#pwm-cells = <2>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm2>;
+	status = "okay";
+};
+
+&snvs_pwrkey {
+	status = "okay";
+};
+
+&uart1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_uart1>;
+	status = "okay";
+};
+
+&uart2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_uart2>;
+	status = "okay";
+};
+
+&uart3 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_uart3>;
+	status = "okay";
+};
+
+&uart4 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_uart4>;
+	status = "okay";
+};
+
+&usb3_phy0 {
+	fsl,phy-tx-vref-tune = <0xe>;
+	fsl,phy-tx-preemp-amp-tune = <3>;
+	fsl,phy-tx-vboost-level = <5>;
+	fsl,phy-comp-dis-tune = <7>;
+	fsl,pcs-tx-deemph-3p5db = <0x21>;
+	fsl,phy-pcs-tx-swing-full = <0x7f>;
+	status = "okay";
+};
+
+&usb3_0 {
+	status = "okay";
+};
+
+&usb_dwc3_0 {
+	dr_mode = "otg";
+	hnp-disable;
+	srp-disable;
+	adp-disable;
+	usb-role-switch;
+	role-switch-default-mode = "peripheral";
+	status = "okay";
+
+	port {
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		dwc3_0_hs_ep: endpoint@0 {
+			reg = <0>;
+			remote-endpoint = <&usb_c_0_hs_ep>;
+		};
+	};
+};
+
+&usb3_phy1 {
+	fsl,phy-tx-preemp-amp-tune = <3>;
+	status = "okay";
+};
+
+&usb3_1 {
+	status = "okay";
+};
+
+&usb_dwc3_1 {
+	dr_mode = "host";
+	status = "okay";
+};
+
+&usdhc1 {
+	pinctrl-names = "default", "state_100mhz", "state_200mhz";
+	pinctrl-0 = <&pinctrl_usdhc1>;
+	pinctrl-1 = <&pinctrl_usdhc1_100mhz>;
+	pinctrl-2 = <&pinctrl_usdhc1_200mhz>;
+	bus-width = <4>;
+	non-removable;
+	status = "okay";
+};
+
+/* SD slot */
+&usdhc2 {
+	pinctrl-names = "default", "state_100mhz", "state_200mhz";
+	pinctrl-0 = <&pinctrl_usdhc2>, <&pinctrl_usdhc2_gpio>;
+	pinctrl-1 = <&pinctrl_usdhc2_100mhz>, <&pinctrl_usdhc2_gpio>;
+	pinctrl-2 = <&pinctrl_usdhc2_200mhz>, <&pinctrl_usdhc2_gpio>;
+	cd-gpios = <&gpio2 12 GPIO_ACTIVE_LOW>;
+	vmmc-supply = <&reg_usdhc2_vmmc>;
+	bus-width = <4>;
+	status = "okay";
+};
+
+/* eMMC */
+&usdhc3 {
+	pinctrl-names = "default", "state_100mhz", "state_200mhz";
+	pinctrl-0 = <&pinctrl_usdhc3>;
+	pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
+	pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
+	vmmc-supply = <&buck4>;
+	vqmmc-supply = <&buck5>;
+	bus-width = <8>;
+	non-removable;
+	status = "okay";
+};
+
+&wdog1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_wdog>;
+	fsl,ext-reset-output;
+	status = "okay";
+};
+
+&iomuxc {
+	pinctrl-names = "default";
+
+	pinctrl_disp_select:  aristainetos3-dispselect-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_RXD2__GPIO3_IO23		0xd6
+		>;
+	};
+
+	pinctrl_ecspi1: aristainetos3-ecspi1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_ECSPI1_SCLK__ECSPI1_SCLK		0x82
+			MX8MP_IOMUXC_ECSPI1_MOSI__ECSPI1_MOSI		0x82
+			MX8MP_IOMUXC_ECSPI1_MISO__ECSPI1_MISO		0x82
+			MX8MP_IOMUXC_ECSPI1_SS0__GPIO5_IO09		0x40000
+		>;
+	};
+
+	pinctrl_ecspi1_cs2: aristainetos3-ecspi1-cs2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO06__GPIO1_IO06		0x40000
+		>;
+	};
+
+	pinctrl_ecspi2: aristainetos3-ecspi2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_ECSPI2_SCLK__ECSPI2_SCLK		0x82
+			MX8MP_IOMUXC_ECSPI2_MOSI__ECSPI2_MOSI		0x82
+			MX8MP_IOMUXC_ECSPI2_MISO__ECSPI2_MISO		0x82
+			MX8MP_IOMUXC_ECSPI2_SS0__GPIO5_IO13		0x40000
+		>;
+	};
+
+	pinctrl_eqos_rgmii: aristainetos3-eqos-rgmii-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_ENET_MDC__ENET_QOS_MDC				0x3
+			MX8MP_IOMUXC_ENET_MDIO__ENET_QOS_MDIO				0x3
+			MX8MP_IOMUXC_ENET_RD0__ENET_QOS_RGMII_RD0			0x91
+			MX8MP_IOMUXC_ENET_RD1__ENET_QOS_RGMII_RD1			0x91
+			MX8MP_IOMUXC_ENET_RD2__ENET_QOS_RGMII_RD2			0x91
+			MX8MP_IOMUXC_ENET_RD3__ENET_QOS_RGMII_RD3			0x91
+			MX8MP_IOMUXC_ENET_RXC__CCM_ENET_QOS_CLOCK_GENERATE_RX_CLK	0x91
+			MX8MP_IOMUXC_ENET_RX_CTL__ENET_QOS_RGMII_RX_CTL			0x91
+			MX8MP_IOMUXC_ENET_TD0__ENET_QOS_RGMII_TD0			0x1f
+			MX8MP_IOMUXC_ENET_TD1__ENET_QOS_RGMII_TD1			0x1f
+			MX8MP_IOMUXC_ENET_TD2__ENET_QOS_RGMII_TD2			0x1f
+			MX8MP_IOMUXC_ENET_TD3__ENET_QOS_RGMII_TD3			0x1f
+			MX8MP_IOMUXC_ENET_TX_CTL__ENET_QOS_RGMII_TX_CTL			0x1f
+			MX8MP_IOMUXC_ENET_TXC__CCM_ENET_QOS_CLOCK_GENERATE_TX_CLK	0x1f
+			MX8MP_IOMUXC_SAI2_RXC__GPIO4_IO22				0x19
+		>;
+	};
+
+	pinctrl_fec_rgmii: aristainetos3-fec-rgmii-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI1_RXD2__ENET1_MDC		0x3
+			MX8MP_IOMUXC_SAI1_RXD3__ENET1_MDIO		0x3
+			MX8MP_IOMUXC_SAI1_RXD4__ENET1_RGMII_RD0		0x91
+			MX8MP_IOMUXC_SAI1_RXD5__ENET1_RGMII_RD1		0x91
+			MX8MP_IOMUXC_SAI1_RXD6__ENET1_RGMII_RD2		0x91
+			MX8MP_IOMUXC_SAI1_RXD7__ENET1_RGMII_RD3		0x91
+			MX8MP_IOMUXC_SAI1_TXC__ENET1_RGMII_RXC		0x91
+			MX8MP_IOMUXC_SAI1_TXFS__ENET1_RGMII_RX_CTL	0x91
+			MX8MP_IOMUXC_SAI1_TXD0__ENET1_RGMII_TD0		0x1f
+			MX8MP_IOMUXC_SAI1_TXD1__ENET1_RGMII_TD1		0x1f
+			MX8MP_IOMUXC_SAI1_TXD2__ENET1_RGMII_TD2		0x1f
+			MX8MP_IOMUXC_SAI1_TXD3__ENET1_RGMII_TD3		0x1f
+			MX8MP_IOMUXC_SAI1_TXD4__ENET1_RGMII_TX_CTL	0x1f
+			MX8MP_IOMUXC_SAI1_TXD5__ENET1_RGMII_TXC		0x1f
+			MX8MP_IOMUXC_SAI1_RXD0__GPIO4_IO02		0x19
+		>;
+	};
+
+	pinctrl_flexcan1: aristainetos3-flexcan1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SPDIF_RX__CAN1_RX			0x154
+			MX8MP_IOMUXC_SPDIF_TX__CAN1_TX			0x154
+		>;
+	};
+
+	pinctrl_flexcan2: aristainetos3-flexcan2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_MCLK__CAN2_RX         0x154
+			MX8MP_IOMUXC_SAI5_RXD3__CAN2_TX         0x154
+		>;
+	};
+
+	pinctrl_flexcan1_reg: aristainetos3-flexcan1-reg-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SPDIF_EXT_CLK__GPIO5_IO05  0x154   /* CAN1_STBY */
+		>;
+	};
+
+	pinctrl_flexcan2_reg: aristainetos3-flexcan2-reg-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI2_MCLK__GPIO4_IO27      0x154   /* CAN2_STBY */
+		>;
+	};
+
+	pinctrl_gpio_led: aristainetos3-gpio-led-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_READY_B__GPIO3_IO16	0x19
+		>;
+	};
+
+	pinctrl_gpio_led_proton2s: aristainetos3-gpio-proton2s-led-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD1_DATA4__GPIO2_IO06	0x19
+			MX8MP_IOMUXC_SD1_DATA5__GPIO2_IO07	0x19
+		>;
+	};
+
+	pinctrl_gpio_proton2s: aristainetos3-gpio-proton2s-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO07__GPIO1_IO07	0x19
+		>;
+	};
+
+	pinctrl_hdmi: aristainetos3-hdmi-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_HDMI_DDC_SCL__HDMIMIX_HDMI_SCL	0x400001c3
+			MX8MP_IOMUXC_HDMI_DDC_SDA__HDMIMIX_HDMI_SDA	0x400001c3
+			MX8MP_IOMUXC_HDMI_HPD__HDMIMIX_HDMI_HPD		0x40000019
+			MX8MP_IOMUXC_HDMI_CEC__HDMIMIX_HDMI_CEC		0x40000019
+		>;
+	};
+
+	pinctrl_i2c1: aristainetos3-i2c1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C1_SCL__I2C1_SCL		0x400001c3
+			MX8MP_IOMUXC_I2C1_SDA__I2C1_SDA		0x400001c3
+		>;
+	};
+
+	pinctrl_i2c1_gpio: aristainetos3-i2c1-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C1_SCL__GPIO5_IO14	0x1c3
+			MX8MP_IOMUXC_I2C1_SDA__GPIO5_IO15	0x1c3
+		>;
+	};
+
+	pinctrl_i2c2: aristainetos3-i2c2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C2_SCL__I2C2_SCL		0x400001c3
+			MX8MP_IOMUXC_I2C2_SDA__I2C2_SDA		0x400001c3
+		>;
+	};
+
+	pinctrl_i2c2_gpio: aristainetos3-i2c2-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C2_SCL__GPIO5_IO16	0x1c3
+			MX8MP_IOMUXC_I2C2_SDA__GPIO5_IO17	0x1c3
+		>;
+	};
+
+	pinctrl_i2c3: aristainetos3-i2c3-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C3_SCL__I2C3_SCL		0x400001c3
+			MX8MP_IOMUXC_I2C3_SDA__I2C3_SDA		0x400001c3
+		>;
+	};
+
+	pinctrl_i2c3_gpio: aristainetos3-i2c3-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C3_SCL__GPIO5_IO18	0x1c3
+			MX8MP_IOMUXC_I2C3_SDA__GPIO5_IO19	0x1c3
+		>;
+	};
+
+	pinctrl_i2c5: aristainetos3-i2c5-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_RXD0__I2C5_SCL	0x400001c3
+			MX8MP_IOMUXC_SAI5_MCLK__I2C5_SDA	0x400001c3
+		>;
+	};
+
+	pinctrl_i2c6: aristainetos3-i2c6-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_RXFS__I2C6_SCL	0x400001c3
+			MX8MP_IOMUXC_SAI5_RXC__I2C6_SDA		0x400001c3
+		>;
+	};
+
+	pinctrl_i2c6_gpio: aristainetos3-i2c6-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_RXFS__GPIO3_IO19	0x1c3
+			MX8MP_IOMUXC_SAI5_RXC__GPIO3_IO20	0x1c3
+		>;
+	};
+
+	pinctrl_i2c6_rtc_irq: aristainetos3-i2c6-rtc-irq-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SAI5_RXD1__GPIO3_IO22		0xd6
+		>;
+	};
+
+	pinctrl_i2c6_tpm_irq: aristainetos3-i2c6-tpm-irq-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_DQS__GPIO3_IO14		0xd6
+		>;
+	};
+
+	pinctrl_lcd0_vdd_en: aristainetos3-lcd0-vdden-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO13__GPIO1_IO13		0xd6
+		>;
+	};
+
+	pinctrl_lcd1_vdd_en: aristainetos3-lcd1-vdden-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO05__GPIO1_IO05		0xd6
+		>;
+	};
+
+	pinctrl_lvds_pwr_en: aristainetos3-lvds-pwren-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO10__GPIO1_IO10		0xd6
+		>;
+	};
+
+	pinctrl_pcie: aristainetos3-pcie-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_I2C4_SCL__PCIE_CLKREQ_B		0x61
+			MX8MP_IOMUXC_SAI1_MCLK__GPIO4_IO20		0x41
+		>;
+	};
+
+	pinctrl_pmic: aristainetos3-pmic-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO03__GPIO1_IO03	0x41
+		>;
+	};
+
+	pinctrl_pwm1: aristainetos3-pwm1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO01__PWM1_OUT	0x116
+		>;
+	};
+
+	pinctrl_pwm2: aristainetos3-pwm2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO11__PWM2_OUT	0x116
+		>;
+	};
+
+	pinctrl_uart1: aristainetos3-uart1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_UART1_RXD__UART1_DCE_RX	0x140
+			MX8MP_IOMUXC_UART1_TXD__UART1_DCE_TX	0x140
+		>;
+	};
+
+	pinctrl_uart1_rs485: aristainetos3-uart1-rs485-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_UART1_RXD__UART1_DCE_RX	0x140
+			MX8MP_IOMUXC_UART1_TXD__UART1_DCE_TX	0x140
+			MX8MP_IOMUXC_SD1_DATA0__GPIO2_IO02	0x140
+		>;
+	};
+
+	pinctrl_uart2: aristainetos3-uart2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_UART2_RXD__UART2_DCE_RX	0x140
+			MX8MP_IOMUXC_UART2_TXD__UART2_DCE_TX	0x140
+			MX8MP_IOMUXC_SD1_DATA4__GPIO2_IO06	0x140
+			MX8MP_IOMUXC_SD1_DATA5__UART2_DCE_CTS	0x140
+		>;
+	};
+
+	pinctrl_uart3: aristainetos3-uart3-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_ALE__UART3_DCE_RX	0x140
+			MX8MP_IOMUXC_NAND_CE0_B__UART3_DCE_TX	0x140
+		>;
+	};
+
+	pinctrl_uart3_rs485: aristainetos3-uart3-rs485-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_ALE__UART3_DCE_RX	0x140
+			MX8MP_IOMUXC_NAND_CE0_B__UART3_DCE_TX	0x140
+			MX8MP_IOMUXC_SD1_RESET_B__GPIO2_IO10	0x140
+		>;
+	};
+
+	pinctrl_uart4: aristainetos3-uart4-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_UART4_RXD__UART4_DCE_RX    0x140
+			MX8MP_IOMUXC_UART4_TXD__UART4_DCE_TX    0x140
+			MX8MP_IOMUXC_NAND_DATA03__GPIO3_IO09	0x140
+			MX8MP_IOMUXC_NAND_DATA02__UART4_DCE_CTS 0x140
+		>;
+	};
+
+	pinctrl_usb1_vbus: aristainetos3-usb1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO14__GPIO1_IO14	0x19
+		>;
+	};
+
+	pinctrl_usdhc1: aristainetos3-usdhc1-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK	0x190
+			MX8MP_IOMUXC_SD1_CMD__USDHC1_CMD	0x1d0
+			MX8MP_IOMUXC_SD1_DATA0__USDHC1_DATA0	0x1d0
+			MX8MP_IOMUXC_SD1_DATA1__USDHC1_DATA1	0x1d0
+			MX8MP_IOMUXC_SD1_DATA2__USDHC1_DATA2	0x1d0
+			MX8MP_IOMUXC_SD1_DATA3__USDHC1_DATA3	0x1d0
+		>;
+	};
+
+	pinctrl_usdhc1_100mhz: aristainetos3-usdhc1-100mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK	0x194
+			MX8MP_IOMUXC_SD1_CMD__USDHC1_CMD	0x1d4
+			MX8MP_IOMUXC_SD1_DATA0__USDHC1_DATA0	0x1d4
+			MX8MP_IOMUXC_SD1_DATA1__USDHC1_DATA1	0x1d4
+			MX8MP_IOMUXC_SD1_DATA2__USDHC1_DATA2	0x1d4
+			MX8MP_IOMUXC_SD1_DATA3__USDHC1_DATA3	0x1d4
+		>;
+	};
+
+	pinctrl_usdhc1_200mhz: aristainetos3-usdhc1-200mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD1_CLK__USDHC1_CLK	0x196
+			MX8MP_IOMUXC_SD1_CMD__USDHC1_CMD	0x1d6
+			MX8MP_IOMUXC_SD1_DATA0__USDHC1_DATA0	0x1d6
+			MX8MP_IOMUXC_SD1_DATA1__USDHC1_DATA1	0x1d6
+			MX8MP_IOMUXC_SD1_DATA2__USDHC1_DATA2	0x1d6
+			MX8MP_IOMUXC_SD1_DATA3__USDHC1_DATA3	0x1d6
+		>;
+	};
+
+	pinctrl_usdhc2: aristainetos3-usdhc2-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD2_CLK__USDHC2_CLK	0x190
+			MX8MP_IOMUXC_SD2_CMD__USDHC2_CMD	0x1d0
+			MX8MP_IOMUXC_SD2_DATA0__USDHC2_DATA0	0x1d0
+			MX8MP_IOMUXC_SD2_DATA1__USDHC2_DATA1	0x1d0
+			MX8MP_IOMUXC_SD2_DATA2__USDHC2_DATA2	0x1d0
+			MX8MP_IOMUXC_SD2_DATA3__USDHC2_DATA3	0x1d0
+			MX8MP_IOMUXC_GPIO1_IO04__USDHC2_VSELECT	0xc1
+
+		>;
+	};
+
+	pinctrl_usdhc2_100mhz: aristainetos3-usdhc2-100mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD2_CLK__USDHC2_CLK	0x194
+			MX8MP_IOMUXC_SD2_CMD__USDHC2_CMD	0x1d4
+			MX8MP_IOMUXC_SD2_DATA0__USDHC2_DATA0	0x1d4
+			MX8MP_IOMUXC_SD2_DATA1__USDHC2_DATA1	0x1d4
+			MX8MP_IOMUXC_SD2_DATA2__USDHC2_DATA2	0x1d4
+			MX8MP_IOMUXC_SD2_DATA3__USDHC2_DATA3	0x1d4
+			MX8MP_IOMUXC_GPIO1_IO04__USDHC2_VSELECT 0xc1
+		>;
+	};
+
+	pinctrl_usdhc2_200mhz: aristainetos3-usdhc2-200mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD2_CLK__USDHC2_CLK	0x196
+			MX8MP_IOMUXC_SD2_CMD__USDHC2_CMD	0x1d6
+			MX8MP_IOMUXC_SD2_DATA0__USDHC2_DATA0	0x1d6
+			MX8MP_IOMUXC_SD2_DATA1__USDHC2_DATA1	0x1d6
+			MX8MP_IOMUXC_SD2_DATA2__USDHC2_DATA2	0x1d6
+			MX8MP_IOMUXC_SD2_DATA3__USDHC2_DATA3	0x1d6
+			MX8MP_IOMUXC_GPIO1_IO04__USDHC2_VSELECT 0xc1
+		>;
+	};
+
+	pinctrl_usdhc2_gpio: aristainetos3-usdhc2-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD2_CD_B__GPIO2_IO12	0x40000080
+		>;
+	};
+
+	pinctrl_usdhc2_vmmc: aristainetos3-usdhc2-vmmc-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_SD2_RESET_B__GPIO2_IO19	0x41
+		>;
+	};
+
+	pinctrl_usdhc3: aristainetos3-usdhc3-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_WE_B__USDHC3_CLK	0x190
+			MX8MP_IOMUXC_NAND_WP_B__USDHC3_CMD	0x1d0
+			MX8MP_IOMUXC_NAND_DATA04__USDHC3_DATA0	0x1d0
+			MX8MP_IOMUXC_NAND_DATA05__USDHC3_DATA1	0x1d0
+			MX8MP_IOMUXC_NAND_DATA06__USDHC3_DATA2	0x1d0
+			MX8MP_IOMUXC_NAND_DATA07__USDHC3_DATA3	0x1d0
+			MX8MP_IOMUXC_NAND_RE_B__USDHC3_DATA4	0x1d0
+			MX8MP_IOMUXC_NAND_CE2_B__USDHC3_DATA5	0x1d0
+			MX8MP_IOMUXC_NAND_CE3_B__USDHC3_DATA6	0x1d0
+			MX8MP_IOMUXC_NAND_CLE__USDHC3_DATA7	0x1d0
+			MX8MP_IOMUXC_NAND_CE1_B__USDHC3_STROBE	0x190
+		>;
+	};
+
+	pinctrl_usdhc3_100mhz: aristainetos3-usdhc3-100mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_WE_B__USDHC3_CLK	0x194
+			MX8MP_IOMUXC_NAND_WP_B__USDHC3_CMD	0x1d4
+			MX8MP_IOMUXC_NAND_DATA04__USDHC3_DATA0	0x1d4
+			MX8MP_IOMUXC_NAND_DATA05__USDHC3_DATA1	0x1d4
+			MX8MP_IOMUXC_NAND_DATA06__USDHC3_DATA2	0x1d4
+			MX8MP_IOMUXC_NAND_DATA07__USDHC3_DATA3	0x1d4
+			MX8MP_IOMUXC_NAND_RE_B__USDHC3_DATA4	0x1d4
+			MX8MP_IOMUXC_NAND_CE2_B__USDHC3_DATA5	0x1d4
+			MX8MP_IOMUXC_NAND_CE3_B__USDHC3_DATA6	0x1d4
+			MX8MP_IOMUXC_NAND_CLE__USDHC3_DATA7	0x1d4
+			MX8MP_IOMUXC_NAND_CE1_B__USDHC3_STROBE	0x194
+		>;
+	};
+
+	pinctrl_usdhc3_200mhz: aristainetos3-usdhc3-200mhz-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_NAND_WE_B__USDHC3_CLK	0x196
+			MX8MP_IOMUXC_NAND_WP_B__USDHC3_CMD	0x1d6
+			MX8MP_IOMUXC_NAND_DATA04__USDHC3_DATA0	0x1d6
+			MX8MP_IOMUXC_NAND_DATA05__USDHC3_DATA1	0x1d6
+			MX8MP_IOMUXC_NAND_DATA06__USDHC3_DATA2	0x1d6
+			MX8MP_IOMUXC_NAND_DATA07__USDHC3_DATA3	0x1d6
+			MX8MP_IOMUXC_NAND_RE_B__USDHC3_DATA4	0x1d6
+			MX8MP_IOMUXC_NAND_CE2_B__USDHC3_DATA5	0x1d6
+			MX8MP_IOMUXC_NAND_CE3_B__USDHC3_DATA6	0x1d6
+			MX8MP_IOMUXC_NAND_CLE__USDHC3_DATA7	0x1d6
+			MX8MP_IOMUXC_NAND_CE1_B__USDHC3_STROBE	0x196
+		>;
+	};
+
+	pinctrl_watchdog_gpio: aristainetos3-wdog-gpio-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO06__GPIO1_IO06		0x19
+		>;
+	};
+
+	pinctrl_wdog: aristainetos3-wdog-grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_GPIO1_IO02__WDOG1_WDOG_B		0xc6
+		>;
+	};
+};
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28  8:23 ` [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support Heiko Schocher
@ 2024-10-28 10:24   ` Krzysztof Kozlowski
  2024-10-28 10:41     ` Heiko Schocher
  0 siblings, 1 reply; 12+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-28 10:24 UTC (permalink / raw)
  To: Heiko Schocher, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

On 28/10/2024 09:23, Heiko Schocher wrote:
> Add support for the i.MX8MP based aristainetos3 boards from ABB.
> 
> The board uses a ABB specific SoM from ADLink, based on NXP
> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
> with small differences, which are all catched up in
> devicetree overlays. The kernel image, the basic dtb
> and all dtbos are collected in a fitimage. As bootloader
> is used U-Boot which detects in his SPL stage the carrier
> board by probing some i2c devices. When the correct
> carrier is probed, the SPL applies all needed dtbos to
> the dtb with which U-Boot gets loaded. Same principle
> later before linux image boot, U-Boot applies the dtbos
> needed for the carrier board before booting Linux.
> 
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> checkpatch dropped the following warnings:
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
> 
> ignored, as this compatible string is usedin other dts too, for example in
> 
> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
> 
>  arch/arm64/boot/dts/freescale/Makefile        |    5 +
>  .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>  .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>  .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>  .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>  .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>  .../imx8mp-aristainetos3-helios.dtso          |   13 +
>  .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>  .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>  .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>  .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>  11 files changed, 1718 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
> 
> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
> index 9d3df8b218a2..7c3586509b8b 100644
> --- a/arch/arm64/boot/dts/freescale/Makefile
> +++ b/arch/arm64/boot/dts/freescale/Makefile
> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>  dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>  
> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
> +			  imx8mp-aristainetos3-adpismarc.dtbo \
> +			  imx8mp-aristainetos3-proton2s.dtbo \
> +			  imx8mp-aristainetos3-helios.dtbo \
> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>  dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
> new file mode 100644
> index 000000000000..cc0cddaa33ea
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
> @@ -0,0 +1,64 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
> + */
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +
> +&ecspi1 {
> +	spidev0: spi@0 {
> +		reg = <0>;
> +		compatible = "rohm,dh2228fv";

Hm? I have some doubts, what device is here?

> +		spi-max-frequency = <500000>;
> +	};
> +};
> +
> +&ecspi2 {
> +	spidev1: spi@0 {
> +		reg = <0>;
> +		compatible = "rohm,dh2228fv";
> +		spi-max-frequency = <500000>;
> +	};
> +};
> +
> +&i2c2 {
> +	/* SX1509(2) U1001@IPi SMARC Plus */
> +	gpio8: i2c2_gpioext0@3e {

Uh, no, please never send us downstream code.

Please follow DTS coding style in all upstream submissions.

Also:
Node names should be generic. See also an explanation and list of
examples (not exhaustive) in DT specification:
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation


> +		/* GPIO Expander 2 Mapping :
> +		 * - 0: E_GPIO1_0	<=>	IPi SMARC Plus CN101_PIN29: E_GPIO1_0
> +		 * - 1: E_GPIO1_1	<=>	IPi SMARC Plus CN101_PIN31: E_GPIO1_1
> +		 * - 2: E_GPIO1_2	<=>	IPi SMARC Plus CN101_PIN32: E_GPIO1_2
> +		 * - 3: E_GPIO1_3	<=>	IPi SMARC Plus CN101_PIN33: E_GPIO1_3
> +		 * - 4: E_GPIO1_4	<=>	IPi SMARC Plus CN101_PIN35: E_GPIO1_4
> +		 * - 5: E_GPIO1_5	<=>	IPi SMARC Plus CN101_PIN36: E_GPIO1_5
> +		 * - 6: E_GPIO1_6	<=>	IPi SMARC Plus CN101_PIN37: E_GPIO1_6
> +		 * - 7: E_GPIO1_7	<=>	IPi SMARC Plus CN101_PIN38: E_GPIO1_7
> +		 * - 8: E_GPIO2_8	<=>	IPi SMARC Plus CN101_PIN40: E_GPIO2_8
> +		 * - 9: TP1002		<=>	IPi SMARC Plus TP1002 (won't use)
> +		 * - 10: TP1003		<=>	IPi SMARC Plus TP1003 (won't use)
> +		 * - 11: TP1004		<=>	IPi SMARC Plus TP1004 (won't use)
> +		 * - 12: TP1005		<=>	IPi SMARC Plus TP1005 (won't use)
> +		 * - 13: TP1006		<=>	IPi SMARC Plus TP1006 (won't use)
> +		 * - 14: TP1007		<=>	IPi SMARC Plus TP1007 (won't use)
> +		 * - 15: TP1008		<=>	IPi SMARC Plus TP1008 (won't use)
> +		 * - 16: OSCIO		<=>	IPi SMARC Plus TP1001 (won't use)
> +		 */
> +		#gpio-cells = <2>;
> +		#interrupt-cells = <2>;
> +		compatible = "semtech,sx1509q";
> +		reg = <0x3e>;
> +
> +		semtech,probe-reset;
> +		gpio-controller;
> +		interrupt-controller;
> +
> +		interrupt-parent = <&gpio6>;
> +		interrupts = <1 IRQ_TYPE_EDGE_FALLING>;
> +	};
> +

Drop

> +};
> +
> +&flexcan1 {
> +	status = "okay";
> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
> new file mode 100644
> index 000000000000..5a9adccbf7cf
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
> @@ -0,0 +1,14 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
> + */
> +/dts-v1/;
> +/plugin/;
> +
> +#include "imx8mp-aristainetos3-adpismarc.dtsi"
> +
> +&{/} {
> +	model = "Aristainetos3 ADLink PI SMARC carrier";
> +	compatible = "abb,aristainetos3-adpismarc", "imx8mp-aristianetos3",
> +		     "abb,aristianetos3-som", "fsl,imx8mp";

It does not look like you tested the DTS against bindings. Please run
`make dtbs_check W=1` (see
Documentation/devicetree/bindings/writing-schema.rst or
https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
for instructions).

And why this is DTSO, I have no clue...

> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
> new file mode 100644
> index 000000000000..55aabd6fc1f7
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
> @@ -0,0 +1,89 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
> + */
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/pwm/pwm.h>
> +
> +&{/} {
> +	panel: panel {
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&pinctrl_lcd0_vdd_en>;
> +		compatible = "lg,lb070wv8";
> +		backlight = <&lvds_backlight>;
> +		enable-gpios = <&gpio1 13 GPIO_ACTIVE_HIGH>;
> +
> +		port {
> +			panel_in: endpoint {
> +				remote-endpoint = <&ldb_lvds_ch0>;
> +			};
> +		};
> +	};
> +};
> +
> +&gpio3 {
> +	mipi_lvds_select {

No, read coding style.


> +		gpio-hog;
> +		gpios = <23 GPIO_ACTIVE_HIGH>;
> +		output-low;
> +		line-name = "mipi_lvds_select";
> +	};
> +};
> +
> +&hdmi_blk_ctrl {
> +	status = "disabled";
> +};
> +
> +&hdmi_pvi {
> +	status = "disabled";
> +};
> +
> +&hdmi_tx {
> +	status = "disabled";
> +};
> +
> +&hdmi_tx_phy {
> +	status = "disabled";
> +};
> +
> +&irqsteer_hdmi {
> +	status = "disabled";
> +};
> +
> +&ldb_lvds_ch0 {
> +	fsl,data-mapping = "jeida";
> +	fsl,data-width = <24>;
> +	remote-endpoint = <&panel_in>;
> +};
> +
> +&lcdif1 {
> +	status = "disabled";
> +};
> +
> +&lcdif2 {
> +	status = "okay";
> +};
> +
> +&lcdif3 {
> +	status = "disabled";
> +};
> +
> +&lvds_backlight {
> +	status = "okay";
> +};
> +
> +&lvds_bridge {
> +	status = "okay";
> +};
> +
> +&media_blk_ctrl {
> +	/*
> +	 * The internal divider will always divide the output LVDS clock by 7
> +	 * so our display needs 33246000 Hz, so set VIDEO_PLL1 to
> +	 * 33246000 * 7 = 232722000 Hz
> +	 */
> +	assigned-clock-rates = <500000000>, <200000000>, <0>, <0>, <232722000>;
> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
> new file mode 100644
> index 000000000000..06d1883b962a
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
> @@ -0,0 +1,13 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
> + */
> +/dts-v1/;
> +/plugin/;
> +
> +#include "imx8mp-aristainetos3-helios-lvds.dtsi"
> +
> +&{/} {
> +	model = "Aristainetos3 helios LVDS carrier";
> +	compatible = "abb,aristainetos3-helios-lvds", "abb,aristainetos3-helios", "abb,aristianetos3-som", "fsl,imx8mp";

Read not only DTS coding style, but also kernel coding style. Lines are
supposed to be wrapped according to kernel coding style.


> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
> new file mode 100644
> index 000000000000..b4b1cb3b0cb3
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
> @@ -0,0 +1,103 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
> + */
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +&{/} {
> +	helios_gpio_leds {

Node names should be generic. See also an explanation and list of
examples (not exhaustive) in DT specification:
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation


> +		compatible = "gpio-leds";
> +
> +		helios_blue {

So this was absolutely never tested.

It does not look like you tested the DTS against bindings. Please run
`make dtbs_check W=1` (see
Documentation/devicetree/bindings/writing-schema.rst or
https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
for instructions).

> +			label = "helios:blue";

Use function and color instead.

I finished review here. Rest of the code does not look good, really. You
have so many, really so many, trivial issues which tools point out, that
using humans for such review is just waste of our time.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28 10:24   ` Krzysztof Kozlowski
@ 2024-10-28 10:41     ` Heiko Schocher
  2024-10-28 10:49       ` Krzysztof Kozlowski
  0 siblings, 1 reply; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28 10:41 UTC (permalink / raw)
  To: Krzysztof Kozlowski, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

Hello Krzysztof,

On 28.10.24 11:24, Krzysztof Kozlowski wrote:
> On 28/10/2024 09:23, Heiko Schocher wrote:
>> Add support for the i.MX8MP based aristainetos3 boards from ABB.
>>
>> The board uses a ABB specific SoM from ADLink, based on NXP
>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>> with small differences, which are all catched up in
>> devicetree overlays. The kernel image, the basic dtb
>> and all dtbos are collected in a fitimage. As bootloader
>> is used U-Boot which detects in his SPL stage the carrier
>> board by probing some i2c devices. When the correct
>> carrier is probed, the SPL applies all needed dtbos to
>> the dtb with which U-Boot gets loaded. Same principle
>> later before linux image boot, U-Boot applies the dtbos
>> needed for the carrier board before booting Linux.
>>
>> Signed-off-by: Heiko Schocher <hs@denx.de>
>> ---
>> checkpatch dropped the following warnings:
>> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
>>
>> ignored, as this compatible string is usedin other dts too, for example in
>>
>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>>
>>   arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>   .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>   .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>   .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>   .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>   .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>   .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>   .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>   .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>   .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>   .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>   11 files changed, 1718 insertions(+)
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>
>> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
>> index 9d3df8b218a2..7c3586509b8b 100644
>> --- a/arch/arm64/boot/dts/freescale/Makefile
>> +++ b/arch/arm64/boot/dts/freescale/Makefile
>> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>>   dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>>   dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>>   
>> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
>> +			  imx8mp-aristainetos3-adpismarc.dtbo \
>> +			  imx8mp-aristainetos3-proton2s.dtbo \
>> +			  imx8mp-aristainetos3-helios.dtbo \
>> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>> new file mode 100644
>> index 000000000000..cc0cddaa33ea
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>> @@ -0,0 +1,64 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>> + */
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/interrupt-controller/irq.h>
>> +
>> +&ecspi1 {
>> +	spidev0: spi@0 {
>> +		reg = <0>;
>> +		compatible = "rohm,dh2228fv";
> 
> Hm? I have some doubts, what device is here?

$ grep -lr dh2228fv drivers/
drivers/spi/spidev.c

Customer uses an userspace implementation...

> 
>> +		spi-max-frequency = <500000>;
>> +	};
>> +};
>> +
>> +&ecspi2 {
>> +	spidev1: spi@0 {
>> +		reg = <0>;
>> +		compatible = "rohm,dh2228fv";
>> +		spi-max-frequency = <500000>;
>> +	};
>> +};
>> +
>> +&i2c2 {
>> +	/* SX1509(2) U1001@IPi SMARC Plus */
>> +	gpio8: i2c2_gpioext0@3e {
> 
> Uh, no, please never send us downstream code.
> 
> Please follow DTS coding style in all upstream submissions.

driver is in here:

$ grep -lr probe-reset drivers/pinctrl/
drivers/pinctrl/pinctrl-sx150x.c

> Also:
> Node names should be generic. See also an explanation and list of
> examples (not exhaustive) in DT specification:
> https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation

Sorry for the nodenames...

And yes, I remove the comments.

>> +		/* GPIO Expander 2 Mapping :
>> +		 * - 0: E_GPIO1_0	<=>	IPi SMARC Plus CN101_PIN29: E_GPIO1_0
>> +		 * - 1: E_GPIO1_1	<=>	IPi SMARC Plus CN101_PIN31: E_GPIO1_1
>> +		 * - 2: E_GPIO1_2	<=>	IPi SMARC Plus CN101_PIN32: E_GPIO1_2
>> +		 * - 3: E_GPIO1_3	<=>	IPi SMARC Plus CN101_PIN33: E_GPIO1_3
>> +		 * - 4: E_GPIO1_4	<=>	IPi SMARC Plus CN101_PIN35: E_GPIO1_4
>> +		 * - 5: E_GPIO1_5	<=>	IPi SMARC Plus CN101_PIN36: E_GPIO1_5
>> +		 * - 6: E_GPIO1_6	<=>	IPi SMARC Plus CN101_PIN37: E_GPIO1_6
>> +		 * - 7: E_GPIO1_7	<=>	IPi SMARC Plus CN101_PIN38: E_GPIO1_7
>> +		 * - 8: E_GPIO2_8	<=>	IPi SMARC Plus CN101_PIN40: E_GPIO2_8
>> +		 * - 9: TP1002		<=>	IPi SMARC Plus TP1002 (won't use)
>> +		 * - 10: TP1003		<=>	IPi SMARC Plus TP1003 (won't use)
>> +		 * - 11: TP1004		<=>	IPi SMARC Plus TP1004 (won't use)
>> +		 * - 12: TP1005		<=>	IPi SMARC Plus TP1005 (won't use)
>> +		 * - 13: TP1006		<=>	IPi SMARC Plus TP1006 (won't use)
>> +		 * - 14: TP1007		<=>	IPi SMARC Plus TP1007 (won't use)
>> +		 * - 15: TP1008		<=>	IPi SMARC Plus TP1008 (won't use)
>> +		 * - 16: OSCIO		<=>	IPi SMARC Plus TP1001 (won't use)
>> +		 */
>> +		#gpio-cells = <2>;
>> +		#interrupt-cells = <2>;
>> +		compatible = "semtech,sx1509q";
>> +		reg = <0x3e>;
>> +
>> +		semtech,probe-reset;
>> +		gpio-controller;
>> +		interrupt-controller;
>> +
>> +		interrupt-parent = <&gpio6>;
>> +		interrupts = <1 IRQ_TYPE_EDGE_FALLING>;
>> +	};
>> +
> 
> Drop
> 
>> +};
>> +
>> +&flexcan1 {
>> +	status = "okay";
>> +};
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>> new file mode 100644
>> index 000000000000..5a9adccbf7cf
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>> @@ -0,0 +1,14 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>> + */
>> +/dts-v1/;
>> +/plugin/;
>> +
>> +#include "imx8mp-aristainetos3-adpismarc.dtsi"
>> +
>> +&{/} {
>> +	model = "Aristainetos3 ADLink PI SMARC carrier";
>> +	compatible = "abb,aristainetos3-adpismarc", "imx8mp-aristianetos3",
>> +		     "abb,aristianetos3-som", "fsl,imx8mp";
> 
> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check W=1` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).

Thanks for the hint! I have to fix my scripts...

> And why this is DTSO, I have no clue...
> 
>> +};
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>> new file mode 100644
>> index 000000000000..55aabd6fc1f7
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>> @@ -0,0 +1,89 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>> + */
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/input/input.h>
>> +#include <dt-bindings/interrupt-controller/arm-gic.h>
>> +#include <dt-bindings/pwm/pwm.h>
>> +
>> +&{/} {
>> +	panel: panel {
>> +		pinctrl-names = "default";
>> +		pinctrl-0 = <&pinctrl_lcd0_vdd_en>;
>> +		compatible = "lg,lb070wv8";
>> +		backlight = <&lvds_backlight>;
>> +		enable-gpios = <&gpio1 13 GPIO_ACTIVE_HIGH>;
>> +
>> +		port {
>> +			panel_in: endpoint {
>> +				remote-endpoint = <&ldb_lvds_ch0>;
>> +			};
>> +		};
>> +	};
>> +};
>> +
>> +&gpio3 {
>> +	mipi_lvds_select {
> 
> No, read coding style.
> 
> 
>> +		gpio-hog;
>> +		gpios = <23 GPIO_ACTIVE_HIGH>;
>> +		output-low;
>> +		line-name = "mipi_lvds_select";
>> +	};
>> +};
>> +
>> +&hdmi_blk_ctrl {
>> +	status = "disabled";
>> +};
>> +
>> +&hdmi_pvi {
>> +	status = "disabled";
>> +};
>> +
>> +&hdmi_tx {
>> +	status = "disabled";
>> +};
>> +
>> +&hdmi_tx_phy {
>> +	status = "disabled";
>> +};
>> +
>> +&irqsteer_hdmi {
>> +	status = "disabled";
>> +};
>> +
>> +&ldb_lvds_ch0 {
>> +	fsl,data-mapping = "jeida";
>> +	fsl,data-width = <24>;
>> +	remote-endpoint = <&panel_in>;
>> +};
>> +
>> +&lcdif1 {
>> +	status = "disabled";
>> +};
>> +
>> +&lcdif2 {
>> +	status = "okay";
>> +};
>> +
>> +&lcdif3 {
>> +	status = "disabled";
>> +};
>> +
>> +&lvds_backlight {
>> +	status = "okay";
>> +};
>> +
>> +&lvds_bridge {
>> +	status = "okay";
>> +};
>> +
>> +&media_blk_ctrl {
>> +	/*
>> +	 * The internal divider will always divide the output LVDS clock by 7
>> +	 * so our display needs 33246000 Hz, so set VIDEO_PLL1 to
>> +	 * 33246000 * 7 = 232722000 Hz
>> +	 */
>> +	assigned-clock-rates = <500000000>, <200000000>, <0>, <0>, <232722000>;
>> +};
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>> new file mode 100644
>> index 000000000000..06d1883b962a
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>> @@ -0,0 +1,13 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>> + */
>> +/dts-v1/;
>> +/plugin/;
>> +
>> +#include "imx8mp-aristainetos3-helios-lvds.dtsi"
>> +
>> +&{/} {
>> +	model = "Aristainetos3 helios LVDS carrier";
>> +	compatible = "abb,aristainetos3-helios-lvds", "abb,aristainetos3-helios", "abb,aristianetos3-som", "fsl,imx8mp";
> 
> Read not only DTS coding style, but also kernel coding style. Lines are
> supposed to be wrapped according to kernel coding style.

Yes, indeed, I have to fix my scripts...

>> +};
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>> new file mode 100644
>> index 000000000000..b4b1cb3b0cb3
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>> @@ -0,0 +1,103 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>> + */
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +
>> +&{/} {
>> +	helios_gpio_leds {
> 
> Node names should be generic. See also an explanation and list of
> examples (not exhaustive) in DT specification:
> https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
> 
> 
>> +		compatible = "gpio-leds";
>> +
>> +		helios_blue {
> 
> So this was absolutely never tested.
> 
> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check W=1` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).
> 
>> +			label = "helios:blue";
> 
> Use function and color instead.
> 
> I finished review here. Rest of the code does not look good, really. You
> have so many, really so many, trivial issues which tools point out, that
> using humans for such review is just waste of our time.

Sorry for that, will call the tools...

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28 10:41     ` Heiko Schocher
@ 2024-10-28 10:49       ` Krzysztof Kozlowski
  2024-10-28 11:21         ` Heiko Schocher
  0 siblings, 1 reply; 12+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-28 10:49 UTC (permalink / raw)
  To: hs, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

On 28/10/2024 11:41, Heiko Schocher wrote:
> Hello Krzysztof,
> 
> On 28.10.24 11:24, Krzysztof Kozlowski wrote:
>> On 28/10/2024 09:23, Heiko Schocher wrote:
>>> Add support for the i.MX8MP based aristainetos3 boards from ABB.
>>>
>>> The board uses a ABB specific SoM from ADLink, based on NXP
>>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>>> with small differences, which are all catched up in
>>> devicetree overlays. The kernel image, the basic dtb
>>> and all dtbos are collected in a fitimage. As bootloader
>>> is used U-Boot which detects in his SPL stage the carrier
>>> board by probing some i2c devices. When the correct
>>> carrier is probed, the SPL applies all needed dtbos to
>>> the dtb with which U-Boot gets loaded. Same principle
>>> later before linux image boot, U-Boot applies the dtbos
>>> needed for the carrier board before booting Linux.
>>>
>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>> ---
>>> checkpatch dropped the following warnings:
>>> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
>>>
>>> ignored, as this compatible string is usedin other dts too, for example in
>>>
>>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>>>
>>>   arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>>   .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>>   .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>>   .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>>   .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>>   .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>>   .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>>   .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>>   .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>>   .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>>   .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>>   11 files changed, 1718 insertions(+)
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>>
>>> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
>>> index 9d3df8b218a2..7c3586509b8b 100644
>>> --- a/arch/arm64/boot/dts/freescale/Makefile
>>> +++ b/arch/arm64/boot/dts/freescale/Makefile
>>> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>>>   dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>>>   dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>>>   
>>> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
>>> +			  imx8mp-aristainetos3-adpismarc.dtbo \
>>> +			  imx8mp-aristainetos3-proton2s.dtbo \
>>> +			  imx8mp-aristainetos3-helios.dtbo \
>>> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>>>   dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
>>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>> new file mode 100644
>>> index 000000000000..cc0cddaa33ea
>>> --- /dev/null
>>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>> @@ -0,0 +1,64 @@
>>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>>> +/*
>>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>>> + */
>>> +
>>> +#include <dt-bindings/gpio/gpio.h>
>>> +#include <dt-bindings/interrupt-controller/irq.h>
>>> +
>>> +&ecspi1 {
>>> +	spidev0: spi@0 {
>>> +		reg = <0>;
>>> +		compatible = "rohm,dh2228fv";
>>
>> Hm? I have some doubts, what device is here?
> 
> $ grep -lr dh2228fv drivers/
> drivers/spi/spidev.c
> 
> Customer uses an userspace implementation...

That's not the question. I asked what device is here.


> 
>>
>>> +		spi-max-frequency = <500000>;
>>> +	};
>>> +};
>>> +
>>> +&ecspi2 {
>>> +	spidev1: spi@0 {
>>> +		reg = <0>;
>>> +		compatible = "rohm,dh2228fv";
>>> +		spi-max-frequency = <500000>;
>>> +	};
>>> +};
>>> +
>>> +&i2c2 {
>>> +	/* SX1509(2) U1001@IPi SMARC Plus */
>>> +	gpio8: i2c2_gpioext0@3e {
>>
>> Uh, no, please never send us downstream code.
>>
>> Please follow DTS coding style in all upstream submissions.
> 
> driver is in here:
> 
> $ grep -lr probe-reset drivers/pinctrl/
> drivers/pinctrl/pinctrl-sx150x.c

This so not related... Your driver does not matter. You send us poor
quality downstream code.

...

> 
>> And why this is DTSO, I have no clue...

Why is this a DTSO, not a DTS?


Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28 10:49       ` Krzysztof Kozlowski
@ 2024-10-28 11:21         ` Heiko Schocher
  2024-10-28 12:44           ` Krzysztof Kozlowski
  0 siblings, 1 reply; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28 11:21 UTC (permalink / raw)
  To: Krzysztof Kozlowski, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

Hello Krzysztof,

On 28.10.24 11:49, Krzysztof Kozlowski wrote:
> On 28/10/2024 11:41, Heiko Schocher wrote:
>> Hello Krzysztof,
>>
>> On 28.10.24 11:24, Krzysztof Kozlowski wrote:
>>> On 28/10/2024 09:23, Heiko Schocher wrote:
>>>> Add support for the i.MX8MP based aristainetos3 boards from ABB.
>>>>
>>>> The board uses a ABB specific SoM from ADLink, based on NXP
>>>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>>>> with small differences, which are all catched up in
>>>> devicetree overlays. The kernel image, the basic dtb
>>>> and all dtbos are collected in a fitimage. As bootloader
>>>> is used U-Boot which detects in his SPL stage the carrier
>>>> board by probing some i2c devices. When the correct
>>>> carrier is probed, the SPL applies all needed dtbos to
>>>> the dtb with which U-Boot gets loaded. Same principle
>>>> later before linux image boot, U-Boot applies the dtbos
>>>> needed for the carrier board before booting Linux.
>>>>
>>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>>> ---
>>>> checkpatch dropped the following warnings:
>>>> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
>>>>
>>>> ignored, as this compatible string is usedin other dts too, for example in
>>>>
>>>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>>>>
>>>>    arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>>>    .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>>>    .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>>>    .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>>>    .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>>>    .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>>>    .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>>>    .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>>>    .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>>>    .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>>>    .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>>>    11 files changed, 1718 insertions(+)
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>>>
>>>> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
>>>> index 9d3df8b218a2..7c3586509b8b 100644
>>>> --- a/arch/arm64/boot/dts/freescale/Makefile
>>>> +++ b/arch/arm64/boot/dts/freescale/Makefile
>>>> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>>>>    
>>>> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
>>>> +			  imx8mp-aristainetos3-adpismarc.dtbo \
>>>> +			  imx8mp-aristainetos3-proton2s.dtbo \
>>>> +			  imx8mp-aristainetos3-helios.dtbo \
>>>> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
>>>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>> new file mode 100644
>>>> index 000000000000..cc0cddaa33ea
>>>> --- /dev/null
>>>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>> @@ -0,0 +1,64 @@
>>>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>>>> +/*
>>>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>>>> + */
>>>> +
>>>> +#include <dt-bindings/gpio/gpio.h>
>>>> +#include <dt-bindings/interrupt-controller/irq.h>
>>>> +
>>>> +&ecspi1 {
>>>> +	spidev0: spi@0 {
>>>> +		reg = <0>;
>>>> +		compatible = "rohm,dh2228fv";
>>>
>>> Hm? I have some doubts, what device is here?
>>
>> $ grep -lr dh2228fv drivers/
>> drivers/spi/spidev.c
>>
>> Customer uses an userspace implementation...
> 
> That's not the question. I asked what device is here.

I do not know, as on carrier boards there are only connectors,
to which a spi device can be attached. So may I need to use here
a more generic entry?

>>
>>>
>>>> +		spi-max-frequency = <500000>;
>>>> +	};
>>>> +};
>>>> +
>>>> +&ecspi2 {
>>>> +	spidev1: spi@0 {
>>>> +		reg = <0>;
>>>> +		compatible = "rohm,dh2228fv";
>>>> +		spi-max-frequency = <500000>;
>>>> +	};
>>>> +};
>>>> +
>>>> +&i2c2 {
>>>> +	/* SX1509(2) U1001@IPi SMARC Plus */
>>>> +	gpio8: i2c2_gpioext0@3e {
>>>
>>> Uh, no, please never send us downstream code.
>>>
>>> Please follow DTS coding style in all upstream submissions.
>>
>> driver is in here:
>>
>> $ grep -lr probe-reset drivers/pinctrl/
>> drivers/pinctrl/pinctrl-sx150x.c
> 
> This so not related... Your driver does not matter. You send us poor
> quality downstream code.

The driver is upstream... see:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/pinctrl/pinctrl-sx150x.c

or may I misunderstood you here too?

Poor is my dts, checks are running and I fix them.

> ...
> 
>>
>>> And why this is DTSO, I have no clue...
> 
> Why is this a DTSO, not a DTS?

Hmm... the idea is, that the bootloader applies the dtbo on runtime,
when it has detected the carrier board it runs on, I tried to explain
in cover letter.

And again... sorry...

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28 11:21         ` Heiko Schocher
@ 2024-10-28 12:44           ` Krzysztof Kozlowski
  2024-10-28 13:52             ` Heiko Schocher
  0 siblings, 1 reply; 12+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-28 12:44 UTC (permalink / raw)
  To: hs, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

On 28/10/2024 12:21, Heiko Schocher wrote:
> Hello Krzysztof,
> 
> On 28.10.24 11:49, Krzysztof Kozlowski wrote:
>> On 28/10/2024 11:41, Heiko Schocher wrote:
>>> Hello Krzysztof,
>>>
>>> On 28.10.24 11:24, Krzysztof Kozlowski wrote:
>>>> On 28/10/2024 09:23, Heiko Schocher wrote:
>>>>> Add support for the i.MX8MP based aristainetos3 boards from ABB.
>>>>>
>>>>> The board uses a ABB specific SoM from ADLink, based on NXP
>>>>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>>>>> with small differences, which are all catched up in
>>>>> devicetree overlays. The kernel image, the basic dtb
>>>>> and all dtbos are collected in a fitimage. As bootloader
>>>>> is used U-Boot which detects in his SPL stage the carrier
>>>>> board by probing some i2c devices. When the correct
>>>>> carrier is probed, the SPL applies all needed dtbos to
>>>>> the dtb with which U-Boot gets loaded. Same principle
>>>>> later before linux image boot, U-Boot applies the dtbos
>>>>> needed for the carrier board before booting Linux.
>>>>>
>>>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>>>> ---
>>>>> checkpatch dropped the following warnings:
>>>>> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
>>>>>
>>>>> ignored, as this compatible string is usedin other dts too, for example in
>>>>>
>>>>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>>>>>
>>>>>    arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>>>>    .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>>>>    .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>>>>    .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>>>>    .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>>>>    .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>>>>    .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>>>>    .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>>>>    .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>>>>    .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>>>>    .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>>>>    11 files changed, 1718 insertions(+)
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>>>>    create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>>>>
>>>>> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
>>>>> index 9d3df8b218a2..7c3586509b8b 100644
>>>>> --- a/arch/arm64/boot/dts/freescale/Makefile
>>>>> +++ b/arch/arm64/boot/dts/freescale/Makefile
>>>>> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>>>>>    
>>>>> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
>>>>> +			  imx8mp-aristainetos3-adpismarc.dtbo \
>>>>> +			  imx8mp-aristainetos3-proton2s.dtbo \
>>>>> +			  imx8mp-aristainetos3-helios.dtbo \
>>>>> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>>>>>    dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
>>>>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>> new file mode 100644
>>>>> index 000000000000..cc0cddaa33ea
>>>>> --- /dev/null
>>>>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>> @@ -0,0 +1,64 @@
>>>>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>>>>> +/*
>>>>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>>>>> + */
>>>>> +
>>>>> +#include <dt-bindings/gpio/gpio.h>
>>>>> +#include <dt-bindings/interrupt-controller/irq.h>
>>>>> +
>>>>> +&ecspi1 {
>>>>> +	spidev0: spi@0 {
>>>>> +		reg = <0>;
>>>>> +		compatible = "rohm,dh2228fv";
>>>>
>>>> Hm? I have some doubts, what device is here?
>>>
>>> $ grep -lr dh2228fv drivers/
>>> drivers/spi/spidev.c
>>>
>>> Customer uses an userspace implementation...
>>
>> That's not the question. I asked what device is here.
> 
> I do not know, as on carrier boards there are only connectors,
> to which a spi device can be attached. So may I need to use here
> a more generic entry?

So this description is just not true. You have here nothing connected
and this cannot be in the DTS.

> 
>>>
>>>>
>>>>> +		spi-max-frequency = <500000>;
>>>>> +	};
>>>>> +};
>>>>> +
>>>>> +&ecspi2 {
>>>>> +	spidev1: spi@0 {
>>>>> +		reg = <0>;
>>>>> +		compatible = "rohm,dh2228fv";
>>>>> +		spi-max-frequency = <500000>;
>>>>> +	};
>>>>> +};
>>>>> +
>>>>> +&i2c2 {
>>>>> +	/* SX1509(2) U1001@IPi SMARC Plus */
>>>>> +	gpio8: i2c2_gpioext0@3e {
>>>>
>>>> Uh, no, please never send us downstream code.
>>>>
>>>> Please follow DTS coding style in all upstream submissions.
>>>
>>> driver is in here:
>>>
>>> $ grep -lr probe-reset drivers/pinctrl/
>>> drivers/pinctrl/pinctrl-sx150x.c
>>
>> This so not related... Your driver does not matter. You send us poor
>> quality downstream code.
> 
> The driver is upstream... see:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/pinctrl/pinctrl-sx150x.c
> 
> or may I misunderstood you here too?
> 
> Poor is my dts, checks are running and I fix them.

My comment was that I see this as you sent DTS code which is taken
straight from some downstream code.

>>>> And why this is DTSO, I have no clue...
>>
>> Why is this a DTSO, not a DTS?
> 
> Hmm... the idea is, that the bootloader applies the dtbo on runtime,
> when it has detected the carrier board it runs on, I tried to explain
> in cover letter.

Then there is some mess here. First, SoM cannot be DTS, because it
cannot be booted. Second, specific board/carrier is the DTS. Third,
overlays bring some subset of features, not new board.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support
  2024-10-28 12:44           ` Krzysztof Kozlowski
@ 2024-10-28 13:52             ` Heiko Schocher
  0 siblings, 0 replies; 12+ messages in thread
From: Heiko Schocher @ 2024-10-28 13:52 UTC (permalink / raw)
  To: Krzysztof Kozlowski, linux-kernel
  Cc: Conor Dooley, Fabio Estevam, Krzysztof Kozlowski,
	Pengutronix Kernel Team, Rob Herring, Sascha Hauer, Shawn Guo,
	devicetree, imx, linux-arm-kernel

Hello Krzysztof,

On 28.10.24 13:44, Krzysztof Kozlowski wrote:
> On 28/10/2024 12:21, Heiko Schocher wrote:
>> Hello Krzysztof,
>>
>> On 28.10.24 11:49, Krzysztof Kozlowski wrote:
>>> On 28/10/2024 11:41, Heiko Schocher wrote:
>>>> Hello Krzysztof,
>>>>
>>>> On 28.10.24 11:24, Krzysztof Kozlowski wrote:
>>>>> On 28/10/2024 09:23, Heiko Schocher wrote:
>>>>>> Add support for the i.MX8MP based aristainetos3 boards from ABB.
>>>>>>
>>>>>> The board uses a ABB specific SoM from ADLink, based on NXP
>>>>>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>>>>>> with small differences, which are all catched up in
>>>>>> devicetree overlays. The kernel image, the basic dtb
>>>>>> and all dtbos are collected in a fitimage. As bootloader
>>>>>> is used U-Boot which detects in his SPL stage the carrier
>>>>>> board by probing some i2c devices. When the correct
>>>>>> carrier is probed, the SPL applies all needed dtbos to
>>>>>> the dtb with which U-Boot gets loaded. Same principle
>>>>>> later before linux image boot, U-Boot applies the dtbos
>>>>>> needed for the carrier board before booting Linux.
>>>>>>
>>>>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>>>>> ---
>>>>>> checkpatch dropped the following warnings:
>>>>>> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:248: warning: DT compatible string "ethernet-phy-id2000.a231" appears un-documented -- check ./Documentation/devicetree/bindings/
>>>>>>
>>>>>> ignored, as this compatible string is usedin other dts too, for example in
>>>>>>
>>>>>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>>>>>>
>>>>>>     arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>>>>>     .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>>>>>     .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>>>>>     .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>>>>>     .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>>>>>     .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>>>>>     .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>>>>>     .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>>>>>     .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>>>>>     .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>>>>>     .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>>>>>     11 files changed, 1718 insertions(+)
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>>>>>     create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>>>>>
>>>>>> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
>>>>>> index 9d3df8b218a2..7c3586509b8b 100644
>>>>>> --- a/arch/arm64/boot/dts/freescale/Makefile
>>>>>> +++ b/arch/arm64/boot/dts/freescale/Makefile
>>>>>> @@ -163,6 +163,11 @@ imx8mn-tqma8mqnl-mba8mx-usbotg-dtbs += imx8mn-tqma8mqnl-mba8mx.dtb imx8mn-tqma8m
>>>>>>     dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-lvds-tm070jvhg33.dtb
>>>>>>     dtb-$(CONFIG_ARCH_MXC) += imx8mn-tqma8mqnl-mba8mx-usbotg.dtb
>>>>>>     
>>>>>> +dtb-$(CONFIG_ARCH_MXC) += imx8mp-aristainetos3a-som-v1.dtb \
>>>>>> +			  imx8mp-aristainetos3-adpismarc.dtbo \
>>>>>> +			  imx8mp-aristainetos3-proton2s.dtbo \
>>>>>> +			  imx8mp-aristainetos3-helios.dtbo \
>>>>>> +			  imx8mp-aristainetos3-helios-lvds.dtbo
>>>>>>     dtb-$(CONFIG_ARCH_MXC) += imx8mp-beacon-kit.dtb
>>>>>>     dtb-$(CONFIG_ARCH_MXC) += imx8mp-data-modul-edm-sbc.dtb
>>>>>>     dtb-$(CONFIG_ARCH_MXC) += imx8mp-debix-model-a.dtb
>>>>>> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>>> new file mode 100644
>>>>>> index 000000000000..cc0cddaa33ea
>>>>>> --- /dev/null
>>>>>> +++ b/arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>>>>> @@ -0,0 +1,64 @@
>>>>>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>>>>>> +/*
>>>>>> + * Copyright (C) 2024 Heiko Schocher <hs@denx.de>
>>>>>> + */
>>>>>> +
>>>>>> +#include <dt-bindings/gpio/gpio.h>
>>>>>> +#include <dt-bindings/interrupt-controller/irq.h>
>>>>>> +
>>>>>> +&ecspi1 {
>>>>>> +	spidev0: spi@0 {
>>>>>> +		reg = <0>;
>>>>>> +		compatible = "rohm,dh2228fv";
>>>>>
>>>>> Hm? I have some doubts, what device is here?
>>>>
>>>> $ grep -lr dh2228fv drivers/
>>>> drivers/spi/spidev.c
>>>>
>>>> Customer uses an userspace implementation...
>>>
>>> That's not the question. I asked what device is here.
>>
>> I do not know, as on carrier boards there are only connectors,
>> to which a spi device can be attached. So may I need to use here
>> a more generic entry?
> 
> So this description is just not true. You have here nothing connected
> and this cannot be in the DTS.

Okay, I try to find out, what devices are connected else I remove them.

>>>>
>>>>>
>>>>>> +		spi-max-frequency = <500000>;
>>>>>> +	};
>>>>>> +};
>>>>>> +
>>>>>> +&ecspi2 {
>>>>>> +	spidev1: spi@0 {
>>>>>> +		reg = <0>;
>>>>>> +		compatible = "rohm,dh2228fv";
>>>>>> +		spi-max-frequency = <500000>;
>>>>>> +	};
>>>>>> +};
>>>>>> +
>>>>>> +&i2c2 {
>>>>>> +	/* SX1509(2) U1001@IPi SMARC Plus */
>>>>>> +	gpio8: i2c2_gpioext0@3e {
>>>>>
>>>>> Uh, no, please never send us downstream code.
>>>>>
>>>>> Please follow DTS coding style in all upstream submissions.
>>>>
>>>> driver is in here:
>>>>
>>>> $ grep -lr probe-reset drivers/pinctrl/
>>>> drivers/pinctrl/pinctrl-sx150x.c
>>>
>>> This so not related... Your driver does not matter. You send us poor
>>> quality downstream code.
>>
>> The driver is upstream... see:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/pinctrl/pinctrl-sx150x.c
>>
>> or may I misunderstood you here too?
>>
>> Poor is my dts, checks are running and I fix them.
> 
> My comment was that I see this as you sent DTS code which is taken
> straight from some downstream code.

Hmm.. I made this based on linux-stable 6.6 and yes, comments
I have from adlink sources.

Again sorry... I was too fast sending my patch after local rebase
to v6.12

>>>>> And why this is DTSO, I have no clue...
>>>
>>> Why is this a DTSO, not a DTS?
>>
>> Hmm... the idea is, that the bootloader applies the dtbo on runtime,
>> when it has detected the carrier board it runs on, I tried to explain
>> in cover letter.
> 
> Then there is some mess here. First, SoM cannot be DTS, because it
> cannot be booted. Second, specific board/carrier is the DTS. Third,
> overlays bring some subset of features, not new board.

I see, and will rework!

BTW: I now finished running

make W=1 O=$BDIR dt_binding_check
make W=1 O=$BDIR dtbs_check

and there are tons of warnings .. yes, a lot come from my changes
(because I wasn't aware of the checks and I made stuff based on linux 6.6
  stable and simply just rebased my work with 6.12-rc5 so I have to
  apologize!)

But not all warnings come from my files...

I will check, fix, rework my patchset!

Thanks for your comments and patience!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier
  2024-10-28  8:23 [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Heiko Schocher
  2024-10-28  8:23 ` [PATCH v1 1/2] dt-bindings: arm: fsl: Add " Heiko Schocher
  2024-10-28  8:23 ` [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support Heiko Schocher
@ 2024-10-29 12:44 ` Rob Herring (Arm)
  2024-10-29 12:50   ` Heiko Schocher
  2 siblings, 1 reply; 12+ messages in thread
From: Rob Herring (Arm) @ 2024-10-29 12:44 UTC (permalink / raw)
  To: Heiko Schocher
  Cc: Hugo Villeneuve, Joao Paulo Goncalves, Fabio Estevam,
	Krzysztof Kozlowski, Hiago De Franco, imx, Gregor Herburger,
	Shawn Guo, Peng Fan, devicetree, Conor Dooley, Mathieu Othacehe,
	linux-kernel, Sascha Hauer, Pengutronix Kernel Team,
	Alexander Stein, Tim Harvey, Michael Walle, Max Merchel,
	linux-arm-kernel


On Mon, 28 Oct 2024 09:23:30 +0100, Heiko Schocher wrote:
> The board uses a ABB specific SoM from ADLink, based on NXP
> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
> with small differences, which are all catched up in
> devicetree overlays. The kernel image, the basic dtb
> and all dtbos are collected in a fitimage. As bootloader
> is used U-Boot which detects in his SPL stage the carrier
> board by probing some i2c devices. When the correct
> carrier is probed, the SPL applies all needed dtbos to
> the dtb with which U-Boot gets loaded. Same principle
> later before linux image boot, U-Boot applies the dtbos
> needed for the carrier board before booting Linux.
> 
> So intentionally, there are no full dtb images for all
> carrier combinations, only dtbos which applied as needed
> on boottime.
> 
> Patches to U-Boot mailinglist not posted yet, as U-Boot
> uses the same dtb as linux, so hope to get the linux
> files first in, and use them when posting U-Boot board
> support.
> 
> series is based on:
> 819837584309 - (tag: v6.12-rc5, origin/master, origin/HEAD) Linux 6.12-rc5
> 
> 
> 
> Heiko Schocher (2):
>   dt-bindings: arm: fsl: Add ABB SoM and carrier
>   arm64: dts: imx8mp: add aristainetos3 board support
> 
>  .../devicetree/bindings/arm/fsl.yaml          |    2 +
>  arch/arm64/boot/dts/freescale/Makefile        |    5 +
>  .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>  .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>  .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>  .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>  .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>  .../imx8mp-aristainetos3-helios.dtso          |   13 +
>  .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>  .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>  .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>  .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>  12 files changed, 1720 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
> 
> --
> 2.20.1
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y freescale/imx8mp-aristainetos3a-som-v1.dtb' for 20241028082332.21672-1-hs@denx.de:

arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:527.14-540.4: Warning (interrupt_provider): /soc@0/bus@30800000/i2c@30ae0000/tpm@2e: '#interrupt-cells' found, but node is not an interrupt provider
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: Warning (interrupt_map): Failed prerequisite 'interrupt_provider'
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
	['abb,imx8mp-aristianetos3', 'abb,imx8mp-aristianetos3-som', 'fsl,imx8mp'] is too long
	['abb,imx8mp-aristianetos3', 'abb,imx8mp-aristianetos3-som', 'fsl,imx8mp'] is too short
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
		'abb,imx8mp-aristianetos3' is not one of ['tq,imx8dxp-tqma8xdp-mba8xx']
		'abb,imx8mp-aristianetos3' is not one of ['tq,imx8qxp-tqma8xqp-mba8xx']
		'tq,imx8dxp-tqma8xdp' was expected
		'tq,imx8qxp-tqma8xqp' was expected
		'fsl,imx8dxp' was expected
		'fsl,imx8qxp' was expected
	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx1-apf9328', 'fsl,imx1ads']
	'abb,imx8mp-aristianetos3' is not one of ['creative,x-fi3', 'fsl,imx23-evk', 'fsl,stmp378x-devb', 'olimex,imx23-olinuxino', 'sandisk,sansa_fuze_plus']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx25-pdk', 'karo,imx25-tx25']
	'abb,imx8mp-aristianetos3' is not one of ['eukrea,mbimxsd25-baseboard']
	'abb,imx8mp-aristianetos3' is not one of ['eukrea,mbimxsd25-baseboard-cmo-qvga', 'eukrea,mbimxsd25-baseboard-dvi-svga', 'eukrea,mbimxsd25-baseboard-dvi-vga']
	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx27-apf27', 'fsl,imx27-pdk']
	'armadeus,imx27-apf27dev' was expected
	'eukrea,mbimxsd27-baseboard' was expected
	'phytec,imx27-pca100-rdk' was expected
	'phytec,imx27-pcm970' was expected
	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx28-apf28', 'bluegiga,apx4devkit', 'crystalfontz,cfa10036', 'eukrea,mbmx28lc', 'fsl,imx28-evk', 'i2se,duckbill', 'i2se,duckbill-2', 'karo,tx28', 'lwn,imx28-xea', 'msr,m28cu3', 'schulercontrol,imx28-sps1', 'technologic,imx28-ts4600']
	'aries,m28' was expected
	'aries,m28evk' was expected
	'armadeus,imx28-apf28dev' was expected
	'abb,imx8mp-aristianetos3' is not one of ['crystalfontz,cfa10037', 'crystalfontz,cfa10049', 'crystalfontz,cfa10057', 'crystalfontz,cfa10058']
	'abb,imx8mp-aristianetos3' is not one of ['crystalfontz,cfa10055', 'crystalfontz,cfa10056']
	'abb,imx8mp-aristianetos3' is not one of ['i2se,duckbill-2-485', 'i2se,duckbill-2-enocean', 'i2se,duckbill-2-spi']
	'eukrea,mbmx283lc' was expected
	'eukrea,mbmx287lc' was expected
	'abb,imx8mp-aristianetos3' is not one of ['buglabs,imx31-bug', 'logicpd,imx31-lite']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx35-pdk']
	'eukrea,mbimxsd35-baseboard' was expected
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx50-evk', 'kobo,aura']
	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx51-apf51', 'fsl,imx51-babbage', 'technologic,imx51-ts4800', 'zii,imx51-scu3-esb', 'zii,imx51-scu2-mezz', 'zii,imx51-rdu1']
	'armadeus,imx51-apf51dev' was expected
	'digi,connectcore-ccxmx51-jsk' was expected
	'eukrea,mbimxsd51' was expected
	'abb,imx8mp-aristianetos3' is not one of ['bhf,cx9020', 'fsl,imx53-ard', 'fsl,imx53-evk', 'fsl,imx53-qsb', 'fsl,imx53-qsrb', 'fsl,imx53-smd', 'ge,imx53-cpuvo', 'inversepath,imx53-usbarmory', 'karo,tx53', 'kiebackpeter,imx53-ddc', 'kiebackpeter,imx53-hsc', 'menlo,m53menlo', 'starterkit,sk-imx53', 'voipac,imx53-dmm-668']
	'aries,imx53-m53evk' was expected
	'tq,mba53' was expected
	'abb,imx8mp-aristianetos3' is not one of ['auvidea,h100', 'bosch,imx6q-acc', 'boundary,imx6q-nitrogen6_max', 'boundary,imx6q-nitrogen6_som2', 'boundary,imx6q-nitrogen6x', 'compulab,cm-fx6', 'dmo,imx6q-edmqmx6', 'ds,imx6q-sbc', 'embest,imx6q-marsboard', 'emtrion,emcon-mx6', 'emtrion,emcon-mx6-avari', 'engicam,imx6-icore', 'engicam,imx6-icore-rqs', 'fsl,imx6q-arm2', 'fsl,imx6q-sabreauto', 'fsl,imx6q-sabrelite', 'fsl,imx6q-sabresd', 'karo,imx6q-tx6q', 'kiebackpeter,imx6q-tpc', 'kontron,imx6q-samx6i', 'kosagi,imx6q-novena', 'kvg,vicut1q', 'logicpd,imx6q-logicpd', 'lwn,display5', 'lwn,mccmon6', 'nutsboard,imx6q-pistachio', 'microsys,sbc6x', 'poslab,imx6q-savageboard', 'prt,prti6q', 'prt,prtwd2', 'rex,imx6q-rex-pro', 'skov,imx6q-skov-revc-lt2', 'skov,imx6q-skov-revc-lt6', 'skov,imx6q-skov-reve-mi1010ait-1cp1', 'solidrun,cubox-i/q', 'solidrun,hummingboard/q', 'solidrun,hummingboard2/q', 'solidrun,solidsense/q', 'tbs,imx6q-tbs2910', 'technexion,imx6q-pico-dwarf', 'technexion,imx6q-pico-hobbit', 'technexion,imx6q-pico-nymph', 'technexion,imx6q-pico-pi', 'technologic,imx6q-ts4900', 'technologic,imx6q-ts7970', 'toradex,apalis_imx6q', 'udoo,imx6q-udoo', 'uniwest,imx6q-evi', 'variscite,dt6customboard', 'wand,imx6q-wandboard', 'ysoft,imx6q-yapp4-crux', 'ysoft,imx6q-yapp4-pegasus', 'zealz,imx6q-gk802', 'zii,imx6q-zii-rdu2']
	'abb,imx8mp-aristianetos3' is not one of ['advantech,imx6q-dms-ba16', 'ge,imx6q-b450v3', 'ge,imx6q-b650v3', 'ge,imx6q-b850v3']
	'armadeus,imx6q-apf6dev' was expected
	'compulab,utilite-pro' was expected
	'dfi,fs700-m60-6qd' was expected
	'dh,imx6q-dhcom-pdk2' was expected
	'abb,imx8mp-aristianetos3' is not one of ['gw,imx6q-gw51xx', 'gw,imx6q-gw52xx', 'gw,imx6q-gw53xx', 'gw,imx6q-gw5400-a', 'gw,imx6q-gw54xx', 'gw,imx6q-gw551x', 'gw,imx6q-gw552x', 'gw,imx6q-gw553x', 'gw,imx6q-gw560x', 'gw,imx6q-gw5903', 'gw,imx6q-gw5904', 'gw,imx6q-gw5907', 'gw,imx6q-gw5910', 'gw,imx6q-gw5912', 'gw,imx6q-gw5913']
	'kontron,imx6q-samx6i-ads2' was expected
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6q-pbac06-emmc', 'phytec,imx6q-pbac06-nand']
	'abb,imx8mp-aristianetos3' is not one of ['comvetia,imx6q-lxr', 'phytec,imx6q-pbab01']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis_imx6q-ixora', 'toradex,apalis_imx6q-ixora-v1.1', 'toradex,apalis_imx6q-ixora-v1.2', 'toradex,apalis_imx6q-eval', 'toradex,apalis_imx6q-eval-v1.2']
	'variscite,mx6customboard' was expected
	'tq,imx6q-mba6x-a' was expected
	'tq,imx6q-mba6x-b' was expected
	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx6qp-nitrogen6_max', 'boundary,imx6qp-nitrogen6_som2', 'fsl,imx6qp-sabreauto', 'fsl,imx6qp-sabresd', 'karo,imx6qp-tx6qp', 'kvg,vicutp', 'prt,prtwd3', 'wand,imx6qp-wandboard', 'ysoft,imx6qp-yapp4-crux-plus', 'ysoft,imx6qp-yapp4-pegasus-plus', 'zii,imx6qp-zii-rdu2']
	'phytec,imx6qp-pbac06-nand' was expected
	'tq,imx6qp-mba6x-b' was expected
	'abb,imx8mp-aristianetos3' is not one of ['abb,aristainetos-imx6dl-4', 'abb,aristainetos-imx6dl-7', 'abb,aristainetos2-imx6dl-4', 'abb,aristainetos2-imx6dl-7', 'alt,alti6p', 'boundary,imx6dl-nit6xlite', 'boundary,imx6dl-nitrogen6x', 'bticino,imx6dl-mamoj', 'eckelmann,imx6dl-ci4x10', 'emtrion,emcon-mx6', 'emtrion,emcon-mx6-avari', 'engicam,imx6-icore', 'engicam,imx6-icore-rqs', 'fsl,imx6dl-sabreauto', 'fsl,imx6dl-sabrelite', 'fsl,imx6dl-sabresd', 'karo,imx6dl-tx6dl', 'kontron,imx6dl-samx6i', 'kvg,victgo', 'kvg,vicut1', 'ply,plybas', 'ply,plym2m', 'poslab,imx6dl-savageboard', 'prt,prtmvt', 'prt,prtrvt', 'prt,prtvt7', 'rex,imx6dl-rex-basic', 'riot,imx6s-riotboard', 'sielaff,imx6dl-board', 'skov,imx6dl-skov-revc-lt2', 'skov,imx6dl-skov-revc-lt6', 'solidrun,cubox-i/dl', 'solidrun,hummingboard/dl', 'solidrun,hummingboard2/dl', 'solidrun,solidsense/dl', 'technexion,imx6dl-pico-dwarf', 'technexion,imx6dl-pico-hobbit', 'technexion,imx6dl-pico-nymph', 'technexion,imx6dl-pico-pi', 'technologic,imx6dl-ts4900', 'technologic,imx6dl-ts7970', 'toradex,colibri_imx6dl', 'udoo,imx6dl-udoo', 'vdl,lanmcu', 'wand,imx6dl-wandboard', 'ysoft,imx6dl-yapp4-draco', 'ysoft,imx6dl-yapp4-hydra', 'ysoft,imx6dl-yapp4-lynx', 'ysoft,imx6dl-yapp4-orion', 'ysoft,imx6dl-yapp4-phoenix', 'ysoft,imx6dl-yapp4-ursa']
	'armadeus,imx6dl-apf6dev' was expected
	'abb,imx8mp-aristianetos3' is not one of ['ge,imx6dl-b105v2', 'ge,imx6dl-b105pv2', 'ge,imx6dl-b125v2', 'ge,imx6dl-b125pv2', 'ge,imx6dl-b155v2']
	'dfi,fs700-m60-6dl' was expected
	'abb,imx8mp-aristianetos3' is not one of ['dh,imx6dl-dhcom-pdk2', 'dh,imx6dl-dhcom-picoitx']
	'abb,imx8mp-aristianetos3' is not one of ['gw,imx6dl-gw51xx', 'gw,imx6dl-gw52xx', 'gw,imx6dl-gw53xx', 'gw,imx6dl-gw54xx', 'gw,imx6dl-gw551x', 'gw,imx6dl-gw552x', 'gw,imx6dl-gw553x', 'gw,imx6dl-gw560x', 'gw,imx6dl-gw5903', 'gw,imx6dl-gw5904', 'gw,imx6dl-gw5907', 'gw,imx6dl-gw5910', 'gw,imx6dl-gw5912', 'gw,imx6dl-gw5913']
	'kontron,imx6dl-samx6i-ads2' was expected
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6dl-pbac06-emmc', 'phytec,imx6dl-pbac06-nand']
	'phytec,imx6dl-pbab01' was expected
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri_imx6dl-aster', 'toradex,colibri_imx6dl-eval-v3', 'toradex,colibri_imx6dl-iris', 'toradex,colibri_imx6dl-iris-v2']
	'dh,imx6s-dhcom-drc02' was expected
	'tq,imx6dl-mba6x-a' was expected
	'tq,imx6dl-mba6x-b' was expected
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6sl-evk', 'kobo,aura2', 'kobo,tolino-shine2hd', 'kobo,tolino-shine3', 'kobo,tolino-vision', 'kobo,tolino-vision5', 'revotics,imx6sl-warp']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6sll-evk', 'kobo,clarahd', 'kobo,librah2o']
	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx6sx-nitrogen6sx', 'fsl,imx6sx-sabreauto', 'fsl,imx6sx-sdb', 'fsl,imx6sx-sdb-reva', 'samtec,imx6sx-vining-2000', 'udoo,neobasic', 'udoo,neoextended', 'udoo,neofull']
	'abb,imx8mp-aristianetos3' is not one of ['engicam,imx6ul-geam', 'engicam,imx6ul-isiot', 'fsl,imx6ul-14x14-evk', 'karo,imx6ul-tx6ul', 'kontron,sl-imx6ul', 'prt,prti6g', 'technexion,imx6ul-pico-dwarf', 'technexion,imx6ul-pico-hobbit', 'technexion,imx6ul-pico-pi']
	'armadeus,imx6ul-opos6uldev' was expected
	'abb,imx8mp-aristianetos3' is not one of ['digi,ccimx6ulsbcexpress', 'digi,ccimx6ulsbcpro']
	'grinn,imx6ul-liteboard' was expected
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ul-pbacd10-emmc', 'phytec,imx6ul-pbacd10-nand']
	'kontron,bl-imx6ul' was expected
	'kontron,bl-imx6ul-43' was expected
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul1-mba6ulx']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul2-mba6ulx']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul2l-mba6ulx', 'tq,imx6ul-tqma6ul2l-mba6ulxl']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6ull-14x14-evk', 'joz,jozacp', 'kontron,sl-imx6ull', 'myir,imx6ull-mys-6ulx-eval', 'toradex,colibri-imx6ull', 'toradex,colibri-imx6ull-emmc', 'toradex,colibri-imx6ull-wifi', 'uni-t,uti260b']
	'armadeus,imx6ull-opos6uldev' was expected
	'abb,imx8mp-aristianetos3' is not one of ['chargebyte,imx6ull-tarragon-master', 'chargebyte,imx6ull-tarragon-micro', 'chargebyte,imx6ull-tarragon-slave', 'chargebyte,imx6ull-tarragon-slavext']
	'abb,imx8mp-aristianetos3' is not one of ['dh,imx6ull-dhcom-drc02', 'dh,imx6ull-dhcom-pdk2', 'dh,imx6ull-dhcom-picoitx']
	'marantec,imx6ull-dhcor-maveo-box' was expected
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ull-pbacd10-emmc', 'phytec,imx6ull-pbacd10-nand']
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ull-phygate-tauri-emmc', 'phytec,imx6ull-phygate-tauri-nand']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-aster', 'toradex,colibri-imx6ull-eval', 'toradex,colibri-imx6ull-iris', 'toradex,colibri-imx6ull-iris-v2']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-emmc-aster', 'toradex,colibri-imx6ull-emmc-eval', 'toradex,colibri-imx6ull-emmc-iris', 'toradex,colibri-imx6ull-emmc-iris-v2']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-wifi-eval', 'toradex,colibri-imx6ull-wifi-aster', 'toradex,colibri-imx6ull-wifi-iris', 'toradex,colibri-imx6ull-wifi-iris-v2']
	'kontron,bl-imx6ull' was expected
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ull-tqma6ull2-mba6ulx']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ull-tqma6ull2l-mba6ulx', 'tq,imx6ull-tqma6ull2l-mba6ulxl']
	'abb,imx8mp-aristianetos3' is not one of ['seeed,imx6ull-seeed-npi-emmc', 'seeed,imx6ull-seeed-npi-nand']
	'abb,imx8mp-aristianetos3' is not one of ['bsh,imx6ulz-bsh-smm-m2', 'fsl,imx6ulz-14x14-evk']
	'abb,imx8mp-aristianetos3' is not one of ['element14,imx7s-warp', 'toradex,colibri-imx7s']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7s-aster', 'toradex,colibri-imx7s-eval-v3', 'toradex,colibri-imx7s-iris', 'toradex,colibri-imx7s-iris-v2']
	'tq,imx7s-mba7' was expected
	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx7d-nitrogen7', 'compulab,cl-som-imx7', 'fsl,imx7d-sdb', 'fsl,imx7d-sdb-reva', 'kam,imx7d-flex-concentrator', 'kam,imx7d-flex-concentrator-mfg', 'novtech,imx7d-meerkat96', 'remarkable,imx7d-remarkable2', 'storopack,imx7d-smegw01', 'technexion,imx7d-pico-dwarf', 'technexion,imx7d-pico-hobbit', 'technexion,imx7d-pico-nymph', 'technexion,imx7d-pico-pi', 'toradex,colibri-imx7d', 'toradex,colibri-imx7d-emmc', 'zii,imx7d-rmu2', 'zii,imx7d-rpu2']
	'tq,imx7d-mba7' was expected
	'compulab,sbc-imx7' was expected
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7d-aster', 'toradex,colibri-imx7d-eval-v3', 'toradex,colibri-imx7d-iris', 'toradex,colibri-imx7d-iris-v2']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7d-emmc-aster', 'toradex,colibri-imx7d-emmc-eval-v3', 'toradex,colibri-imx7d-emmc-iris', 'toradex,colibri-imx7d-emmc-iris-v2']
	'abb,imx8mp-aristianetos3' is not one of ['ea,imx7ulp-com', 'fsl,imx7ulp-evk']
	'abb,imx8mp-aristianetos3' is not one of ['beacon,imx8mm-beacon-kit', 'boundary,imx8mm-nitrogen8mm', 'dmo,imx8mm-data-modul-edm-sbc', 'emtrion,emcon-mx8mm-avari', 'fsl,imx8mm-ddr4-evk', 'fsl,imx8mm-evk', 'fsl,imx8mm-evkb', 'gateworks,imx8mm-gw75xx-0x', 'gateworks,imx8mm-gw7904', 'gw,imx8mm-gw71xx-0x', 'gw,imx8mm-gw72xx-0x', 'gw,imx8mm-gw73xx-0x', 'gw,imx8mm-gw7901', 'gw,imx8mm-gw7902', 'gw,imx8mm-gw7903', 'innocomm,wb15-evk', 'kontron,imx8mm-sl', 'kontron,imx8mm-osm-s', 'toradex,verdin-imx8mm', 'toradex,verdin-imx8mm-nonwifi', 'toradex,verdin-imx8mm-wifi', 'prt,prt8mm']
	'abb,imx8mp-aristianetos3' is not one of ['compulab,imx8mm-iot-gateway']
	'ees,imx8mm-emtop-baseboard' was expected
	'abb,imx8mp-aristianetos3' is not one of ['engicam,icore-mx8mm-ctouch2', 'engicam,icore-mx8mm-edimm2.2']
	'kontron,imx8mm-bl' was expected
	'kontron,imx8mm-bl-osm-s' was expected
	'abb,imx8mp-aristianetos3' is not one of ['menlo,mx8menlo', 'toradex,verdin-imx8mm-nonwifi-dahlia', 'toradex,verdin-imx8mm-nonwifi-dev', 'toradex,verdin-imx8mm-nonwifi-ivy', 'toradex,verdin-imx8mm-nonwifi-mallow', 'toradex,verdin-imx8mm-nonwifi-yavia']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mm-wifi-dahlia', 'toradex,verdin-imx8mm-wifi-dev', 'toradex,verdin-imx8mm-wifi-ivy', 'toradex,verdin-imx8mm-wifi-mallow', 'toradex,verdin-imx8mm-wifi-yavia']
	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx8mm-phyboard-polis-rdk', 'phytec,imx8mm-phygate-tauri-l']
	'variscite,var-som-mx8mm-symphony' was expected
	'abb,imx8mp-aristianetos3' is not one of ['cloos,imx8mm-phg', 'tq,imx8mm-tqma8mqml-mba8mx']
	'abb,imx8mp-aristianetos3' is not one of ['beacon,imx8mn-beacon-kit', 'bsh,imx8mn-bsh-smm-s2', 'bsh,imx8mn-bsh-smm-s2pro', 'fsl,imx8mn-ddr3l-evk', 'fsl,imx8mn-ddr4-evk', 'fsl,imx8mn-evk', 'gw,imx8mn-gw7902']
	'abb,imx8mp-aristianetos3' is not one of ['dimonoff,gateway-evk', 'rve,gateway', 'variscite,var-som-mx8mn-symphony']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mn-tqma8mqnl-mba8mx']
	'avnet,sm2s-imx8mp-14N0600E-ep1' was expected
	'abb,imx8mp-aristianetos3' is not one of ['dh,imx8mp-dhcom-drc02', 'dh,imx8mp-dhcom-pdk2', 'dh,imx8mp-dhcom-pdk3', 'dh,imx8mp-dhcom-picoitx']
	'abb,imx8mp-aristianetos3' is not one of ['engicam,icore-mx8mp-edimm2.2']
	'kontron,imx8mp-bl-osm-s' was expected
	'kontron,imx8mp-smarc-eval-carrier' was expected
	'phytec,imx8mp-phyboard-pollux-rdk' was expected
	'abb,imx8mp-aristianetos3' is not one of ['polyhex,imx8mp-debix-model-a']
	'abb,imx8mp-aristianetos3' is not one of ['polyhex,imx8mp-debix-som-a-bmb-08']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mp-nonwifi-dahlia', 'toradex,verdin-imx8mp-nonwifi-dev', 'toradex,verdin-imx8mp-nonwifi-ivy', 'toradex,verdin-imx8mp-nonwifi-mallow', 'toradex,verdin-imx8mp-nonwifi-yavia']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mp-wifi-dahlia', 'toradex,verdin-imx8mp-wifi-dev', 'toradex,verdin-imx8mp-wifi-ivy', 'toradex,verdin-imx8mp-wifi-mallow', 'toradex,verdin-imx8mp-wifi-yavia']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mp-tqma8mpql-mba8mpxl', 'tq,imx8mp-tqma8mpql-mba8mp-ras314']
	'variscite,var-som-mx8mp-symphony' was expected
	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx8mq-nitrogen8m', 'boundary,imx8mq-nitrogen8m-som', 'einfochips,imx8mq-thor96', 'fsl,imx8mq-evk', 'google,imx8mq-phanbell', 'kontron,pitx-imx8m', 'purism,librem5-devkit', 'solidrun,hummingboard-pulse', 'technexion,pico-pi-imx8m']
	'mntre,reform2' was expected
	'abb,imx8mp-aristianetos3' is not one of ['purism,librem5r2', 'purism,librem5r3', 'purism,librem5r4']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mq-tqma8mq-mba8mx']
	'abb,imx8mp-aristianetos3' is not one of ['zii,imx8mq-ultra-rmb3', 'zii,imx8mq-ultra-zest']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8qm-mek', 'toradex,apalis-imx8', 'toradex,apalis-imx8-v1.1']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis-imx8-eval', 'toradex,apalis-imx8-eval-v1.2', 'toradex,apalis-imx8-ixora-v1.1']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis-imx8-v1.1-eval', 'toradex,apalis-imx8-v1.1-eval-v1.2', 'toradex,apalis-imx8-v1.1-ixora-v1.1', 'toradex,apalis-imx8-v1.1-ixora-v1.2']
	'abb,imx8mp-aristianetos3' is not one of ['einfochips,imx8qxp-ai_ml', 'fsl,imx8qxp-mek']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8dxl-evk']
	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx8x-aster', 'toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x-iris', 'toradex,colibri-imx8x-iris-v2']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8ulp-evk']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx93-9x9-qsb', 'fsl,imx93-11x11-evk', 'fsl,imx93-14x14-evk']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx95-19x19-evk']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imxrt1050-evk']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,imxrt1170-evk']
	'abb,imx8mp-aristianetos3' is not one of ['tq,imx93-tqma9352-mba93xxca', 'tq,imx93-tqma9352-mba93xxla']
	'phytec,imx93-phyboard-segin' was expected
	'variscite,var-som-mx93-symphony' was expected
	'kontron,imx93-bl-osm-s' was expected
	'abb,imx8mp-aristianetos3' is not one of ['fsl,vf500', 'fsl,vf510', 'fsl,vf600', 'fsl,vf610', 'fsl,vf610m4']
	'toradex,vf500-colibri_vf50-on-eval' was expected
	'abb,imx8mp-aristianetos3' is not one of ['fsl,vf610-twr', 'lwn,bk4', 'phytec,vf610-cosmic', 'toradex,vf610-colibri_vf61']
	'toradex,vf610-colibri_vf61-on-eval' was expected
	'abb,imx8mp-aristianetos3' is not one of ['zii,vf610cfu1', 'zii,vf610dev-c', 'zii,vf610dev-b', 'zii,vf610scu4-aib', 'zii,vf610dtu', 'zii,vf610spu3', 'zii,vf610spb4']
	'abb,imx8mp-aristianetos3' is not one of ['ebs-systart,oxalis', 'fsl,ls1012a-rdb', 'fsl,ls1012a-frdm', 'fsl,ls1012a-frwy', 'fsl,ls1012a-qds']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1021a-iot', 'fsl,ls1021a-moxa-uc-8410a', 'fsl,ls1021a-qds', 'fsl,ls1021a-tsn', 'fsl,ls1021a-twr']
	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1021a-tqmls1021a-mbls102xa']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1028a-qds', 'fsl,ls1028a-rdb']
	'kontron,kbox-a-230-ls' was expected
	'abb,imx8mp-aristianetos3' is not one of ['kontron,sl28-var1-ads2', 'kontron,sl28-var2-ads2', 'kontron,sl28-var3-ads2', 'kontron,sl28-var4-ads2']
	'abb,imx8mp-aristianetos3' is not one of ['kontron,sl28-var1', 'kontron,sl28-var2', 'kontron,sl28-var3', 'kontron,sl28-var4']
	'kontron,sl28' was expected
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1043a-rdb', 'fsl,ls1043a-qds']
	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1043a-tqmls1043a-mbls10xxa']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1046a-frwy', 'fsl,ls1046a-qds', 'fsl,ls1046a-rdb']
	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1046a-tqmls1046a-mbls10xxa']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1088a-qds', 'fsl,ls1088a-rdb']
	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1088a-tqmls1088a-mbls10xxa']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2080a-simu', 'fsl,ls2080a-qds', 'fsl,ls2080a-rdb']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2081a-rdb']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2088a-qds', 'fsl,ls2088a-rdb']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,lx2160a-bluebox3', 'fsl,lx2160a-bluebox3-rev-a', 'fsl,lx2160a-qds', 'fsl,lx2160a-rdb', 'fsl,lx2162a-qds']
	'abb,imx8mp-aristianetos3' is not one of ['solidrun,clearfog-cx', 'solidrun,honeycomb']
	'abb,imx8mp-aristianetos3' is not one of ['solidrun,lx2162a-clearfog']
	'abb,imx8mp-aristianetos3' is not one of ['tq,lx2160a-tqmlx2160a-mblx2160a']
	'abb,imx8mp-aristianetos3' is not one of ['nxp,s32g274a-evb', 'nxp,s32g274a-rdb2']
	'abb,imx8mp-aristianetos3' is not one of ['nxp,s32g399a-rdb3']
	'abb,imx8mp-aristianetos3' is not one of ['fsl,s32v234-evb']
	'abb,imx8mp-aristianetos3' is not one of ['traverse,ten64']
	'fsl,imx1' was expected
	'fsl,imx23' was expected
	'fsl,imx25' was expected
	'eukrea,cpuimx25' was expected
	'eukrea,mbimxsd25-baseboard' was expected
	'fsl,imx27' was expected
	'armadeus,imx27-apf27' was expected
	'eukrea,cpuimx27' was expected
	'phytec,imx27-pca100' was expected
	'phytec,imx27-pcm038' was expected
	'fsl,imx28' was expected
	'denx,m28' was expected
	'denx,m28evk' was expected
	'armadeus,imx28-apf28' was expected
	'crystalfontz,cfa10036' was expected
	'crystalfontz,cfa10037' was expected
	'i2se,duckbill-2' was expected
	'eukrea,mbmx28lc' was expected
	'fsl,imx31' was expected
	'fsl,imx35' was expected
	'eukrea,cpuimx35' was expected
	'fsl,imx50' was expected
	'fsl,imx51' was expected
	'armadeus,imx51-apf51' was expected
	'digi,connectcore-ccxmx51-som' was expected
	'eukrea,cpuimx51' was expected
	'fsl,imx53' was expected
	'denx,imx53-m53evk' was expected
	'tq,tqma53' was expected
	'fsl,imx6q' was expected
	'advantech,imx6q-ba16' was expected
	'armadeus,imx6q-apf6' was expected
	'compulab,cm-fx6' was expected
	'dfi,fs700e-m60' was expected
	'dh,imx6q-dhcom-som' was expected
	'gw,ventana' was expected
	'kontron,imx6q-samx6i' was expected
	'phytec,imx6q-pbac06' was expected
	'phytec,imx6q-pfla02' was expected
	'toradex,apalis_imx6q' was expected
	'variscite,var-som-imx6q' was expected
	'tq,mba6a' was expected
	'tq,mba6b' was expected
	'fsl,imx6qp' was expected
	'phytec,imx6qp-pbac06' was expected
	'fsl,imx6dl' was expected
	'armadeus,imx6dl-apf6' was expected
	'congatec,qmx6' was expected
	'dh,imx6dl-dhcom-som' was expected
	'kontron,imx6dl-samx6i' was expected
	'phytec,imx6dl-pbac06' was expected
	'phytec,imx6dl-pfla02' was expected
	'toradex,colibri_imx6dl' was expected
	'dh,imx6s-dhcom-som' was expected
	'fsl,imx6sl' was expected
	'fsl,imx6sll' was expected
	'fsl,imx6sx' was expected
	'fsl,imx6ul' was expected
	'armadeus,imx6ul-opos6ul' was expected
	'digi,ccimx6ulsom' was expected
	'grinn,imx6ul-litesom' was expected
	'phytec,imx6ul-pbacd10' was expected
	'kontron,sl-imx6ul' was expected
	'tq,imx6ul-tqma6ul1' was expected
	'tq,imx6ul-tqma6ul2' was expected
	'tq,imx6ul-tqma6ul2l' was expected
	'fsl,imx6ull' was expected
	'armadeus,imx6ull-opos6ul' was expected
	'dh,imx6ull-dhcom-som' was expected
	'dh,imx6ull-dhcor-som' was expected
	'phytec,imx6ull-pbacd10' was expected
	'phytec,imx6ull-phygate-tauri' was expected
	'toradex,colibri-imx6ull' was expected
	'toradex,colibri-imx6ull-emmc' was expected
	'toradex,colibri-imx6ull-wifi' was expected
	'kontron,sl-imx6ull' was expected
	'tq,imx6ull-tqma6ull2' was expected
	'tq,imx6ull-tqma6ull2l' was expected
	'seeed,imx6ull-seeed-npi' was expected
	'fsl,imx7s' was expected
	'toradex,colibri-imx7s' was expected
	'tq,imx7s-tqma7' was expected
	'fsl,imx7d' was expected
	'tq,imx7d-tqma7' was expected
	'compulab,cl-som-imx7' was expected
	'toradex,colibri-imx7d' was expected
	'toradex,colibri-imx7d-emmc' was expected
	'fsl,imx7ulp' was expected
	'fsl,imx8mm' was expected
	'compulab,imx8mm-ucm-som' was expected
	'ees,imx8mm-emtop-som' was expected
	'engicam,icore-mx8mm' was expected
	'kontron,imx8mm-sl' was expected
	'kontron,imx8mm-osm-s' was expected
	'toradex,verdin-imx8mm-nonwifi' was expected
	'toradex,verdin-imx8mm-wifi' was expected
	'phytec,imx8mm-phycore-som' was expected
	'variscite,var-som-mx8mm' was expected
	'tq,imx8mm-tqma8mqml' was expected
	'fsl,imx8mn' was expected
	'variscite,var-som-mx8mn' was expected
	'tq,imx8mn-tqma8mqnl' was expected
	'fsl,imx8mp' was expected
	'avnet,sm2s-imx8mp-14N0600E' was expected
	'dh,imx8mp-dhcom-som' was expected
	'engicam,icore-mx8mp' was expected
	'kontron,imx8mp-osm-s' was expected
	'kontron,imx8mp-smarc' was expected
	'phytec,imx8mp-phycore-som' was expected
	'polyhex,imx8mp-debix' was expected
	'polyhex,imx8mp-debix-som-a' was expected
	'toradex,verdin-imx8mp-nonwifi' was expected
	'toradex,verdin-imx8mp-wifi' was expected
	'tq,imx8mp-tqma8mpql' was expected
	'variscite,var-som-mx8mp' was expected
	'fsl,imx8mq' was expected
	'boundary,imx8mq-nitrogen8m-som' was expected
	'purism,librem5' was expected
	'tq,imx8mq-tqma8mq' was expected
	'zii,imx8mq-ultra' was expected
	'fsl,imx8qm' was expected
	'toradex,apalis-imx8' was expected
	'toradex,apalis-imx8-v1.1' was expected
	'fsl,imx8dxl' was expected
	'toradex,colibri-imx8x' was expected
	'fsl,imx8ulp' was expected
	'fsl,imx93' was expected
	'fsl,imx95' was expected
	'fsl,imxrt1050' was expected
	'fsl,imxrt1170' was expected
	'tq,imx93-tqma9352' was expected
	'phytec,imx93-phycore-som' was expected
	'variscite,var-som-mx93' was expected
	'kontron,imx93-osm-s' was expected
	'toradex,vf500-colibri_vf50' was expected
	'fsl,vf610' was expected
	'toradex,vf610-colibri_vf61' was expected
	'zii,vf610dev' was expected
	'fsl,ls1012a' was expected
	'fsl,ls1021a' was expected
	'tq,ls1021a-tqmls1021a' was expected
	'fsl,ls1028a' was expected
	'kontron,sl28-var4' was expected
	'abb,imx8mp-aristianetos3-som' is not one of ['kontron,sl28-var1', 'kontron,sl28-var2', 'kontron,sl28-var3', 'kontron,sl28-var4']
	'fsl,ls1043a' was expected
	'tq,ls1043a-tqmls1043a' was expected
	'fsl,ls1046a' was expected
	'tq,ls1046a-tqmls1046a' was expected
	'fsl,ls1088a' was expected
	'tq,ls1088a-tqmls1088a' was expected
	'fsl,ls2080a' was expected
	'fsl,ls2081a' was expected
	'fsl,ls2088a' was expected
	'fsl,lx2160a' was expected
	'solidrun,lx2160a-cex7' was expected
	'solidrun,lx2162a-som' was expected
	'tq,lx2160a-tqmlx2160a' was expected
	'nxp,s32g2' was expected
	'nxp,s32g3' was expected
	'fsl,s32v234' was expected
	'phytec,imx6qdl-pcm058' was expected
	'tq,imx6q-tqma6q-a' was expected
	'tq,imx6q-tqma6q-b' was expected
	'tq,imx6qp-tqma6qp-b' was expected
	'tq,imx6dl-tqma6dl-a' was expected
	'tq,imx6dl-tqma6dl-b' was expected
	'phytec,imx6ul-pcl063' was expected
	'phytec,imx6ull-pcl063' was expected
	'fsl,imx6ulz' was expected
	'toradex,verdin-imx8mm' was expected
	'avnet,sm2s-imx8mp' was expected
	'toradex,verdin-imx8mp' was expected
	'fsl,imx8mp' is not one of ['fsl,imx8qxp', 'fsl,imx8dx']
	'fsl,vf500' was expected
	from schema $id: http://devicetree.org/schemas/arm/fsl.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pinctrl@30330000: 'pinctrl-0' is a dependency of 'pinctrl-names'
	from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: clock-controller@30380000: 'init-on-array' does not match any of the regexes: 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/clock/imx8m-clock.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pwm@30670000: #pwm-cells: 3 was expected
	from schema $id: http://devicetree.org/schemas/pwm/imx-pwm.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: spi@30820000: Unevaluated properties are not allowed ('fsl,spi-num-chipselects' was unexpected)
	from schema $id: http://devicetree.org/schemas/spi/fsl-imx-cspi.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: spi@30830000: Unevaluated properties are not allowed ('fsl,spi-num-chipselects' was unexpected)
	from schema $id: http://devicetree.org/schemas/spi/fsl-imx-cspi.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO1: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO2: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO3: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO4: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO5: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK1: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK2: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK4: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK5: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK6: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: compatible: 'oneOf' conditional failed, one must be fixed:
	['st,st33ktpm2xi2c'] is too short
	'st,st33ktpm2xi2c' is not one of ['atmel,at97sc3204t', 'infineon,slb9635tt', 'infineon,slb9645tt', 'infineon,tpm_i2c_infineon', 'nuvoton,npct501', 'nuvoton,npct601', 'st,st33zp24-i2c', 'winbond,wpct301']
	from schema $id: http://devicetree.org/schemas/tpm/tcg,tpm-tis-i2c.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: Unevaluated properties are not allowed ('#gpio-cells', '#interrupt-cells', 'compatible', 'reset-gpio' were unexpected)
	from schema $id: http://devicetree.org/schemas/tpm/tcg,tpm-tis-i2c.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: 'gpio-controller' is a dependency of '#gpio-cells'
	from schema $id: http://devicetree.org/schemas/gpio/gpio.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext0@3f: $nodename:0: 'i2c6_gpioext0@3f' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext1@70: $nodename:0: 'i2c6_gpioext1@70' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext1@70: 'aristainetos3-tpm-grp' does not match any of the regexes: '-cfg$', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: tx-queues-config: 'snps,tx-sched-sp' does not match any of the regexes: '^queue[0-9]$', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/net/nxp,dwmac-imx.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts', 'mdio', 'phy-handle', 'phy-mode', 'reg', 'rx-queues-config', 'snps,force_thresh_dma_mode', 'snps,mtl-rx-config', 'snps,mtl-tx-config', 'tx-queues-config' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/nxp,dwmac-imx.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: tx-queues-config: 'snps,tx-sched-sp' does not match any of the regexes: '^queue[0-9]$', 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/net/snps,dwmac.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names: ['pcie', 'pcie_aux', 'pcie_phy', 'pcie_bus'] is too long
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clocks: [[2, 268], [2, 120], [2, 311], [2, 217]] is too long
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names:1: 'pcie_bus' was expected
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names:2: 'pcie_aux' was expected
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names: ['pcie', 'pcie_aux', 'pcie_phy', 'pcie_bus'] is too long
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: Unevaluated properties are not allowed ('ext_osc', 'fsl,max-link-speed', 'fsl,tx-deemph-gen1', 'power-domains', 'reserved-region' were unexpected)
	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: usb-phy@381f0040: Unevaluated properties are not allowed ('fsl,pcs-tx-deemph-3p5db', 'fsl,phy-comp-dis-tune', 'fsl,phy-pcs-tx-swing-full', 'fsl,phy-tx-preemp-amp-tune', 'fsl,phy-tx-vboost-level', 'fsl,phy-tx-vref-tune' were unexpected)
	from schema $id: http://devicetree.org/schemas/phy/fsl,imx8mq-usb-phy.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: usb-phy@382f0040: Unevaluated properties are not allowed ('fsl,phy-tx-preemp-amp-tune' was unexpected)
	from schema $id: http://devicetree.org/schemas/phy/fsl,imx8mq-usb-phy.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: $nodename:0: 'fixed-regulators' does not match '^([a-z][a-z0-9\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: #size-cells: 0 is not one of [1, 2]
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: regulator-enet: 'ranges' is a required property
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: 'ranges' is a required property
	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#






^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier
  2024-10-29 12:44 ` [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Rob Herring (Arm)
@ 2024-10-29 12:50   ` Heiko Schocher
  0 siblings, 0 replies; 12+ messages in thread
From: Heiko Schocher @ 2024-10-29 12:50 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Hugo Villeneuve, Joao Paulo Goncalves, Fabio Estevam,
	Krzysztof Kozlowski, Hiago De Franco, imx, Gregor Herburger,
	Shawn Guo, Peng Fan, devicetree, Conor Dooley, Mathieu Othacehe,
	linux-kernel, Sascha Hauer, Pengutronix Kernel Team,
	Alexander Stein, Tim Harvey, Michael Walle, Max Merchel,
	linux-arm-kernel

Hello Rob,

On 29.10.24 13:44, Rob Herring (Arm) wrote:
> 
> On Mon, 28 Oct 2024 09:23:30 +0100, Heiko Schocher wrote:
>> The board uses a ABB specific SoM from ADLink, based on NXP
>> i.MX8MP SoC. The SoM is used on 3 different carrier boards,
>> with small differences, which are all catched up in
>> devicetree overlays. The kernel image, the basic dtb
>> and all dtbos are collected in a fitimage. As bootloader
>> is used U-Boot which detects in his SPL stage the carrier
>> board by probing some i2c devices. When the correct
>> carrier is probed, the SPL applies all needed dtbos to
>> the dtb with which U-Boot gets loaded. Same principle
>> later before linux image boot, U-Boot applies the dtbos
>> needed for the carrier board before booting Linux.
>>
>> So intentionally, there are no full dtb images for all
>> carrier combinations, only dtbos which applied as needed
>> on boottime.
>>
>> Patches to U-Boot mailinglist not posted yet, as U-Boot
>> uses the same dtb as linux, so hope to get the linux
>> files first in, and use them when posting U-Boot board
>> support.
>>
>> series is based on:
>> 819837584309 - (tag: v6.12-rc5, origin/master, origin/HEAD) Linux 6.12-rc5
>>
>>
>>
>> Heiko Schocher (2):
>>    dt-bindings: arm: fsl: Add ABB SoM and carrier
>>    arm64: dts: imx8mp: add aristainetos3 board support
>>
>>   .../devicetree/bindings/arm/fsl.yaml          |    2 +
>>   arch/arm64/boot/dts/freescale/Makefile        |    5 +
>>   .../imx8mp-aristainetos3-adpismarc.dtsi       |   64 +
>>   .../imx8mp-aristainetos3-adpismarc.dtso       |   14 +
>>   .../imx8mp-aristainetos3-helios-lvds.dtsi     |   89 ++
>>   .../imx8mp-aristainetos3-helios-lvds.dtso     |   13 +
>>   .../imx8mp-aristainetos3-helios.dtsi          |  103 ++
>>   .../imx8mp-aristainetos3-helios.dtso          |   13 +
>>   .../imx8mp-aristainetos3-proton2s.dtsi        |  176 +++
>>   .../imx8mp-aristainetos3-proton2s.dtso        |   13 +
>>   .../imx8mp-aristainetos3a-som-v1.dts          |   18 +
>>   .../imx8mp-aristainetos3a-som-v1.dtsi         | 1210 +++++++++++++++++
>>   12 files changed, 1720 insertions(+)
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-adpismarc.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios-lvds.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-helios.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtsi
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3-proton2s.dtso
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dts
>>   create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi
>>
>> --
>> 2.20.1
>>
>>
>>
> 
> 
> My bot found new DTB warnings on the .dts files added or changed in this
> series.
> 
> Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
> are fixed by another series. Ultimately, it is up to the platform
> maintainer whether these warnings are acceptable or not. No need to reply
> unless the platform maintainer has comments.

I already work on this problems, as Krzysztof already commented
this issue, I am sorry!

bye,
Heiko
> 
> If you already ran DT checks and didn't see these error(s), then
> make sure dt-schema is up to date:
> 
>    pip3 install dtschema --upgrade
> 
> 
> New warnings running 'make CHECK_DTBS=y freescale/imx8mp-aristainetos3a-som-v1.dtb' for 20241028082332.21672-1-hs@denx.de:
> 
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtsi:527.14-540.4: Warning (interrupt_provider): /soc@0/bus@30800000/i2c@30ae0000/tpm@2e: '#interrupt-cells' found, but node is not an interrupt provider
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: Warning (interrupt_map): Failed prerequisite 'interrupt_provider'
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
> 	['abb,imx8mp-aristianetos3', 'abb,imx8mp-aristianetos3-som', 'fsl,imx8mp'] is too long
> 	['abb,imx8mp-aristianetos3', 'abb,imx8mp-aristianetos3-som', 'fsl,imx8mp'] is too short
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
> 		'abb,imx8mp-aristianetos3' is not one of ['tq,imx8dxp-tqma8xdp-mba8xx']
> 		'abb,imx8mp-aristianetos3' is not one of ['tq,imx8qxp-tqma8xqp-mba8xx']
> 		'tq,imx8dxp-tqma8xdp' was expected
> 		'tq,imx8qxp-tqma8xqp' was expected
> 		'fsl,imx8dxp' was expected
> 		'fsl,imx8qxp' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx1-apf9328', 'fsl,imx1ads']
> 	'abb,imx8mp-aristianetos3' is not one of ['creative,x-fi3', 'fsl,imx23-evk', 'fsl,stmp378x-devb', 'olimex,imx23-olinuxino', 'sandisk,sansa_fuze_plus']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx25-pdk', 'karo,imx25-tx25']
> 	'abb,imx8mp-aristianetos3' is not one of ['eukrea,mbimxsd25-baseboard']
> 	'abb,imx8mp-aristianetos3' is not one of ['eukrea,mbimxsd25-baseboard-cmo-qvga', 'eukrea,mbimxsd25-baseboard-dvi-svga', 'eukrea,mbimxsd25-baseboard-dvi-vga']
> 	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx27-apf27', 'fsl,imx27-pdk']
> 	'armadeus,imx27-apf27dev' was expected
> 	'eukrea,mbimxsd27-baseboard' was expected
> 	'phytec,imx27-pca100-rdk' was expected
> 	'phytec,imx27-pcm970' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx28-apf28', 'bluegiga,apx4devkit', 'crystalfontz,cfa10036', 'eukrea,mbmx28lc', 'fsl,imx28-evk', 'i2se,duckbill', 'i2se,duckbill-2', 'karo,tx28', 'lwn,imx28-xea', 'msr,m28cu3', 'schulercontrol,imx28-sps1', 'technologic,imx28-ts4600']
> 	'aries,m28' was expected
> 	'aries,m28evk' was expected
> 	'armadeus,imx28-apf28dev' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['crystalfontz,cfa10037', 'crystalfontz,cfa10049', 'crystalfontz,cfa10057', 'crystalfontz,cfa10058']
> 	'abb,imx8mp-aristianetos3' is not one of ['crystalfontz,cfa10055', 'crystalfontz,cfa10056']
> 	'abb,imx8mp-aristianetos3' is not one of ['i2se,duckbill-2-485', 'i2se,duckbill-2-enocean', 'i2se,duckbill-2-spi']
> 	'eukrea,mbmx283lc' was expected
> 	'eukrea,mbmx287lc' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['buglabs,imx31-bug', 'logicpd,imx31-lite']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx35-pdk']
> 	'eukrea,mbimxsd35-baseboard' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx50-evk', 'kobo,aura']
> 	'abb,imx8mp-aristianetos3' is not one of ['armadeus,imx51-apf51', 'fsl,imx51-babbage', 'technologic,imx51-ts4800', 'zii,imx51-scu3-esb', 'zii,imx51-scu2-mezz', 'zii,imx51-rdu1']
> 	'armadeus,imx51-apf51dev' was expected
> 	'digi,connectcore-ccxmx51-jsk' was expected
> 	'eukrea,mbimxsd51' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['bhf,cx9020', 'fsl,imx53-ard', 'fsl,imx53-evk', 'fsl,imx53-qsb', 'fsl,imx53-qsrb', 'fsl,imx53-smd', 'ge,imx53-cpuvo', 'inversepath,imx53-usbarmory', 'karo,tx53', 'kiebackpeter,imx53-ddc', 'kiebackpeter,imx53-hsc', 'menlo,m53menlo', 'starterkit,sk-imx53', 'voipac,imx53-dmm-668']
> 	'aries,imx53-m53evk' was expected
> 	'tq,mba53' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['auvidea,h100', 'bosch,imx6q-acc', 'boundary,imx6q-nitrogen6_max', 'boundary,imx6q-nitrogen6_som2', 'boundary,imx6q-nitrogen6x', 'compulab,cm-fx6', 'dmo,imx6q-edmqmx6', 'ds,imx6q-sbc', 'embest,imx6q-marsboard', 'emtrion,emcon-mx6', 'emtrion,emcon-mx6-avari', 'engicam,imx6-icore', 'engicam,imx6-icore-rqs', 'fsl,imx6q-arm2', 'fsl,imx6q-sabreauto', 'fsl,imx6q-sabrelite', 'fsl,imx6q-sabresd', 'karo,imx6q-tx6q', 'kiebackpeter,imx6q-tpc', 'kontron,imx6q-samx6i', 'kosagi,imx6q-novena', 'kvg,vicut1q', 'logicpd,imx6q-logicpd', 'lwn,display5', 'lwn,mccmon6', 'nutsboard,imx6q-pistachio', 'microsys,sbc6x', 'poslab,imx6q-savageboard', 'prt,prti6q', 'prt,prtwd2', 'rex,imx6q-rex-pro', 'skov,imx6q-skov-revc-lt2', 'skov,imx6q-skov-revc-lt6', 'skov,imx6q-skov-reve-mi1010ait-1cp1', 'solidrun,cubox-i/q', 'solidrun,hummingboard/q', 'solidrun,hummingboard2/q', 'solidrun,solidsense/q', 'tbs,imx6q-tbs2910', 'technexion,imx6q-pico-dwarf', 'technexion,imx6q-pico-hobb
>   it', 'technexion,imx6q-pico-nymph', 'technexion,imx6q-pico-pi', 'technologic,imx6q-ts4900', 'technologic,imx6q-ts7970', 'toradex,apalis_imx6q', 'udoo,imx6q-udoo', 'uniwest,imx6q-evi', 'variscite,dt6customboard', 'wand,imx6q-wandboard', 'ysoft,imx6q-yapp4-crux', 'ysoft,imx6q-yapp4-pegasus', 'zealz,imx6q-gk802', 'zii,imx6q-zii-rdu2']
> 	'abb,imx8mp-aristianetos3' is not one of ['advantech,imx6q-dms-ba16', 'ge,imx6q-b450v3', 'ge,imx6q-b650v3', 'ge,imx6q-b850v3']
> 	'armadeus,imx6q-apf6dev' was expected
> 	'compulab,utilite-pro' was expected
> 	'dfi,fs700-m60-6qd' was expected
> 	'dh,imx6q-dhcom-pdk2' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['gw,imx6q-gw51xx', 'gw,imx6q-gw52xx', 'gw,imx6q-gw53xx', 'gw,imx6q-gw5400-a', 'gw,imx6q-gw54xx', 'gw,imx6q-gw551x', 'gw,imx6q-gw552x', 'gw,imx6q-gw553x', 'gw,imx6q-gw560x', 'gw,imx6q-gw5903', 'gw,imx6q-gw5904', 'gw,imx6q-gw5907', 'gw,imx6q-gw5910', 'gw,imx6q-gw5912', 'gw,imx6q-gw5913']
> 	'kontron,imx6q-samx6i-ads2' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6q-pbac06-emmc', 'phytec,imx6q-pbac06-nand']
> 	'abb,imx8mp-aristianetos3' is not one of ['comvetia,imx6q-lxr', 'phytec,imx6q-pbab01']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis_imx6q-ixora', 'toradex,apalis_imx6q-ixora-v1.1', 'toradex,apalis_imx6q-ixora-v1.2', 'toradex,apalis_imx6q-eval', 'toradex,apalis_imx6q-eval-v1.2']
> 	'variscite,mx6customboard' was expected
> 	'tq,imx6q-mba6x-a' was expected
> 	'tq,imx6q-mba6x-b' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx6qp-nitrogen6_max', 'boundary,imx6qp-nitrogen6_som2', 'fsl,imx6qp-sabreauto', 'fsl,imx6qp-sabresd', 'karo,imx6qp-tx6qp', 'kvg,vicutp', 'prt,prtwd3', 'wand,imx6qp-wandboard', 'ysoft,imx6qp-yapp4-crux-plus', 'ysoft,imx6qp-yapp4-pegasus-plus', 'zii,imx6qp-zii-rdu2']
> 	'phytec,imx6qp-pbac06-nand' was expected
> 	'tq,imx6qp-mba6x-b' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['abb,aristainetos-imx6dl-4', 'abb,aristainetos-imx6dl-7', 'abb,aristainetos2-imx6dl-4', 'abb,aristainetos2-imx6dl-7', 'alt,alti6p', 'boundary,imx6dl-nit6xlite', 'boundary,imx6dl-nitrogen6x', 'bticino,imx6dl-mamoj', 'eckelmann,imx6dl-ci4x10', 'emtrion,emcon-mx6', 'emtrion,emcon-mx6-avari', 'engicam,imx6-icore', 'engicam,imx6-icore-rqs', 'fsl,imx6dl-sabreauto', 'fsl,imx6dl-sabrelite', 'fsl,imx6dl-sabresd', 'karo,imx6dl-tx6dl', 'kontron,imx6dl-samx6i', 'kvg,victgo', 'kvg,vicut1', 'ply,plybas', 'ply,plym2m', 'poslab,imx6dl-savageboard', 'prt,prtmvt', 'prt,prtrvt', 'prt,prtvt7', 'rex,imx6dl-rex-basic', 'riot,imx6s-riotboard', 'sielaff,imx6dl-board', 'skov,imx6dl-skov-revc-lt2', 'skov,imx6dl-skov-revc-lt6', 'solidrun,cubox-i/dl', 'solidrun,hummingboard/dl', 'solidrun,hummingboard2/dl', 'solidrun,solidsense/dl', 'technexion,imx6dl-pico-dwarf', 'technexion,imx6dl-pico-hobbit', 'technexion,imx6dl-pico-nymph', 'technexion,imx6dl-pico-pi', 'technologic
>   ,imx6dl-ts4900', 'technologic,imx6dl-ts7970', 'toradex,colibri_imx6dl', 'udoo,imx6dl-udoo', 'vdl,lanmcu', 'wand,imx6dl-wandboard', 'ysoft,imx6dl-yapp4-draco', 'ysoft,imx6dl-yapp4-hydra', 'ysoft,imx6dl-yapp4-lynx', 'ysoft,imx6dl-yapp4-orion', 'ysoft,imx6dl-yapp4-phoenix', 'ysoft,imx6dl-yapp4-ursa']
> 	'armadeus,imx6dl-apf6dev' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['ge,imx6dl-b105v2', 'ge,imx6dl-b105pv2', 'ge,imx6dl-b125v2', 'ge,imx6dl-b125pv2', 'ge,imx6dl-b155v2']
> 	'dfi,fs700-m60-6dl' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['dh,imx6dl-dhcom-pdk2', 'dh,imx6dl-dhcom-picoitx']
> 	'abb,imx8mp-aristianetos3' is not one of ['gw,imx6dl-gw51xx', 'gw,imx6dl-gw52xx', 'gw,imx6dl-gw53xx', 'gw,imx6dl-gw54xx', 'gw,imx6dl-gw551x', 'gw,imx6dl-gw552x', 'gw,imx6dl-gw553x', 'gw,imx6dl-gw560x', 'gw,imx6dl-gw5903', 'gw,imx6dl-gw5904', 'gw,imx6dl-gw5907', 'gw,imx6dl-gw5910', 'gw,imx6dl-gw5912', 'gw,imx6dl-gw5913']
> 	'kontron,imx6dl-samx6i-ads2' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6dl-pbac06-emmc', 'phytec,imx6dl-pbac06-nand']
> 	'phytec,imx6dl-pbab01' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri_imx6dl-aster', 'toradex,colibri_imx6dl-eval-v3', 'toradex,colibri_imx6dl-iris', 'toradex,colibri_imx6dl-iris-v2']
> 	'dh,imx6s-dhcom-drc02' was expected
> 	'tq,imx6dl-mba6x-a' was expected
> 	'tq,imx6dl-mba6x-b' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6sl-evk', 'kobo,aura2', 'kobo,tolino-shine2hd', 'kobo,tolino-shine3', 'kobo,tolino-vision', 'kobo,tolino-vision5', 'revotics,imx6sl-warp']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6sll-evk', 'kobo,clarahd', 'kobo,librah2o']
> 	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx6sx-nitrogen6sx', 'fsl,imx6sx-sabreauto', 'fsl,imx6sx-sdb', 'fsl,imx6sx-sdb-reva', 'samtec,imx6sx-vining-2000', 'udoo,neobasic', 'udoo,neoextended', 'udoo,neofull']
> 	'abb,imx8mp-aristianetos3' is not one of ['engicam,imx6ul-geam', 'engicam,imx6ul-isiot', 'fsl,imx6ul-14x14-evk', 'karo,imx6ul-tx6ul', 'kontron,sl-imx6ul', 'prt,prti6g', 'technexion,imx6ul-pico-dwarf', 'technexion,imx6ul-pico-hobbit', 'technexion,imx6ul-pico-pi']
> 	'armadeus,imx6ul-opos6uldev' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['digi,ccimx6ulsbcexpress', 'digi,ccimx6ulsbcpro']
> 	'grinn,imx6ul-liteboard' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ul-pbacd10-emmc', 'phytec,imx6ul-pbacd10-nand']
> 	'kontron,bl-imx6ul' was expected
> 	'kontron,bl-imx6ul-43' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul1-mba6ulx']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul2-mba6ulx']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ul-tqma6ul2l-mba6ulx', 'tq,imx6ul-tqma6ul2l-mba6ulxl']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx6ull-14x14-evk', 'joz,jozacp', 'kontron,sl-imx6ull', 'myir,imx6ull-mys-6ulx-eval', 'toradex,colibri-imx6ull', 'toradex,colibri-imx6ull-emmc', 'toradex,colibri-imx6ull-wifi', 'uni-t,uti260b']
> 	'armadeus,imx6ull-opos6uldev' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['chargebyte,imx6ull-tarragon-master', 'chargebyte,imx6ull-tarragon-micro', 'chargebyte,imx6ull-tarragon-slave', 'chargebyte,imx6ull-tarragon-slavext']
> 	'abb,imx8mp-aristianetos3' is not one of ['dh,imx6ull-dhcom-drc02', 'dh,imx6ull-dhcom-pdk2', 'dh,imx6ull-dhcom-picoitx']
> 	'marantec,imx6ull-dhcor-maveo-box' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ull-pbacd10-emmc', 'phytec,imx6ull-pbacd10-nand']
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx6ull-phygate-tauri-emmc', 'phytec,imx6ull-phygate-tauri-nand']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-aster', 'toradex,colibri-imx6ull-eval', 'toradex,colibri-imx6ull-iris', 'toradex,colibri-imx6ull-iris-v2']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-emmc-aster', 'toradex,colibri-imx6ull-emmc-eval', 'toradex,colibri-imx6ull-emmc-iris', 'toradex,colibri-imx6ull-emmc-iris-v2']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx6ull-wifi-eval', 'toradex,colibri-imx6ull-wifi-aster', 'toradex,colibri-imx6ull-wifi-iris', 'toradex,colibri-imx6ull-wifi-iris-v2']
> 	'kontron,bl-imx6ull' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ull-tqma6ull2-mba6ulx']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx6ull-tqma6ull2l-mba6ulx', 'tq,imx6ull-tqma6ull2l-mba6ulxl']
> 	'abb,imx8mp-aristianetos3' is not one of ['seeed,imx6ull-seeed-npi-emmc', 'seeed,imx6ull-seeed-npi-nand']
> 	'abb,imx8mp-aristianetos3' is not one of ['bsh,imx6ulz-bsh-smm-m2', 'fsl,imx6ulz-14x14-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['element14,imx7s-warp', 'toradex,colibri-imx7s']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7s-aster', 'toradex,colibri-imx7s-eval-v3', 'toradex,colibri-imx7s-iris', 'toradex,colibri-imx7s-iris-v2']
> 	'tq,imx7s-mba7' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx7d-nitrogen7', 'compulab,cl-som-imx7', 'fsl,imx7d-sdb', 'fsl,imx7d-sdb-reva', 'kam,imx7d-flex-concentrator', 'kam,imx7d-flex-concentrator-mfg', 'novtech,imx7d-meerkat96', 'remarkable,imx7d-remarkable2', 'storopack,imx7d-smegw01', 'technexion,imx7d-pico-dwarf', 'technexion,imx7d-pico-hobbit', 'technexion,imx7d-pico-nymph', 'technexion,imx7d-pico-pi', 'toradex,colibri-imx7d', 'toradex,colibri-imx7d-emmc', 'zii,imx7d-rmu2', 'zii,imx7d-rpu2']
> 	'tq,imx7d-mba7' was expected
> 	'compulab,sbc-imx7' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7d-aster', 'toradex,colibri-imx7d-eval-v3', 'toradex,colibri-imx7d-iris', 'toradex,colibri-imx7d-iris-v2']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx7d-emmc-aster', 'toradex,colibri-imx7d-emmc-eval-v3', 'toradex,colibri-imx7d-emmc-iris', 'toradex,colibri-imx7d-emmc-iris-v2']
> 	'abb,imx8mp-aristianetos3' is not one of ['ea,imx7ulp-com', 'fsl,imx7ulp-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['beacon,imx8mm-beacon-kit', 'boundary,imx8mm-nitrogen8mm', 'dmo,imx8mm-data-modul-edm-sbc', 'emtrion,emcon-mx8mm-avari', 'fsl,imx8mm-ddr4-evk', 'fsl,imx8mm-evk', 'fsl,imx8mm-evkb', 'gateworks,imx8mm-gw75xx-0x', 'gateworks,imx8mm-gw7904', 'gw,imx8mm-gw71xx-0x', 'gw,imx8mm-gw72xx-0x', 'gw,imx8mm-gw73xx-0x', 'gw,imx8mm-gw7901', 'gw,imx8mm-gw7902', 'gw,imx8mm-gw7903', 'innocomm,wb15-evk', 'kontron,imx8mm-sl', 'kontron,imx8mm-osm-s', 'toradex,verdin-imx8mm', 'toradex,verdin-imx8mm-nonwifi', 'toradex,verdin-imx8mm-wifi', 'prt,prt8mm']
> 	'abb,imx8mp-aristianetos3' is not one of ['compulab,imx8mm-iot-gateway']
> 	'ees,imx8mm-emtop-baseboard' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['engicam,icore-mx8mm-ctouch2', 'engicam,icore-mx8mm-edimm2.2']
> 	'kontron,imx8mm-bl' was expected
> 	'kontron,imx8mm-bl-osm-s' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['menlo,mx8menlo', 'toradex,verdin-imx8mm-nonwifi-dahlia', 'toradex,verdin-imx8mm-nonwifi-dev', 'toradex,verdin-imx8mm-nonwifi-ivy', 'toradex,verdin-imx8mm-nonwifi-mallow', 'toradex,verdin-imx8mm-nonwifi-yavia']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mm-wifi-dahlia', 'toradex,verdin-imx8mm-wifi-dev', 'toradex,verdin-imx8mm-wifi-ivy', 'toradex,verdin-imx8mm-wifi-mallow', 'toradex,verdin-imx8mm-wifi-yavia']
> 	'abb,imx8mp-aristianetos3' is not one of ['phytec,imx8mm-phyboard-polis-rdk', 'phytec,imx8mm-phygate-tauri-l']
> 	'variscite,var-som-mx8mm-symphony' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['cloos,imx8mm-phg', 'tq,imx8mm-tqma8mqml-mba8mx']
> 	'abb,imx8mp-aristianetos3' is not one of ['beacon,imx8mn-beacon-kit', 'bsh,imx8mn-bsh-smm-s2', 'bsh,imx8mn-bsh-smm-s2pro', 'fsl,imx8mn-ddr3l-evk', 'fsl,imx8mn-ddr4-evk', 'fsl,imx8mn-evk', 'gw,imx8mn-gw7902']
> 	'abb,imx8mp-aristianetos3' is not one of ['dimonoff,gateway-evk', 'rve,gateway', 'variscite,var-som-mx8mn-symphony']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mn-tqma8mqnl-mba8mx']
> 	'avnet,sm2s-imx8mp-14N0600E-ep1' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['dh,imx8mp-dhcom-drc02', 'dh,imx8mp-dhcom-pdk2', 'dh,imx8mp-dhcom-pdk3', 'dh,imx8mp-dhcom-picoitx']
> 	'abb,imx8mp-aristianetos3' is not one of ['engicam,icore-mx8mp-edimm2.2']
> 	'kontron,imx8mp-bl-osm-s' was expected
> 	'kontron,imx8mp-smarc-eval-carrier' was expected
> 	'phytec,imx8mp-phyboard-pollux-rdk' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['polyhex,imx8mp-debix-model-a']
> 	'abb,imx8mp-aristianetos3' is not one of ['polyhex,imx8mp-debix-som-a-bmb-08']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mp-nonwifi-dahlia', 'toradex,verdin-imx8mp-nonwifi-dev', 'toradex,verdin-imx8mp-nonwifi-ivy', 'toradex,verdin-imx8mp-nonwifi-mallow', 'toradex,verdin-imx8mp-nonwifi-yavia']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,verdin-imx8mp-wifi-dahlia', 'toradex,verdin-imx8mp-wifi-dev', 'toradex,verdin-imx8mp-wifi-ivy', 'toradex,verdin-imx8mp-wifi-mallow', 'toradex,verdin-imx8mp-wifi-yavia']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mp-tqma8mpql-mba8mpxl', 'tq,imx8mp-tqma8mpql-mba8mp-ras314']
> 	'variscite,var-som-mx8mp-symphony' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['boundary,imx8mq-nitrogen8m', 'boundary,imx8mq-nitrogen8m-som', 'einfochips,imx8mq-thor96', 'fsl,imx8mq-evk', 'google,imx8mq-phanbell', 'kontron,pitx-imx8m', 'purism,librem5-devkit', 'solidrun,hummingboard-pulse', 'technexion,pico-pi-imx8m']
> 	'mntre,reform2' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['purism,librem5r2', 'purism,librem5r3', 'purism,librem5r4']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx8mq-tqma8mq-mba8mx']
> 	'abb,imx8mp-aristianetos3' is not one of ['zii,imx8mq-ultra-rmb3', 'zii,imx8mq-ultra-zest']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8qm-mek', 'toradex,apalis-imx8', 'toradex,apalis-imx8-v1.1']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis-imx8-eval', 'toradex,apalis-imx8-eval-v1.2', 'toradex,apalis-imx8-ixora-v1.1']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,apalis-imx8-v1.1-eval', 'toradex,apalis-imx8-v1.1-eval-v1.2', 'toradex,apalis-imx8-v1.1-ixora-v1.1', 'toradex,apalis-imx8-v1.1-ixora-v1.2']
> 	'abb,imx8mp-aristianetos3' is not one of ['einfochips,imx8qxp-ai_ml', 'fsl,imx8qxp-mek']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8dxl-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['toradex,colibri-imx8x-aster', 'toradex,colibri-imx8x-eval-v3', 'toradex,colibri-imx8x-iris', 'toradex,colibri-imx8x-iris-v2']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx8ulp-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx93-9x9-qsb', 'fsl,imx93-11x11-evk', 'fsl,imx93-14x14-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imx95-19x19-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imxrt1050-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,imxrt1170-evk']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,imx93-tqma9352-mba93xxca', 'tq,imx93-tqma9352-mba93xxla']
> 	'phytec,imx93-phyboard-segin' was expected
> 	'variscite,var-som-mx93-symphony' was expected
> 	'kontron,imx93-bl-osm-s' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,vf500', 'fsl,vf510', 'fsl,vf600', 'fsl,vf610', 'fsl,vf610m4']
> 	'toradex,vf500-colibri_vf50-on-eval' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,vf610-twr', 'lwn,bk4', 'phytec,vf610-cosmic', 'toradex,vf610-colibri_vf61']
> 	'toradex,vf610-colibri_vf61-on-eval' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['zii,vf610cfu1', 'zii,vf610dev-c', 'zii,vf610dev-b', 'zii,vf610scu4-aib', 'zii,vf610dtu', 'zii,vf610spu3', 'zii,vf610spb4']
> 	'abb,imx8mp-aristianetos3' is not one of ['ebs-systart,oxalis', 'fsl,ls1012a-rdb', 'fsl,ls1012a-frdm', 'fsl,ls1012a-frwy', 'fsl,ls1012a-qds']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1021a-iot', 'fsl,ls1021a-moxa-uc-8410a', 'fsl,ls1021a-qds', 'fsl,ls1021a-tsn', 'fsl,ls1021a-twr']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1021a-tqmls1021a-mbls102xa']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1028a-qds', 'fsl,ls1028a-rdb']
> 	'kontron,kbox-a-230-ls' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['kontron,sl28-var1-ads2', 'kontron,sl28-var2-ads2', 'kontron,sl28-var3-ads2', 'kontron,sl28-var4-ads2']
> 	'abb,imx8mp-aristianetos3' is not one of ['kontron,sl28-var1', 'kontron,sl28-var2', 'kontron,sl28-var3', 'kontron,sl28-var4']
> 	'kontron,sl28' was expected
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1043a-rdb', 'fsl,ls1043a-qds']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1043a-tqmls1043a-mbls10xxa']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1046a-frwy', 'fsl,ls1046a-qds', 'fsl,ls1046a-rdb']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1046a-tqmls1046a-mbls10xxa']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls1088a-qds', 'fsl,ls1088a-rdb']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,ls1088a-tqmls1088a-mbls10xxa']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2080a-simu', 'fsl,ls2080a-qds', 'fsl,ls2080a-rdb']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2081a-rdb']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,ls2088a-qds', 'fsl,ls2088a-rdb']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,lx2160a-bluebox3', 'fsl,lx2160a-bluebox3-rev-a', 'fsl,lx2160a-qds', 'fsl,lx2160a-rdb', 'fsl,lx2162a-qds']
> 	'abb,imx8mp-aristianetos3' is not one of ['solidrun,clearfog-cx', 'solidrun,honeycomb']
> 	'abb,imx8mp-aristianetos3' is not one of ['solidrun,lx2162a-clearfog']
> 	'abb,imx8mp-aristianetos3' is not one of ['tq,lx2160a-tqmlx2160a-mblx2160a']
> 	'abb,imx8mp-aristianetos3' is not one of ['nxp,s32g274a-evb', 'nxp,s32g274a-rdb2']
> 	'abb,imx8mp-aristianetos3' is not one of ['nxp,s32g399a-rdb3']
> 	'abb,imx8mp-aristianetos3' is not one of ['fsl,s32v234-evb']
> 	'abb,imx8mp-aristianetos3' is not one of ['traverse,ten64']
> 	'fsl,imx1' was expected
> 	'fsl,imx23' was expected
> 	'fsl,imx25' was expected
> 	'eukrea,cpuimx25' was expected
> 	'eukrea,mbimxsd25-baseboard' was expected
> 	'fsl,imx27' was expected
> 	'armadeus,imx27-apf27' was expected
> 	'eukrea,cpuimx27' was expected
> 	'phytec,imx27-pca100' was expected
> 	'phytec,imx27-pcm038' was expected
> 	'fsl,imx28' was expected
> 	'denx,m28' was expected
> 	'denx,m28evk' was expected
> 	'armadeus,imx28-apf28' was expected
> 	'crystalfontz,cfa10036' was expected
> 	'crystalfontz,cfa10037' was expected
> 	'i2se,duckbill-2' was expected
> 	'eukrea,mbmx28lc' was expected
> 	'fsl,imx31' was expected
> 	'fsl,imx35' was expected
> 	'eukrea,cpuimx35' was expected
> 	'fsl,imx50' was expected
> 	'fsl,imx51' was expected
> 	'armadeus,imx51-apf51' was expected
> 	'digi,connectcore-ccxmx51-som' was expected
> 	'eukrea,cpuimx51' was expected
> 	'fsl,imx53' was expected
> 	'denx,imx53-m53evk' was expected
> 	'tq,tqma53' was expected
> 	'fsl,imx6q' was expected
> 	'advantech,imx6q-ba16' was expected
> 	'armadeus,imx6q-apf6' was expected
> 	'compulab,cm-fx6' was expected
> 	'dfi,fs700e-m60' was expected
> 	'dh,imx6q-dhcom-som' was expected
> 	'gw,ventana' was expected
> 	'kontron,imx6q-samx6i' was expected
> 	'phytec,imx6q-pbac06' was expected
> 	'phytec,imx6q-pfla02' was expected
> 	'toradex,apalis_imx6q' was expected
> 	'variscite,var-som-imx6q' was expected
> 	'tq,mba6a' was expected
> 	'tq,mba6b' was expected
> 	'fsl,imx6qp' was expected
> 	'phytec,imx6qp-pbac06' was expected
> 	'fsl,imx6dl' was expected
> 	'armadeus,imx6dl-apf6' was expected
> 	'congatec,qmx6' was expected
> 	'dh,imx6dl-dhcom-som' was expected
> 	'kontron,imx6dl-samx6i' was expected
> 	'phytec,imx6dl-pbac06' was expected
> 	'phytec,imx6dl-pfla02' was expected
> 	'toradex,colibri_imx6dl' was expected
> 	'dh,imx6s-dhcom-som' was expected
> 	'fsl,imx6sl' was expected
> 	'fsl,imx6sll' was expected
> 	'fsl,imx6sx' was expected
> 	'fsl,imx6ul' was expected
> 	'armadeus,imx6ul-opos6ul' was expected
> 	'digi,ccimx6ulsom' was expected
> 	'grinn,imx6ul-litesom' was expected
> 	'phytec,imx6ul-pbacd10' was expected
> 	'kontron,sl-imx6ul' was expected
> 	'tq,imx6ul-tqma6ul1' was expected
> 	'tq,imx6ul-tqma6ul2' was expected
> 	'tq,imx6ul-tqma6ul2l' was expected
> 	'fsl,imx6ull' was expected
> 	'armadeus,imx6ull-opos6ul' was expected
> 	'dh,imx6ull-dhcom-som' was expected
> 	'dh,imx6ull-dhcor-som' was expected
> 	'phytec,imx6ull-pbacd10' was expected
> 	'phytec,imx6ull-phygate-tauri' was expected
> 	'toradex,colibri-imx6ull' was expected
> 	'toradex,colibri-imx6ull-emmc' was expected
> 	'toradex,colibri-imx6ull-wifi' was expected
> 	'kontron,sl-imx6ull' was expected
> 	'tq,imx6ull-tqma6ull2' was expected
> 	'tq,imx6ull-tqma6ull2l' was expected
> 	'seeed,imx6ull-seeed-npi' was expected
> 	'fsl,imx7s' was expected
> 	'toradex,colibri-imx7s' was expected
> 	'tq,imx7s-tqma7' was expected
> 	'fsl,imx7d' was expected
> 	'tq,imx7d-tqma7' was expected
> 	'compulab,cl-som-imx7' was expected
> 	'toradex,colibri-imx7d' was expected
> 	'toradex,colibri-imx7d-emmc' was expected
> 	'fsl,imx7ulp' was expected
> 	'fsl,imx8mm' was expected
> 	'compulab,imx8mm-ucm-som' was expected
> 	'ees,imx8mm-emtop-som' was expected
> 	'engicam,icore-mx8mm' was expected
> 	'kontron,imx8mm-sl' was expected
> 	'kontron,imx8mm-osm-s' was expected
> 	'toradex,verdin-imx8mm-nonwifi' was expected
> 	'toradex,verdin-imx8mm-wifi' was expected
> 	'phytec,imx8mm-phycore-som' was expected
> 	'variscite,var-som-mx8mm' was expected
> 	'tq,imx8mm-tqma8mqml' was expected
> 	'fsl,imx8mn' was expected
> 	'variscite,var-som-mx8mn' was expected
> 	'tq,imx8mn-tqma8mqnl' was expected
> 	'fsl,imx8mp' was expected
> 	'avnet,sm2s-imx8mp-14N0600E' was expected
> 	'dh,imx8mp-dhcom-som' was expected
> 	'engicam,icore-mx8mp' was expected
> 	'kontron,imx8mp-osm-s' was expected
> 	'kontron,imx8mp-smarc' was expected
> 	'phytec,imx8mp-phycore-som' was expected
> 	'polyhex,imx8mp-debix' was expected
> 	'polyhex,imx8mp-debix-som-a' was expected
> 	'toradex,verdin-imx8mp-nonwifi' was expected
> 	'toradex,verdin-imx8mp-wifi' was expected
> 	'tq,imx8mp-tqma8mpql' was expected
> 	'variscite,var-som-mx8mp' was expected
> 	'fsl,imx8mq' was expected
> 	'boundary,imx8mq-nitrogen8m-som' was expected
> 	'purism,librem5' was expected
> 	'tq,imx8mq-tqma8mq' was expected
> 	'zii,imx8mq-ultra' was expected
> 	'fsl,imx8qm' was expected
> 	'toradex,apalis-imx8' was expected
> 	'toradex,apalis-imx8-v1.1' was expected
> 	'fsl,imx8dxl' was expected
> 	'toradex,colibri-imx8x' was expected
> 	'fsl,imx8ulp' was expected
> 	'fsl,imx93' was expected
> 	'fsl,imx95' was expected
> 	'fsl,imxrt1050' was expected
> 	'fsl,imxrt1170' was expected
> 	'tq,imx93-tqma9352' was expected
> 	'phytec,imx93-phycore-som' was expected
> 	'variscite,var-som-mx93' was expected
> 	'kontron,imx93-osm-s' was expected
> 	'toradex,vf500-colibri_vf50' was expected
> 	'fsl,vf610' was expected
> 	'toradex,vf610-colibri_vf61' was expected
> 	'zii,vf610dev' was expected
> 	'fsl,ls1012a' was expected
> 	'fsl,ls1021a' was expected
> 	'tq,ls1021a-tqmls1021a' was expected
> 	'fsl,ls1028a' was expected
> 	'kontron,sl28-var4' was expected
> 	'abb,imx8mp-aristianetos3-som' is not one of ['kontron,sl28-var1', 'kontron,sl28-var2', 'kontron,sl28-var3', 'kontron,sl28-var4']
> 	'fsl,ls1043a' was expected
> 	'tq,ls1043a-tqmls1043a' was expected
> 	'fsl,ls1046a' was expected
> 	'tq,ls1046a-tqmls1046a' was expected
> 	'fsl,ls1088a' was expected
> 	'tq,ls1088a-tqmls1088a' was expected
> 	'fsl,ls2080a' was expected
> 	'fsl,ls2081a' was expected
> 	'fsl,ls2088a' was expected
> 	'fsl,lx2160a' was expected
> 	'solidrun,lx2160a-cex7' was expected
> 	'solidrun,lx2162a-som' was expected
> 	'tq,lx2160a-tqmlx2160a' was expected
> 	'nxp,s32g2' was expected
> 	'nxp,s32g3' was expected
> 	'fsl,s32v234' was expected
> 	'phytec,imx6qdl-pcm058' was expected
> 	'tq,imx6q-tqma6q-a' was expected
> 	'tq,imx6q-tqma6q-b' was expected
> 	'tq,imx6qp-tqma6qp-b' was expected
> 	'tq,imx6dl-tqma6dl-a' was expected
> 	'tq,imx6dl-tqma6dl-b' was expected
> 	'phytec,imx6ul-pcl063' was expected
> 	'phytec,imx6ull-pcl063' was expected
> 	'fsl,imx6ulz' was expected
> 	'toradex,verdin-imx8mm' was expected
> 	'avnet,sm2s-imx8mp' was expected
> 	'toradex,verdin-imx8mp' was expected
> 	'fsl,imx8mp' is not one of ['fsl,imx8qxp', 'fsl,imx8dx']
> 	'fsl,vf500' was expected
> 	from schema $id: http://devicetree.org/schemas/arm/fsl.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pinctrl@30330000: 'pinctrl-0' is a dependency of 'pinctrl-names'
> 	from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-consumer.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: clock-controller@30380000: 'init-on-array' does not match any of the regexes: 'pinctrl-[0-9]+'
> 	from schema $id: http://devicetree.org/schemas/clock/imx8m-clock.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pwm@30670000: #pwm-cells: 3 was expected
> 	from schema $id: http://devicetree.org/schemas/pwm/imx-pwm.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: spi@30820000: Unevaluated properties are not allowed ('fsl,spi-num-chipselects' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/spi/fsl-imx-cspi.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: spi@30830000: Unevaluated properties are not allowed ('fsl,spi-num-chipselects' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/spi/fsl-imx-cspi.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO1: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO2: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO3: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO4: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:LDO5: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK1: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK2: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK4: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK5: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pmic@25: regulators:BUCK6: Unevaluated properties are not allowed ('regulator-compatible' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/regulator/nxp,pca9450-regulator.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: compatible: 'oneOf' conditional failed, one must be fixed:
> 	['st,st33ktpm2xi2c'] is too short
> 	'st,st33ktpm2xi2c' is not one of ['atmel,at97sc3204t', 'infineon,slb9635tt', 'infineon,slb9645tt', 'infineon,tpm_i2c_infineon', 'nuvoton,npct501', 'nuvoton,npct601', 'st,st33zp24-i2c', 'winbond,wpct301']
> 	from schema $id: http://devicetree.org/schemas/tpm/tcg,tpm-tis-i2c.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: Unevaluated properties are not allowed ('#gpio-cells', '#interrupt-cells', 'compatible', 'reset-gpio' were unexpected)
> 	from schema $id: http://devicetree.org/schemas/tpm/tcg,tpm-tis-i2c.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: tpm@2e: 'gpio-controller' is a dependency of '#gpio-cells'
> 	from schema $id: http://devicetree.org/schemas/gpio/gpio.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext0@3f: $nodename:0: 'i2c6_gpioext0@3f' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
> 	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext1@70: $nodename:0: 'i2c6_gpioext1@70' does not match '^(pinctrl|pinmux)(@[0-9a-f]+)?$'
> 	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: i2c6_gpioext1@70: 'aristainetos3-tpm-grp' does not match any of the regexes: '-cfg$', 'pinctrl-[0-9]+'
> 	from schema $id: http://devicetree.org/schemas/pinctrl/semtech,sx1501q.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: tx-queues-config: 'snps,tx-sched-sp' does not match any of the regexes: '^queue[0-9]$', 'pinctrl-[0-9]+'
> 	from schema $id: http://devicetree.org/schemas/net/nxp,dwmac-imx.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: Unevaluated properties are not allowed ('interrupt-names', 'interrupts', 'mdio', 'phy-handle', 'phy-mode', 'reg', 'rx-queues-config', 'snps,force_thresh_dma_mode', 'snps,mtl-rx-config', 'snps,mtl-tx-config', 'tx-queues-config' were unexpected)
> 	from schema $id: http://devicetree.org/schemas/net/nxp,dwmac-imx.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: ethernet@30bf0000: tx-queues-config: 'snps,tx-sched-sp' does not match any of the regexes: '^queue[0-9]$', 'pinctrl-[0-9]+'
> 	from schema $id: http://devicetree.org/schemas/net/snps,dwmac.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names: ['pcie', 'pcie_aux', 'pcie_phy', 'pcie_bus'] is too long
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clocks: [[2, 268], [2, 120], [2, 311], [2, 217]] is too long
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names:1: 'pcie_bus' was expected
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names:2: 'pcie_aux' was expected
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: clock-names: ['pcie', 'pcie_aux', 'pcie_phy', 'pcie_bus'] is too long
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: pcie@33800000: Unevaluated properties are not allowed ('ext_osc', 'fsl,max-link-speed', 'fsl,tx-deemph-gen1', 'power-domains', 'reserved-region' were unexpected)
> 	from schema $id: http://devicetree.org/schemas/pci/fsl,imx6q-pcie.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: usb-phy@381f0040: Unevaluated properties are not allowed ('fsl,pcs-tx-deemph-3p5db', 'fsl,phy-comp-dis-tune', 'fsl,phy-pcs-tx-swing-full', 'fsl,phy-tx-preemp-amp-tune', 'fsl,phy-tx-vboost-level', 'fsl,phy-tx-vref-tune' were unexpected)
> 	from schema $id: http://devicetree.org/schemas/phy/fsl,imx8mq-usb-phy.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: usb-phy@382f0040: Unevaluated properties are not allowed ('fsl,phy-tx-preemp-amp-tune' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/phy/fsl,imx8mq-usb-phy.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: $nodename:0: 'fixed-regulators' does not match '^([a-z][a-z0-9\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
> 	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: #size-cells: 0 is not one of [1, 2]
> 	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: regulator-enet: 'ranges' is a required property
> 	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
> arch/arm64/boot/dts/freescale/imx8mp-aristainetos3a-som-v1.dtb: fixed-regulators: 'ranges' is a required property
> 	from schema $id: http://devicetree.org/schemas/simple-bus.yaml#
> 
> 
> 
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-10-29 12:52 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-28  8:23 [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Heiko Schocher
2024-10-28  8:23 ` [PATCH v1 1/2] dt-bindings: arm: fsl: Add " Heiko Schocher
2024-10-28 10:20   ` Krzysztof Kozlowski
2024-10-28  8:23 ` [PATCH v1 2/2] arm64: dts: imx8mp: add aristainetos3 board support Heiko Schocher
2024-10-28 10:24   ` Krzysztof Kozlowski
2024-10-28 10:41     ` Heiko Schocher
2024-10-28 10:49       ` Krzysztof Kozlowski
2024-10-28 11:21         ` Heiko Schocher
2024-10-28 12:44           ` Krzysztof Kozlowski
2024-10-28 13:52             ` Heiko Schocher
2024-10-29 12:44 ` [PATCH v1 0/2] arm64: dts: imx8mp: add support for the ABB SoM and carrier Rob Herring (Arm)
2024-10-29 12:50   ` Heiko Schocher

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).