public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] arm64: dts: a7k: add COM Express boards
@ 2023-12-18 15:44 Elad Nachman
  0 siblings, 0 replies; 15+ messages in thread
From: Elad Nachman @ 2023-12-18 15:44 UTC (permalink / raw)
  To: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
	chris.packham, devicetree, linux-kernel, linux-arm-kernel
  Cc: enachman, cyuval

From: Elad Nachman <enachman@marvell.com>

Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
Add device tree bindings for this board.
Define this COM Express CPU module as dtsi and provide a dtsi file for
the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).

Since memory is soldered on CPU module, memory node is on CPU module
dtsi file.

This Carrier board only utilizes the PCIe link, hence no special device
or driver support is provided by this dtsi file.
Devise a dts file for the combined com express carrier and CPU module.

The Aramda 7020 CPU COM Express board offers the following features:

1.  Armada 7020 CPU, with dual ARM A72 cores
2.  DDR4 memory, 8GB, on board soldered
3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
    all are present on A7K CPU module (none on the carrier)
4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
5.  On-board 8 Gbit, 8-bit bus width NAND flash
6.  On-board 512 Mbit SPI flash
7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
    COM Express type 7 connector
8.  m.2 SATA connector
9.  Micro-SD card connector
10. USB 2.0 via COM Express type 7 connector
11. Two i2c interfaces - one to the CPU module, and one to the
    carrier board via the COM Express type 7 connector
12. UART (mini USB connector by virtue of FT2232D UART to USB
    converter, connected to the Armada 7020 UART0)

Elad Nachman (2):
  dt-bindings: arm64: add Marvell 7k COMe boards
  arm64: dts: a7k: add COM Express boards

 .../bindings/arm/marvell/armada-7k-8k.yaml    |  11 ++
 arch/arm64/boot/dts/marvell/Makefile          |   1 +
 .../dts/marvell/armada-7020-comexpress.dtsi   | 161 ++++++++++++++++++
 arch/arm64/boot/dts/marvell/armada-70x0.dtsi  |   7 +
 .../boot/dts/marvell/armada-ap806-dual.dtsi   |   4 +-
 .../dts/marvell/db-falcon-carrier-a7k.dts     |  27 +++
 .../boot/dts/marvell/db-falcon-carrier.dtsi   |  22 +++
 7 files changed, 231 insertions(+), 2 deletions(-)
 create mode 100644 arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi

-- 
2.25.1


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

* [PATCH 0/2] arm64: dts: a7k: add COM Express boards
@ 2026-01-22 16:59 Elad Nachman
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
                   ` (3 more replies)
  0 siblings, 4 replies; 15+ messages in thread
From: Elad Nachman @ 2026-01-22 16:59 UTC (permalink / raw)
  To: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
	chris.packham, devicetree, linux-kernel, linux-arm-kernel
  Cc: enachman

From: Elad Nachman <enachman@marvell.com>

Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
Add device tree bindings for this board.
Define this COM Express CPU module as dtsi and provide a dtsi file for
the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
Add the Falcon DB to the MAINTAINERS list

Since memory is soldered on CPU module, memory node is on CPU module
dtsi file.

This Carrier board only utilizes the PCIe link, hence no special device
or driver support is provided by this dtsi file.
Devise a dts file for the combined com express carrier and CPU module.

The Aramda 7020 CPU COM Express board offers the following features:

1.  Armada 7020 CPU, with dual ARM A72 cores
2.  DDR4 memory, 8GB, on board soldered
3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
    all are present on A7K CPU module (none on the carrier)
4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
5.  On-board 8 Gbit, 8-bit bus width NAND flash
6.  On-board 512 Mbit SPI flash
7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
    COM Express type 7 connector
8.  m.2 SATA connector
9.  Micro-SD card connector
10. USB 2.0 via COM Express type 7 connector
11. Two i2c interfaces - one to the CPU module, and one to the
    carrier board via the COM Express type 7 connector
12. UART (mini USB connector by virtue of FT2232D UART to USB
    converter, connected to the Armada 7020 UART0)

Elad Nachman (3):
  dt-bindings: arm64: add Marvell 7k COMe boards
  arm64: dts: a7k: add COM Express boards
  MAINTAINERS: Add Falcon DB

 .../bindings/arm/marvell/armada-7k-8k.yaml    |  11 ++
 MAINTAINERS                                   |   1 +
 arch/arm64/boot/dts/marvell/Makefile          |   1 +
 .../dts/marvell/armada-7020-comexpress.dtsi   | 161 ++++++++++++++++++
 arch/arm64/boot/dts/marvell/armada-70x0.dtsi  |   7 +
 .../boot/dts/marvell/armada-ap806-dual.dtsi   |   4 +-
 .../dts/marvell/db-falcon-carrier-a7k.dts     |  27 +++
 .../boot/dts/marvell/db-falcon-carrier.dtsi   |  22 +++
 8 files changed, 232 insertions(+), 2 deletions(-)
 create mode 100644 arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi

-- 
2.25.1


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

* [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-01-22 16:59 [PATCH 0/2] arm64: dts: a7k: add COM Express boards Elad Nachman
@ 2026-01-22 16:59 ` Elad Nachman
  2026-01-22 23:21   ` Rob Herring (Arm)
                     ` (2 more replies)
  2026-01-22 16:59 ` [PATCH 2/3] arm64: dts: a7k: add COM Express boards Elad Nachman
                   ` (2 subsequent siblings)
  3 siblings, 3 replies; 15+ messages in thread
From: Elad Nachman @ 2026-01-22 16:59 UTC (permalink / raw)
  To: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
	chris.packham, devicetree, linux-kernel, linux-arm-kernel
  Cc: enachman

From: Elad Nachman <enachman@marvell.com>

Add dt bindings for:
Armada 7020 COM Express CPU module
Falcon DB-98CX85x0 COM Express type 7 Carrier board
Falcon DB-98CX85x0 COM Express type 7 Carrier board
with an Armada 7020 COM Express CPU module

Signed-off-by: Elad Nachman <enachman@marvell.com>
---
 .../devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
index 4bc7454a5d3a..7e77310da626 100644
--- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
+++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
@@ -21,6 +21,17 @@ properties:
           - const: marvell,armada-ap806-dual
           - const: marvell,armada-ap806
 
+      - description:
+          Falcon (DB-98CX85x0) Development board COM Express Carrier plus
+          Armada 7020 SoC COM Express CPU module
+        items:
+          - const: marvell,armada7020-falcon-carrier
+          - const: marvell,db-falcon-carrier
+          - const: marvell,armada7020-cpu-module
+          - const: marvell,armada7020
+          - const: marvell,armada-ap806-dual
+          - const: marvell,armada-ap806
+
       - description: Armada 7040 SoC
         items:
           - enum:
-- 
2.25.1


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

* [PATCH 2/3] arm64: dts: a7k: add COM Express boards
  2026-01-22 16:59 [PATCH 0/2] arm64: dts: a7k: add COM Express boards Elad Nachman
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
@ 2026-01-22 16:59 ` Elad Nachman
  2026-01-22 16:59 ` [PATCH 3/3] MAINTAINERS: Add Falcon DB Elad Nachman
  2026-01-22 18:07 ` [PATCH 0/2] arm64: dts: a7k: add COM Express boards Rob Herring
  3 siblings, 0 replies; 15+ messages in thread
From: Elad Nachman @ 2026-01-22 16:59 UTC (permalink / raw)
  To: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
	chris.packham, devicetree, linux-kernel, linux-arm-kernel
  Cc: enachman

From: Elad Nachman <enachman@marvell.com>

Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
Define this COM Express CPU module as dtsi and provide a dtsi file for
the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).

Since memory is soldered on CPU module, memory node is on CPU module
dtsi file.

This Carrier board only utilizes the PCIe link, hence no special device
or driver support is provided by this dtsi file.
Devise a dts file for the combined com express carrier and CPU module.

The Aramda 7020 CPU COM Express board offers the following features:

1.  Armada 7020 CPU, with dual ARM A72 cores
2.  DDR4 memory, 8GB, on board soldered
3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
    all are present on A7K CPU module (none on the carrier)
4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
5.  On-board 8 Gbit, 8-bit bus width NAND flash
6.  On-board 512 Mbit SPI flash
7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
    COM Express type 7 connector
8.  m.2 SATA connector
9.  Micro-SD card connector
10. USB 2.0 via COM Express type 7 connector
11. Two i2c interfaces - one to the CPU module, and one to the
    carrier board via the COM Express type 7 connector
12. UART (mini USB connector by virtue of FT2232D UART to USB
    converter, connected to the Armada 7020 UART0)

Signed-off-by: Elad Nachman <enachman@marvell.com>
---
 arch/arm64/boot/dts/marvell/Makefile          |   1 +
 .../dts/marvell/armada-7020-comexpress.dtsi   | 161 ++++++++++++++++++
 arch/arm64/boot/dts/marvell/armada-70x0.dtsi  |   7 +
 .../boot/dts/marvell/armada-ap806-dual.dtsi   |   4 +-
 .../dts/marvell/db-falcon-carrier-a7k.dts     |  27 +++
 .../boot/dts/marvell/db-falcon-carrier.dtsi   |  22 +++
 6 files changed, 220 insertions(+), 2 deletions(-)
 create mode 100644 arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
 create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi

diff --git a/arch/arm64/boot/dts/marvell/Makefile b/arch/arm64/boot/dts/marvell/Makefile
index a774bc74a0a0..09d1544041af 100644
--- a/arch/arm64/boot/dts/marvell/Makefile
+++ b/arch/arm64/boot/dts/marvell/Makefile
@@ -11,6 +11,7 @@ dtb-$(CONFIG_ARCH_MVEBU) += armada-3720-espressobin-v7-emmc.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-3720-gl-mv1000.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-3720-turris-mox.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-3720-uDPU.dtb
+dtb-$(CONFIG_ARCH_MVEBU) += db-falcon-carrier-a7k.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-7040-db.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-7040-mochabin.dtb
 dtb-$(CONFIG_ARCH_MVEBU) += armada-8040-clearfog-gt-8k.dtb
diff --git a/arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi b/arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
new file mode 100644
index 000000000000..dd91bfa9f880
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
@@ -0,0 +1,161 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell Technology Group Ltd.
+ *
+ * Device Tree file for Marvell Armada 7020 Com Express CPU module board.
+ */
+
+#include "armada-7020.dtsi"
+
+/ {
+	model = "Marvell Armada-7020 COMEXPRESS board setup";
+	compatible = "marvell,armada7020-cpu-module", "marvell,armada7020",
+			"marvell,armada-ap806-dual", "marvell,armada-ap806";
+
+	memory@0 {
+		device_type = "memory";
+		reg = <0x0 0x0 0x2 0x00000000>;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	aliases {
+		ethernet0 = &cp0_eth0;
+		ethernet1 = &cp0_eth1;
+	};
+};
+
+&ap_clk {
+	status = "okay";
+};
+
+&gic {
+	status = "okay";
+};
+
+&i2c0 {
+	status = "okay";
+	clock-frequency = <100000>;
+};
+
+&spi0 {
+	status = "okay";
+};
+
+&uart0 {
+	status = "okay";
+};
+
+&cp0_mdio {
+	status = "okay";
+
+	phy0: ethernet-phy@10 {
+		reg = <0x10>;
+	};
+};
+
+&cp0_ethernet {
+	status = "okay";
+};
+
+&cp0_eth0 {
+	status = "okay";
+	phy-mode = "10gbase-kr";
+	managed = "in-band-status";
+	/* Generic PHY, providing serdes lanes */
+	phys = <&cp0_comphy4 0>;
+};
+
+&cp0_eth1 {
+	status = "okay";
+	phy = <&phy0>;
+	phy-mode = "rgmii-id";
+};
+
+&cp0_usb3_0 {
+	status = "okay";
+};
+
+&cp0_usb3_1 {
+	status = "okay";
+};
+
+&cp0_clk {
+	status = "okay";
+};
+
+&cp0_i2c0 {
+	status = "okay";
+	clock-frequency = <100000>;
+};
+
+&cp0_nand_controller {
+	status = "okay";
+
+	nand@0 {
+		reg = <0>;
+		label = "main-storage";
+		nand-rb = <0>;
+		nand-ecc-mode = "hw";
+		nand-on-flash-bbt;
+		nand-ecc-strength = <8>;
+		nand-ecc-step-size = <512>;
+
+		partitions {
+			compatible = "fixed-partitions";
+			#address-cells = <1>;
+			#size-cells = <1>;
+
+			partition@0 {
+				label = "U-Boot";
+				reg = <0 0x400000>;
+			};
+			partition@200000 {
+				label = "Linux";
+				reg = <0x400000 0x100000>;
+			};
+			partition@1000000 {
+				label = "Filesystem";
+				reg = <0x500000 0x1e00000>;
+			};
+		};
+	};
+};
+
+&cp0_pcie0 {
+	status = "okay";
+	num-lanes = <4>;
+	num-viewport = <8>;
+
+	ranges = <0x81000000 0x0 0xfa000000 0x0 0xfa000000 0x0 0x00010000
+		  0x82000000 0x0 0x00000000 0x0 0xc0000000 0x0 0x30000000>;
+
+	/* Generic PHY, providing serdes lanes */
+	phys = <&cp0_comphy0 0
+		&cp0_comphy1 0
+		&cp0_comphy2 0
+		&cp0_comphy3 0>;
+};
+
+&cp0_sata0 {
+	/* CPM Lane 0 - U29 */
+	status = "okay";
+
+	sata-port@1 {
+		status = "okay";
+		/* Generic PHY, providing serdes lanes */
+		phys = <&cp0_comphy5 1>;
+	};
+};
+
+&cp0_sdhci0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&sdhci_pins>;
+	status = "okay";
+	bus-width = <4>;
+	no-1-8-v;
+	broken-cd;
+};
+
diff --git a/arch/arm64/boot/dts/marvell/armada-70x0.dtsi b/arch/arm64/boot/dts/marvell/armada-70x0.dtsi
index df939426d258..36e0a8a0ade3 100644
--- a/arch/arm64/boot/dts/marvell/armada-70x0.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-70x0.dtsi
@@ -47,6 +47,13 @@ &cp0_syscon0 {
 	cp0_pinctrl: pinctrl {
 		compatible = "marvell,armada-7k-pinctrl";
 
+
+		sdhci_pins: sdhci-pins {
+			marvell,pins = "mpp56", "mpp57", "mpp58",
+				       "mpp59", "mpp60", "mpp61", "mpp62";
+			marvell,function = "sdio";
+		};
+
 		nand_pins: nand-pins {
 			marvell,pins =
 			"mpp15", "mpp16", "mpp17", "mpp18",
diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
index 82f4dedfc25e..0868d59d561b 100644
--- a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
@@ -54,7 +54,7 @@ l2: l2-cache {
 	};
 
 	thermal-zones {
-		/delete-node/ ap-thermal-cpu2;
-		/delete-node/ ap-thermal-cpu3;
+		/delete-node/ ap-cpu2-thermal;
+		/delete-node/ ap-cpu3-thermal;
 	};
 };
diff --git a/arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts b/arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
new file mode 100644
index 000000000000..5d1ae7b35b62
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
@@ -0,0 +1,27 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the Falcon DB Type 7 Com Express carrier board,
+ * Utilizing the Armada 7020 COM Express CPU module board.
+ * This specific carrier board (DB-98CX8540/80)
+ * only maintains a PCIe link with the CPU module,
+ * which does not require any special DTS definitions.
+ *
+ * There is no Linux CPU booting in this mode on the carrier, only on the
+ * Armada 7020 COM Express CPU module.
+ * What runs the Linux is the Armada 7020 on the COM Express CPU module,
+ * And it accesses the switch end-point on the Falcon DB portion of the carrier
+ * via PCIe.
+ */
+
+#include "armada-7020-comexpress.dtsi"
+#include "db-falcon-carrier.dtsi"
+
+/ {
+	model = "Marvell Falcon DB COM EXPRESS type 7 carrier board with Armada 7020 CPU module";
+	compatible = "marvell,armada7020-falcon-carrier", "marvell,db-falcon-carrier",
+			"marvell,armada7020-cpu-module", "marvell,armada7020",
+			"marvell,armada-ap806-dual", "marvell,armada-ap806";
+
+};
diff --git a/arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi b/arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi
new file mode 100644
index 000000000000..c85ad1547ec5
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi
@@ -0,0 +1,22 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the Falcon DB Type 7 Com Express carrier board,
+ * This (DB-98CX8540/80) specific carrier board only maintains
+ * a PCIe link with the COM Express CPU module, which does not
+ * require any special DTS definitions.
+ *
+ * The board contains the 98CX8540/80 Switch, which connects by
+ * PCIe to the COM Express CPU module.
+ * This CPU module can be any standard COM Express CPU module with
+ * PCIe support.
+ *
+ * There is no Linux CPU booting in this mode on the carrier,
+ * only on the COM Express CPU module.
+ */
+
+/ {
+	model = "Marvell Armada Falcon DB COM EXPRESS type 7 carrier board";
+	compatible = "marvell,db-falcon-carrier";
+};
-- 
2.25.1


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

* [PATCH 3/3] MAINTAINERS: Add Falcon DB
  2026-01-22 16:59 [PATCH 0/2] arm64: dts: a7k: add COM Express boards Elad Nachman
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
  2026-01-22 16:59 ` [PATCH 2/3] arm64: dts: a7k: add COM Express boards Elad Nachman
@ 2026-01-22 16:59 ` Elad Nachman
  2026-01-22 18:07 ` [PATCH 0/2] arm64: dts: a7k: add COM Express boards Rob Herring
  3 siblings, 0 replies; 15+ messages in thread
From: Elad Nachman @ 2026-01-22 16:59 UTC (permalink / raw)
  To: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
	chris.packham, devicetree, linux-kernel, linux-arm-kernel
  Cc: enachman

From: Elad Nachman <enachman@marvell.com>

Add Falcon DB to the list of maintained Marvell Armada dts files

Signed-off-by: Elad Nachman <enachman@marvell.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index a56f8f00aebb..e3f8cec31d30 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2971,6 +2971,7 @@ S:	Maintained
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/gclement/mvebu.git
 F:	Documentation/devicetree/bindings/arm/marvell/
 F:	arch/arm/boot/dts/marvell/armada*
+F:	arch/arm/boot/dts/marvell/db-falcon*
 F:	arch/arm/boot/dts/marvell/kirkwood*
 F:	arch/arm/configs/mvebu_*_defconfig
 F:	arch/arm/mach-mvebu/
-- 
2.25.1


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

* Re: [PATCH 0/2] arm64: dts: a7k: add COM Express boards
  2026-01-22 16:59 [PATCH 0/2] arm64: dts: a7k: add COM Express boards Elad Nachman
                   ` (2 preceding siblings ...)
  2026-01-22 16:59 ` [PATCH 3/3] MAINTAINERS: Add Falcon DB Elad Nachman
@ 2026-01-22 18:07 ` Rob Herring
  2026-01-23  9:10   ` Gregory CLEMENT
  3 siblings, 1 reply; 15+ messages in thread
From: Rob Herring @ 2026-01-22 18:07 UTC (permalink / raw)
  To: Elad Nachman
  Cc: conor+dt, linux-kernel, robh+dt, chris.packham, pali, andrew,
	devicetree, mrkiko.rs, sebastian.hesselbarth, linux-arm-kernel,
	gregory.clement, krzysztof.kozlowski+dt


On Thu, 22 Jan 2026 18:59:20 +0200, Elad Nachman wrote:
> From: Elad Nachman <enachman@marvell.com>
> 
> Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
> Add device tree bindings for this board.
> Define this COM Express CPU module as dtsi and provide a dtsi file for
> the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
> Add the Falcon DB to the MAINTAINERS list
> 
> Since memory is soldered on CPU module, memory node is on CPU module
> dtsi file.
> 
> This Carrier board only utilizes the PCIe link, hence no special device
> or driver support is provided by this dtsi file.
> Devise a dts file for the combined com express carrier and CPU module.
> 
> The Aramda 7020 CPU COM Express board offers the following features:
> 
> 1.  Armada 7020 CPU, with dual ARM A72 cores
> 2.  DDR4 memory, 8GB, on board soldered
> 3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
>     all are present on A7K CPU module (none on the carrier)
> 4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
> 5.  On-board 8 Gbit, 8-bit bus width NAND flash
> 6.  On-board 512 Mbit SPI flash
> 7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
>     COM Express type 7 connector
> 8.  m.2 SATA connector
> 9.  Micro-SD card connector
> 10. USB 2.0 via COM Express type 7 connector
> 11. Two i2c interfaces - one to the CPU module, and one to the
>     carrier board via the COM Express type 7 connector
> 12. UART (mini USB connector by virtue of FT2232D UART to USB
>     converter, connected to the Armada 7020 UART0)
> 
> Elad Nachman (3):
>   dt-bindings: arm64: add Marvell 7k COMe boards
>   arm64: dts: a7k: add COM Express boards
>   MAINTAINERS: Add Falcon DB
> 
>  .../bindings/arm/marvell/armada-7k-8k.yaml    |  11 ++
>  MAINTAINERS                                   |   1 +
>  arch/arm64/boot/dts/marvell/Makefile          |   1 +
>  .../dts/marvell/armada-7020-comexpress.dtsi   | 161 ++++++++++++++++++
>  arch/arm64/boot/dts/marvell/armada-70x0.dtsi  |   7 +
>  .../boot/dts/marvell/armada-ap806-dual.dtsi   |   4 +-
>  .../dts/marvell/db-falcon-carrier-a7k.dts     |  27 +++
>  .../boot/dts/marvell/db-falcon-carrier.dtsi   |  22 +++
>  8 files changed, 232 insertions(+), 2 deletions(-)
>  create mode 100644 arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
>  create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
>  create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi
> 
> --
> 2.25.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


This patch series was applied (using b4) to base:
 Base: attempting to guess base-commit...
 Base: tags/next-20260121 (exact match)
 Base: tags/next-20260121 (use --merge-base to override)

If this is not the correct base, please add 'base-commit' tag
(or use b4 which does this automatically)


New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/marvell/' for 20260122165923.2316510-1-enachman@marvell.com:

arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dtb: ethernet@0 (marvell,armada-7k-pp22): ethernet-port@0:phy-mode:0: '10gbase-kr' is not one of ['gmii', 'sgmii', 'rgmii-id', '1000base-x', '2500base-x', '5gbase-r', 'rxaui', '10gbase-r']
	from schema $id: http://devicetree.org/schemas/net/marvell,pp2.yaml


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

* Re: [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
@ 2026-01-22 23:21   ` Rob Herring (Arm)
  2026-03-02 15:43   ` Gregory CLEMENT
  2026-03-13 15:49   ` Gregory CLEMENT
  2 siblings, 0 replies; 15+ messages in thread
From: Rob Herring (Arm) @ 2026-01-22 23:21 UTC (permalink / raw)
  To: Elad Nachman
  Cc: chris.packham, linux-kernel, krzysztof.kozlowski+dt,
	linux-arm-kernel, pali, mrkiko.rs, devicetree, robh+dt, andrew,
	gregory.clement, conor+dt, sebastian.hesselbarth


On Thu, 22 Jan 2026 18:59:21 +0200, Elad Nachman wrote:
> From: Elad Nachman <enachman@marvell.com>
> 
> Add dt bindings for:
> Armada 7020 COM Express CPU module
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> with an Armada 7020 COM Express CPU module
> 
> Signed-off-by: Elad Nachman <enachman@marvell.com>
> ---
>  .../devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>


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

* Re: [PATCH 0/2] arm64: dts: a7k: add COM Express boards
  2026-01-22 18:07 ` [PATCH 0/2] arm64: dts: a7k: add COM Express boards Rob Herring
@ 2026-01-23  9:10   ` Gregory CLEMENT
  2026-01-23  9:20     ` Maxime Chevallier
  0 siblings, 1 reply; 15+ messages in thread
From: Gregory CLEMENT @ 2026-01-23  9:10 UTC (permalink / raw)
  To: Rob Herring, Elad Nachman, andrew, Maxime Chevallier
  Cc: conor+dt, linux-kernel, robh+dt, chris.packham, pali, devicetree,
	mrkiko.rs, sebastian.hesselbarth, linux-arm-kernel,
	krzysztof.kozlowski+dt

Hello,

> On Thu, 22 Jan 2026 18:59:20 +0200, Elad Nachman wrote:
>> From: Elad Nachman <enachman@marvell.com>
>> 
>> Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
>> Add device tree bindings for this board.
>> Define this COM Express CPU module as dtsi and provide a dtsi file for
>> the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
>> Add the Falcon DB to the MAINTAINERS list
>> 
>> Since memory is soldered on CPU module, memory node is on CPU module
>> dtsi file.
>> 
>> This Carrier board only utilizes the PCIe link, hence no special device
>> or driver support is provided by this dtsi file.
>> Devise a dts file for the combined com express carrier and CPU module.
>> 
>> The Aramda 7020 CPU COM Express board offers the following features:
>> 
>> 1.  Armada 7020 CPU, with dual ARM A72 cores
>> 2.  DDR4 memory, 8GB, on board soldered
>> 3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
>>     all are present on A7K CPU module (none on the carrier)
>> 4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
>> 5.  On-board 8 Gbit, 8-bit bus width NAND flash
>> 6.  On-board 512 Mbit SPI flash
>> 7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
>>     COM Express type 7 connector
>> 8.  m.2 SATA connector
>> 9.  Micro-SD card connector
>> 10. USB 2.0 via COM Express type 7 connector
>> 11. Two i2c interfaces - one to the CPU module, and one to the
>>     carrier board via the COM Express type 7 connector
>> 12. UART (mini USB connector by virtue of FT2232D UART to USB
>>     converter, connected to the Armada 7020 UART0)
>> 
>> Elad Nachman (3):
>>   dt-bindings: arm64: add Marvell 7k COMe boards
>>   arm64: dts: a7k: add COM Express boards
>>   MAINTAINERS: Add Falcon DB
>> 
>>  .../bindings/arm/marvell/armada-7k-8k.yaml    |  11 ++
>>  MAINTAINERS                                   |   1 +
>>  arch/arm64/boot/dts/marvell/Makefile          |   1 +
>>  .../dts/marvell/armada-7020-comexpress.dtsi   | 161 ++++++++++++++++++
>>  arch/arm64/boot/dts/marvell/armada-70x0.dtsi  |   7 +
>>  .../boot/dts/marvell/armada-ap806-dual.dtsi   |   4 +-
>>  .../dts/marvell/db-falcon-carrier-a7k.dts     |  27 +++
>>  .../boot/dts/marvell/db-falcon-carrier.dtsi   |  22 +++
>>  8 files changed, 232 insertions(+), 2 deletions(-)
>>  create mode 100644 arch/arm64/boot/dts/marvell/armada-7020-comexpress.dtsi
>>  create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dts
>>  create mode 100644 arch/arm64/boot/dts/marvell/db-falcon-carrier.dtsi
>> 
>> --
>> 2.25.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
>
>
> This patch series was applied (using b4) to base:
>  Base: attempting to guess base-commit...
>  Base: tags/next-20260121 (exact match)
>  Base: tags/next-20260121 (use --merge-base to override)
>
> If this is not the correct base, please add 'base-commit' tag
> (or use b4 which does this automatically)
>
>
> New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/marvell/' for 20260122165923.2316510-1-enachman@marvell.com:
>
> arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dtb: ethernet@0 (marvell,armada-7k-pp22): ethernet-port@0:phy-mode:0: '10gbase-kr' is not one of ['gmii', 'sgmii', 'rgmii-id', '1000base-x', '2500base-x', '5gbase-r', 'rxaui', '10gbase-r']
> 	from schema $id:
> http://devicetree.org/schemas/net/marvell,pp2.yaml

Does this mean we should add 10gbase-kr to the phy-mode enum list in
Documentation/devicetree/bindings/net/marvell?

Adding it would likely resolve the warning, but I’m not sure if it’s the
correct approach from the driver’s perspective. That’s why I’ve included
Andrew in the To field and added Maxime as well.

Regards,

Gregory

>

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 0/2] arm64: dts: a7k: add COM Express boards
  2026-01-23  9:10   ` Gregory CLEMENT
@ 2026-01-23  9:20     ` Maxime Chevallier
  2026-01-23  9:27       ` Gregory CLEMENT
  0 siblings, 1 reply; 15+ messages in thread
From: Maxime Chevallier @ 2026-01-23  9:20 UTC (permalink / raw)
  To: Gregory CLEMENT, Rob Herring, Elad Nachman, andrew
  Cc: conor+dt, linux-kernel, robh+dt, chris.packham, pali, devicetree,
	mrkiko.rs, sebastian.hesselbarth, linux-arm-kernel,
	krzysztof.kozlowski+dt

Hi,

On 23/01/2026 10:10, Gregory CLEMENT wrote:
> Hello,
> 
>> On Thu, 22 Jan 2026 18:59:20 +0200, Elad Nachman wrote:
>>> From: Elad Nachman <enachman@marvell.com>
>>>
>>> Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
>>> Add device tree bindings for this board.
>>> Define this COM Express CPU module as dtsi and provide a dtsi file for
>>> the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
>>> Add the Falcon DB to the MAINTAINERS list
>>>
>>> Since memory is soldered on CPU module, memory node is on CPU module
>>> dtsi file.
>>>
>>> This Carrier board only utilizes the PCIe link, hence no special device
>>> or driver support is provided by this dtsi file.
>>> Devise a dts file for the combined com express carrier and CPU module.
>>>
>>> The Aramda 7020 CPU COM Express board offers the following features:
>>>
>>> 1.  Armada 7020 CPU, with dual ARM A72 cores
>>> 2.  DDR4 memory, 8GB, on board soldered
>>> 3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
>>>     all are present on A7K CPU module (none on the carrier)
>>> 4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
>>> 5.  On-board 8 Gbit, 8-bit bus width NAND flash
>>> 6.  On-board 512 Mbit SPI flash
>>> 7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
>>>     COM Express type 7 connector
>>> 8.  m.2 SATA connector
>>> 9.  Micro-SD card connector
>>> 10. USB 2.0 via COM Express type 7 connector
>>> 11. Two i2c interfaces - one to the CPU module, and one to the
>>>     carrier board via the COM Express type 7 connector
>>> 12. UART (mini USB connector by virtue of FT2232D UART to USB
>>>     converter, connected to the Armada 7020 UART0)
>>>

 [...]

>> 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
>>
>>
>> This patch series was applied (using b4) to base:
>>  Base: attempting to guess base-commit...
>>  Base: tags/next-20260121 (exact match)
>>  Base: tags/next-20260121 (use --merge-base to override)
>>
>> If this is not the correct base, please add 'base-commit' tag
>> (or use b4 which does this automatically)
>>
>>
>> New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/marvell/' for 20260122165923.2316510-1-enachman@marvell.com:
>>
>> arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dtb: ethernet@0 (marvell,armada-7k-pp22): ethernet-port@0:phy-mode:0: '10gbase-kr' is not one of ['gmii', 'sgmii', 'rgmii-id', '1000base-x', '2500base-x', '5gbase-r', 'rxaui', '10gbase-r']
>> 	from schema $id:
>> http://devicetree.org/schemas/net/marvell,pp2.yaml
> 
> Does this mean we should add 10gbase-kr to the phy-mode enum list in
> Documentation/devicetree/bindings/net/marvell?

No, 10gbase-kr is legacy, one should use "10gbase-r" instead, cf commit

e0f909bc3a24 ("net: switch to using PHY_INTERFACE_MODE_10GBASER rather than 10GKR")

That should probably be documented in the bindings at some point :)

Maxime

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

* Re: [PATCH 0/2] arm64: dts: a7k: add COM Express boards
  2026-01-23  9:20     ` Maxime Chevallier
@ 2026-01-23  9:27       ` Gregory CLEMENT
  2026-01-27 16:23         ` Rob Herring
  0 siblings, 1 reply; 15+ messages in thread
From: Gregory CLEMENT @ 2026-01-23  9:27 UTC (permalink / raw)
  To: Maxime Chevallier, Rob Herring, Elad Nachman, andrew
  Cc: conor+dt, linux-kernel, robh+dt, chris.packham, pali, devicetree,
	mrkiko.rs, sebastian.hesselbarth, linux-arm-kernel,
	krzysztof.kozlowski+dt


> Hi,
>
> On 23/01/2026 10:10, Gregory CLEMENT wrote:
>> Hello,
>> 
>>> On Thu, 22 Jan 2026 18:59:20 +0200, Elad Nachman wrote:
>>>> From: Elad Nachman <enachman@marvell.com>
>>>>
>>>> Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
>>>> Add device tree bindings for this board.
>>>> Define this COM Express CPU module as dtsi and provide a dtsi file for
>>>> the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
>>>> Add the Falcon DB to the MAINTAINERS list
>>>>
>>>> Since memory is soldered on CPU module, memory node is on CPU module
>>>> dtsi file.
>>>>
>>>> This Carrier board only utilizes the PCIe link, hence no special device
>>>> or driver support is provided by this dtsi file.
>>>> Devise a dts file for the combined com express carrier and CPU module.
>>>>
>>>> The Aramda 7020 CPU COM Express board offers the following features:
>>>>
>>>> 1.  Armada 7020 CPU, with dual ARM A72 cores
>>>> 2.  DDR4 memory, 8GB, on board soldered
>>>> 3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
>>>>     all are present on A7K CPU module (none on the carrier)
>>>> 4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
>>>> 5.  On-board 8 Gbit, 8-bit bus width NAND flash
>>>> 6.  On-board 512 Mbit SPI flash
>>>> 7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
>>>>     COM Express type 7 connector
>>>> 8.  m.2 SATA connector
>>>> 9.  Micro-SD card connector
>>>> 10. USB 2.0 via COM Express type 7 connector
>>>> 11. Two i2c interfaces - one to the CPU module, and one to the
>>>>     carrier board via the COM Express type 7 connector
>>>> 12. UART (mini USB connector by virtue of FT2232D UART to USB
>>>>     converter, connected to the Armada 7020 UART0)
>>>>
>
>  [...]
>
>>> 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
>>>
>>>
>>> This patch series was applied (using b4) to base:
>>>  Base: attempting to guess base-commit...
>>>  Base: tags/next-20260121 (exact match)
>>>  Base: tags/next-20260121 (use --merge-base to override)
>>>
>>> If this is not the correct base, please add 'base-commit' tag
>>> (or use b4 which does this automatically)
>>>
>>>
>>> New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/marvell/' for 20260122165923.2316510-1-enachman@marvell.com:
>>>
>>> arch/arm64/boot/dts/marvell/db-falcon-carrier-a7k.dtb: ethernet@0 (marvell,armada-7k-pp22): ethernet-port@0:phy-mode:0: '10gbase-kr' is not one of ['gmii', 'sgmii', 'rgmii-id', '1000base-x', '2500base-x', '5gbase-r', 'rxaui', '10gbase-r']
>>> 	from schema $id:
>>> http://devicetree.org/schemas/net/marvell,pp2.yaml
>> 
>> Does this mean we should add 10gbase-kr to the phy-mode enum list in
>> Documentation/devicetree/bindings/net/marvell?
>
> No, 10gbase-kr is legacy, one should use "10gbase-r" instead, cf commit
>
> e0f909bc3a24 ("net: switch to using PHY_INTERFACE_MODE_10GBASER rather than 10GKR")
>
> That should probably be documented in the bindings at some point :)

Thanks, Maxime!

Elad, I’ll amend the commit unless you have a strong reason against it.

Gregory

>
> Maxime

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 0/2] arm64: dts: a7k: add COM Express boards
  2026-01-23  9:27       ` Gregory CLEMENT
@ 2026-01-27 16:23         ` Rob Herring
  0 siblings, 0 replies; 15+ messages in thread
From: Rob Herring @ 2026-01-27 16:23 UTC (permalink / raw)
  To: Gregory CLEMENT
  Cc: Maxime Chevallier, Elad Nachman, andrew, conor+dt, linux-kernel,
	chris.packham, pali, devicetree, mrkiko.rs, sebastian.hesselbarth,
	linux-arm-kernel, krzysztof.kozlowski+dt

On Fri, Jan 23, 2026 at 3:27 AM Gregory CLEMENT
<gregory.clement@bootlin.com> wrote:
>
>
> > Hi,
> >
> > On 23/01/2026 10:10, Gregory CLEMENT wrote:
> >> Hello,
> >>
> >>> On Thu, 22 Jan 2026 18:59:20 +0200, Elad Nachman wrote:
> >>>> From: Elad Nachman <enachman@marvell.com>
> >>>>
> >>>> Add support for Armada 7020 Express Type 7 CPU module board by Marvell.
> >>>> Add device tree bindings for this board.
> >>>> Define this COM Express CPU module as dtsi and provide a dtsi file for
> >>>> the carrier board (Marvell DB-98CX85x0 COM Express type 7 carrier board).
> >>>> Add the Falcon DB to the MAINTAINERS list
> >>>>
> >>>> Since memory is soldered on CPU module, memory node is on CPU module
> >>>> dtsi file.
> >>>>
> >>>> This Carrier board only utilizes the PCIe link, hence no special device
> >>>> or driver support is provided by this dtsi file.
> >>>> Devise a dts file for the combined com express carrier and CPU module.
> >>>>
> >>>> The Aramda 7020 CPU COM Express board offers the following features:
> >>>>
> >>>> 1.  Armada 7020 CPU, with dual ARM A72 cores
> >>>> 2.  DDR4 memory, 8GB, on board soldered
> >>>> 3.  1Gbit Out of Band Ethernet via RGMII to PHY and RJ45 connector,
> >>>>     all are present on A7K CPU module (none on the carrier)
> >>>> 4.  Optional 10G KR Ethernet going via the COM Express type 7 connector
> >>>> 5.  On-board 8 Gbit, 8-bit bus width NAND flash
> >>>> 6.  On-board 512 Mbit SPI flash
> >>>> 7.  PCIe Root Complex, 4 lanes PCIe gen3 connectivity, going via the
> >>>>     COM Express type 7 connector
> >>>> 8.  m.2 SATA connector
> >>>> 9.  Micro-SD card connector
> >>>> 10. USB 2.0 via COM Express type 7 connector
> >>>> 11. Two i2c interfaces - one to the CPU module, and one to the
> >>>>     carrier board via the COM Express type 7 connector
> >>>> 12. UART (mini USB connector by virtue of FT2232D UART to USB
> >>>>     converter, connected to the Armada 7020 UART0)
> >>>>
> >
> >  [...]

> >>
> >> Does this mean we should add 10gbase-kr to the phy-mode enum list in
> >> Documentation/devicetree/bindings/net/marvell?
> >
> > No, 10gbase-kr is legacy, one should use "10gbase-r" instead, cf commit
> >
> > e0f909bc3a24 ("net: switch to using PHY_INTERFACE_MODE_10GBASER rather than 10GKR")
> >
> > That should probably be documented in the bindings at some point :)
>
> Thanks, Maxime!
>
> Elad, I’ll amend the commit unless you have a strong reason against it.

You applied the .dts, but not the binding. So now more warnings...
Please apply the binding too.

Rob

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

* Re: [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
  2026-01-22 23:21   ` Rob Herring (Arm)
@ 2026-03-02 15:43   ` Gregory CLEMENT
  2026-03-06 23:46     ` Rob Herring
  2026-03-13 15:49   ` Gregory CLEMENT
  2 siblings, 1 reply; 15+ messages in thread
From: Gregory CLEMENT @ 2026-03-02 15:43 UTC (permalink / raw)
  To: Elad Nachman, robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	sebastian.hesselbarth, pali, mrkiko.rs, chris.packham, devicetree,
	linux-kernel, linux-arm-kernel
  Cc: enachman

Elad Nachman <enachman@marvell.com> writes:

> From: Elad Nachman <enachman@marvell.com>
>
> Add dt bindings for:
> Armada 7020 COM Express CPU module
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> with an Armada 7020 COM Express CPU module
>
> Signed-off-by: Elad Nachman <enachman@marvell.com>

Applied on mvebu/dt64

Thanks,

Gregory
> ---
>  .../devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> index 4bc7454a5d3a..7e77310da626 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> @@ -21,6 +21,17 @@ properties:
>            - const: marvell,armada-ap806-dual
>            - const: marvell,armada-ap806
>  
> +      - description:
> +          Falcon (DB-98CX85x0) Development board COM Express Carrier plus
> +          Armada 7020 SoC COM Express CPU module
> +        items:
> +          - const: marvell,armada7020-falcon-carrier
> +          - const: marvell,db-falcon-carrier
> +          - const: marvell,armada7020-cpu-module
> +          - const: marvell,armada7020
> +          - const: marvell,armada-ap806-dual
> +          - const: marvell,armada-ap806
> +
>        - description: Armada 7040 SoC
>          items:
>            - enum:
> -- 
> 2.25.1
>

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-03-02 15:43   ` Gregory CLEMENT
@ 2026-03-06 23:46     ` Rob Herring
  2026-03-13 15:49       ` Gregory CLEMENT
  0 siblings, 1 reply; 15+ messages in thread
From: Rob Herring @ 2026-03-06 23:46 UTC (permalink / raw)
  To: Gregory CLEMENT
  Cc: Elad Nachman, krzysztof.kozlowski+dt, conor+dt, andrew,
	sebastian.hesselbarth, pali, mrkiko.rs, chris.packham, devicetree,
	linux-kernel, linux-arm-kernel

On Mon, Mar 2, 2026 at 9:44 AM Gregory CLEMENT
<gregory.clement@bootlin.com> wrote:
>
> Elad Nachman <enachman@marvell.com> writes:
>
> > From: Elad Nachman <enachman@marvell.com>
> >
> > Add dt bindings for:
> > Armada 7020 COM Express CPU module
> > Falcon DB-98CX85x0 COM Express type 7 Carrier board
> > Falcon DB-98CX85x0 COM Express type 7 Carrier board
> > with an Armada 7020 COM Express CPU module
> >
> > Signed-off-by: Elad Nachman <enachman@marvell.com>
>
> Applied on mvebu/dt64

It seems the .dts files are in 7.0 and the binding is only in next.
The binding needs to go into 7.0 too.

Please pay attention to the warnings. There are only 19 (unique ones)
left for marvell since I fixed most of them.

Rob

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

* Re: [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-03-06 23:46     ` Rob Herring
@ 2026-03-13 15:49       ` Gregory CLEMENT
  0 siblings, 0 replies; 15+ messages in thread
From: Gregory CLEMENT @ 2026-03-13 15:49 UTC (permalink / raw)
  To: Rob Herring
  Cc: Elad Nachman, krzysztof.kozlowski+dt, conor+dt, andrew,
	sebastian.hesselbarth, pali, mrkiko.rs, chris.packham, devicetree,
	linux-kernel, linux-arm-kernel

Hello Rob,

> On Mon, Mar 2, 2026 at 9:44 AM Gregory CLEMENT
> <gregory.clement@bootlin.com> wrote:
>>
>> Elad Nachman <enachman@marvell.com> writes:
>>
>> > From: Elad Nachman <enachman@marvell.com>
>> >
>> > Add dt bindings for:
>> > Armada 7020 COM Express CPU module
>> > Falcon DB-98CX85x0 COM Express type 7 Carrier board
>> > Falcon DB-98CX85x0 COM Express type 7 Carrier board
>> > with an Armada 7020 COM Express CPU module
>> >
>> > Signed-off-by: Elad Nachman <enachman@marvell.com>
>>
>> Applied on mvebu/dt64
>
> It seems the .dts files are in 7.0 and the binding is only in next.
> The binding needs to go into 7.0 too.
>
> Please pay attention to the warnings. There are only 19 (unique ones)
> left for marvell since I fixed most of them.
>
Sorry for this. I have now applied the binding, and it will be included
in my next fixes PR.

Gregory

> Rob

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards
  2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
  2026-01-22 23:21   ` Rob Herring (Arm)
  2026-03-02 15:43   ` Gregory CLEMENT
@ 2026-03-13 15:49   ` Gregory CLEMENT
  2 siblings, 0 replies; 15+ messages in thread
From: Gregory CLEMENT @ 2026-03-13 15:49 UTC (permalink / raw)
  To: Elad Nachman, robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
	sebastian.hesselbarth, pali, mrkiko.rs, chris.packham, devicetree,
	linux-kernel, linux-arm-kernel
  Cc: enachman

Elad Nachman <enachman@marvell.com> writes:

> From: Elad Nachman <enachman@marvell.com>
>
> Add dt bindings for:
> Armada 7020 COM Express CPU module
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> Falcon DB-98CX85x0 COM Express type 7 Carrier board
> with an Armada 7020 COM Express CPU module
>
> Signed-off-by: Elad Nachman <enachman@marvell.com>

Applied on mvebu/fixes

Thanks,

Gregory
> ---
>  .../devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> index 4bc7454a5d3a..7e77310da626 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> @@ -21,6 +21,17 @@ properties:
>            - const: marvell,armada-ap806-dual
>            - const: marvell,armada-ap806
>  
> +      - description:
> +          Falcon (DB-98CX85x0) Development board COM Express Carrier plus
> +          Armada 7020 SoC COM Express CPU module
> +        items:
> +          - const: marvell,armada7020-falcon-carrier
> +          - const: marvell,db-falcon-carrier
> +          - const: marvell,armada7020-cpu-module
> +          - const: marvell,armada7020
> +          - const: marvell,armada-ap806-dual
> +          - const: marvell,armada-ap806
> +
>        - description: Armada 7040 SoC
>          items:
>            - enum:
> -- 
> 2.25.1
>

-- 
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

end of thread, other threads:[~2026-03-13 15:49 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-22 16:59 [PATCH 0/2] arm64: dts: a7k: add COM Express boards Elad Nachman
2026-01-22 16:59 ` [PATCH 1/3] dt-bindings: arm64: add Marvell 7k COMe boards Elad Nachman
2026-01-22 23:21   ` Rob Herring (Arm)
2026-03-02 15:43   ` Gregory CLEMENT
2026-03-06 23:46     ` Rob Herring
2026-03-13 15:49       ` Gregory CLEMENT
2026-03-13 15:49   ` Gregory CLEMENT
2026-01-22 16:59 ` [PATCH 2/3] arm64: dts: a7k: add COM Express boards Elad Nachman
2026-01-22 16:59 ` [PATCH 3/3] MAINTAINERS: Add Falcon DB Elad Nachman
2026-01-22 18:07 ` [PATCH 0/2] arm64: dts: a7k: add COM Express boards Rob Herring
2026-01-23  9:10   ` Gregory CLEMENT
2026-01-23  9:20     ` Maxime Chevallier
2026-01-23  9:27       ` Gregory CLEMENT
2026-01-27 16:23         ` Rob Herring
  -- strict thread matches above, loose matches on Subject: below --
2023-12-18 15:44 Elad Nachman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox