* [PATCH v5 0/3] arm64: dts: cn913x: add COM Express boards
@ 2023-11-19 10:26 Elad Nachman
2023-11-19 10:26 ` [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files Elad Nachman
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Elad Nachman @ 2023-11-19 10:26 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 CN9130 and CN9131 COM Express Type 7 CPU
module boards by Marvell.
Add device tree bindings for this board.
Define these COM Express CPU modules as dtsi, and
provide a dtsi file for a carrier board (Marvell AC5X RD
COM Express type 7 carrier board).
This Carrier board only utilizes the PCIe link, hence no
special device / driver support is provided by this dtsi file.
Finally, add dts file for the combined carrier and CPU module.
v5:
1) List only carrier compatibility on carrier dtsi
2) Fix dt_bindings_check warnings using latest yamllint/dtschema
3) Fix subject lines to remove unnecessary wordings.
4) Remove dt bindings for standalone CPU modules
5) Move CN913x dt bindings to A7K dt bindings file
6) Fix dtbs_check warnings for dtb and bindings,
using latest yamllint/dtschema.
7) Move memory definition to main dts file, as memory
is socket based.
v4:
1) reorder patches - dt bindings before dts/dtsi files
2) correct description in dt bindings
3) separate dt bindings for CPU module, carrier and combination
4) make carrier board dts into dtsi, make dts for combination of
carrier and CPU module
5) correct compatibility strings and file names to use dashes
instead of underscores
v3:
1) Remove acronym which creates warnings for checkpatch.pl
2) Correct compatibility string for ac5x rd board
3) Add above compatibility string to dt bindings
4) update MAINTAINERS file with ac5 series dts files
5) remove memory property from carrier dts
6) add comment explaining that OOB RGMII ethernet port
connector and PHY are both on CPU module
v2:
1) add compatibility string for the board
2) remove unneeded hard-coded PHY LED blinking mode initialization
3) Split the CPU portion of the carrier board to
dtsi files, and define a dts file for the AC5X RD
carrier board.
Elad Nachman (3):
MAINTAINERS: add ac5 to list of maintained Marvell dts files
dt-bindings: arm64: add Marvell COM Express boards
arm64: dts: cn913x: add device trees for COM Express boards
.../bindings/arm/marvell/armada-7k-8k.yaml | 12 ++
MAINTAINERS | 1 +
arch/arm64/boot/dts/marvell/Makefile | 1 +
.../marvell/ac5x-rd-carrier-with-cn9131.dts | 25 ++++
.../boot/dts/marvell/ac5x-rd-carrier.dtsi | 14 +++
.../dts/marvell/cn9130-db-comexpress.dtsi | 96 ++++++++++++++++
.../dts/marvell/cn9131-db-comexpress.dtsi | 108 ++++++++++++++++++
7 files changed, 257 insertions(+)
create mode 100644 arch/arm64/boot/dts/marvell/ac5x-rd-carrier-with-cn9131.dts
create mode 100644 arch/arm64/boot/dts/marvell/ac5x-rd-carrier.dtsi
create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi
create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db-comexpress.dtsi
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files
2023-11-19 10:26 [PATCH v5 0/3] arm64: dts: cn913x: add COM Express boards Elad Nachman
@ 2023-11-19 10:26 ` Elad Nachman
2023-11-19 16:32 ` Andrew Lunn
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
2023-11-19 10:26 ` [PATCH v5 3/3] arm64: dts: cn913x: add device trees for " Elad Nachman
2 siblings, 1 reply; 9+ messages in thread
From: Elad Nachman @ 2023-11-19 10:26 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 ac5 dts files 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 973568cae9e5..81779c436fde 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2331,6 +2331,7 @@ F: arch/arm/boot/dts/marvell/armada*
F: arch/arm/boot/dts/marvell/kirkwood*
F: arch/arm/configs/mvebu_*_defconfig
F: arch/arm/mach-mvebu/
+F: arch/arm64/boot/dts/marvell/ac5*
F: arch/arm64/boot/dts/marvell/armada*
F: arch/arm64/boot/dts/marvell/cn913*
F: drivers/clk/mvebu/
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
2023-11-19 10:26 [PATCH v5 0/3] arm64: dts: cn913x: add COM Express boards Elad Nachman
2023-11-19 10:26 ` [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files Elad Nachman
@ 2023-11-19 10:26 ` Elad Nachman
2023-11-19 13:34 ` Conor Dooley
` (2 more replies)
2023-11-19 10:26 ` [PATCH v5 3/3] arm64: dts: cn913x: add device trees for " Elad Nachman
2 siblings, 3 replies; 9+ messages in thread
From: Elad Nachman @ 2023-11-19 10:26 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 dt bindings for:
CN9130 COM Express CPU module
CN9131 COM Express CPU module
AC5X RD COM Express Type 7 carrier board.
AC5X RD COM Express board with a CN9131 COM Express Type 7 CPU module.
Signed-off-by: Elad Nachman <enachman@marvell.com>
---
.../bindings/arm/marvell/armada-7k-8k.yaml | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
index 52d78521e412..6b19f23bcf63 100644
--- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
+++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
@@ -60,4 +60,16 @@ properties:
- const: marvell,armada-ap807-quad
- const: marvell,armada-ap807
+ - description:
+ Alleycat5X (98DX35xx) Reference Design as COM Express Carrier plus
+ Armada CN9131 COM Express CPU module
+ items:
+ - enum:
+ - marvell,ac5x-car-with-cn9131
+ - const: marvell,rd-ac5x-carrier
+ - const: marvell,cn9131-cpu-module
+ - const: marvell,cn9131
+ - const: marvell,armada-ap807-quad
+ - const: marvell,armada-ap807
+
additionalProperties: true
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v5 3/3] arm64: dts: cn913x: add device trees for COM Express boards
2023-11-19 10:26 [PATCH v5 0/3] arm64: dts: cn913x: add COM Express boards Elad Nachman
2023-11-19 10:26 ` [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files Elad Nachman
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
@ 2023-11-19 10:26 ` Elad Nachman
2 siblings, 0 replies; 9+ messages in thread
From: Elad Nachman @ 2023-11-19 10:26 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 CN9130 and CN9131 COM Express Type 7 CPU
module boards by Marvell.
Define these COM Express CPU modules as dtsi and
provide a dtsi file for a carrier board (Marvell AC5X RD
COM Express type 7 carrier board).
This Carrier board only utilizes the PCIe link, hence no
special device / driver support is provided by this dtsi file.
Finally, provide a dts file for the com express carrier and
CPU module combination.
These COM Express boards differ from the existing CN913x DB
boards by the type of ethernet connection (RGMII),
the type of voltage regulators (not i2c expander based)
and the USB phy (not UTMI based).
Note - PHY + RGMII connector is OOB on CPU module.
CN9131 COM Express board is basically CN9130 COM Express board
with an additional CP115 I/O co-processor, which in this case
provides an additional USB host controller on the board.
Signed-off-by: Elad Nachman <enachman@marvell.com>
---
arch/arm64/boot/dts/marvell/Makefile | 1 +
.../marvell/ac5x-rd-carrier-with-cn9131.dts | 25 ++++
.../boot/dts/marvell/ac5x-rd-carrier.dtsi | 14 +++
.../dts/marvell/cn9130-db-comexpress.dtsi | 96 ++++++++++++++++
.../dts/marvell/cn9131-db-comexpress.dtsi | 108 ++++++++++++++++++
5 files changed, 244 insertions(+)
create mode 100644 arch/arm64/boot/dts/marvell/ac5x-rd-carrier-with-cn9131.dts
create mode 100644 arch/arm64/boot/dts/marvell/ac5x-rd-carrier.dtsi
create mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi
create mode 100644 arch/arm64/boot/dts/marvell/cn9131-db-comexpress.dtsi
diff --git a/arch/arm64/boot/dts/marvell/Makefile b/arch/arm64/boot/dts/marvell/Makefile
index 79ac09b58a89..88c0f357a778 100644
--- a/arch/arm64/boot/dts/marvell/Makefile
+++ b/arch/arm64/boot/dts/marvell/Makefile
@@ -26,4 +26,5 @@ dtb-$(CONFIG_ARCH_MVEBU) += cn9132-db.dtb
dtb-$(CONFIG_ARCH_MVEBU) += cn9132-db-B.dtb
dtb-$(CONFIG_ARCH_MVEBU) += cn9130-crb-A.dtb
dtb-$(CONFIG_ARCH_MVEBU) += cn9130-crb-B.dtb
+dtb-$(CONFIG_ARCH_MVEBU) += ac5x-rd-carrier-with-cn9131.dtb
dtb-$(CONFIG_ARCH_MVEBU) += ac5-98dx35xx-rd.dtb
diff --git a/arch/arm64/boot/dts/marvell/ac5x-rd-carrier-with-cn9131.dts b/arch/arm64/boot/dts/marvell/ac5x-rd-carrier-with-cn9131.dts
new file mode 100644
index 000000000000..9ca8af11ac87
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/ac5x-rd-carrier-with-cn9131.dts
@@ -0,0 +1,25 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the AC5X RD Type 7 Com Express carrier board,
+ * Utilizing the CN913x COM Express CPU module board.
+ * This specific board only maintains a PCIe link with the CPU CPU module
+ * module, which does not require any special DTS definitions.
+ */
+
+#include "cn9131-db-comexpress.dtsi"
+#include "ac5x-rd-carrier.dtsi"
+
+/ {
+ model = "Marvell Armada AC5X RD COM EXPRESS type 7 carrier board with CN9131 CPU module";
+ compatible = "marvell,ac5x-car-with-cn9131", "marvell,rd-ac5x-carrier",
+ "marvell,cn9131-cpu-module", "marvell,cn9131",
+ "marvell,armada-ap807-quad", "marvell,armada-ap807";
+
+ memory@0 {
+ device_type = "memory";
+ reg = <0x0 0x0 0x2 0x00000000>;
+ };
+
+};
diff --git a/arch/arm64/boot/dts/marvell/ac5x-rd-carrier.dtsi b/arch/arm64/boot/dts/marvell/ac5x-rd-carrier.dtsi
new file mode 100644
index 000000000000..fd45d5582233
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/ac5x-rd-carrier.dtsi
@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the AC5X RD Type 7 Com Express carrier board,
+ * This specific board only maintains a PCIe link with the CPU CPU module
+ * module, which does not require any special DTS definitions.
+ */
+
+/ {
+ model = "Marvell Armada AC5X RD COM EXPRESS type 7 carrier board";
+ compatible = "marvell,rd-ac5x-carrier";
+
+};
diff --git a/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi b/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi
new file mode 100644
index 000000000000..028496ebc473
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi
@@ -0,0 +1,96 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the CN9130-DB Com Express CPU module board.
+ */
+
+#include "cn9130-db.dtsi"
+
+/ {
+ model = "Marvell Armada CN9130-DB COM EXPRESS type 7 CPU module board";
+ compatible = "marvell,cn9130-cpu-module", "marvell,cn9130",
+ "marvell,armada-ap807-quad", "marvell,armada-ap807";
+
+};
+
+&ap0_reg_sd_vccq {
+ regulator-max-microvolt = <1800000>;
+ states = <1800000 0x1 1800000 0x0>;
+ /delete-property/ gpios;
+};
+
+&cp0_reg_usb3_vbus0 {
+ /delete-property/ gpio;
+};
+
+&cp0_reg_usb3_vbus1 {
+ /delete-property/ gpio;
+};
+
+&cp0_reg_sd_vcc {
+ status = "disabled";
+};
+
+&cp0_reg_sd_vccq {
+ status = "disabled";
+};
+
+&cp0_sdhci0 {
+ status = "disabled";
+};
+
+&cp0_eth0 {
+ status = "disabled";
+};
+
+&cp0_eth1 {
+ status = "okay";
+ phy = <&phy0>;
+ phy-mode = "rgmii-id";
+};
+
+&cp0_eth2 {
+ status = "disabled";
+};
+
+&cp0_mdio {
+ status = "okay";
+ pinctrl-0 = <&cp0_ge_mdio_pins>;
+ phy0: ethernet-phy@0 {
+ status = "okay";
+ };
+};
+
+&cp0_syscon0 {
+ cp0_pinctrl: pinctrl {
+ compatible = "marvell,cp115-standalone-pinctrl";
+
+ cp0_ge_mdio_pins: ge-mdio-pins {
+ marvell,pins = "mpp40", "mpp41";
+ marvell,function = "ge";
+ };
+ };
+};
+
+&cp0_sdhci0 {
+ status = "disabled";
+};
+
+&cp0_spi1 {
+ status = "okay";
+};
+
+&cp0_usb3_0 {
+ status = "okay";
+ usb-phy = <&cp0_usb3_0_phy0>;
+ phy-names = "usb";
+ /delete-property/ phys;
+};
+
+&cp0_usb3_1 {
+ status = "okay";
+ usb-phy = <&cp0_usb3_0_phy1>;
+ phy-names = "usb";
+ /delete-property/ phys;
+};
diff --git a/arch/arm64/boot/dts/marvell/cn9131-db-comexpress.dtsi b/arch/arm64/boot/dts/marvell/cn9131-db-comexpress.dtsi
new file mode 100644
index 000000000000..6f3914bcfd01
--- /dev/null
+++ b/arch/arm64/boot/dts/marvell/cn9131-db-comexpress.dtsi
@@ -0,0 +1,108 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (C) 2023 Marvell International Ltd.
+ *
+ * Device tree for the CN9131-DB Com Express CPU module board.
+ */
+
+#include "cn9131-db.dtsi"
+
+/ {
+ model = "Marvell Armada CN9131-DB COM EXPRESS type 7 CPU module board";
+ compatible = "marvell,cn9131-cpu-module", "marvell,cn9131", "marvell,cn9130",
+ "marvell,armada-ap807-quad", "marvell,armada-ap807";
+
+};
+
+&ap0_reg_sd_vccq {
+ regulator-max-microvolt = <1800000>;
+ states = <1800000 0x1 1800000 0x0>;
+ /delete-property/ gpios;
+};
+
+&cp0_reg_usb3_vbus0 {
+ /delete-property/ gpio;
+};
+
+&cp0_reg_usb3_vbus1 {
+ /delete-property/ gpio;
+};
+
+&cp1_reg_usb3_vbus0 {
+ /delete-property/ gpio;
+};
+
+&cp0_reg_sd_vcc {
+ status = "disabled";
+};
+
+&cp0_reg_sd_vccq {
+ status = "disabled";
+};
+
+&cp0_sdhci0 {
+ status = "disabled";
+};
+
+&cp0_eth0 {
+ status = "disabled";
+};
+
+&cp0_eth1 {
+ status = "okay";
+ phy = <&phy0>;
+ phy-mode = "rgmii-id";
+};
+
+&cp0_eth2 {
+ status = "disabled";
+};
+
+&cp0_mdio {
+ status = "okay";
+ pinctrl-0 = <&cp0_ge_mdio_pins>;
+ phy0: ethernet-phy@0 {
+ status = "okay";
+ };
+};
+
+&cp0_syscon0 {
+ cp0_pinctrl: pinctrl {
+ compatible = "marvell,cp115-standalone-pinctrl";
+
+ cp0_ge_mdio_pins: ge-mdio-pins {
+ marvell,pins = "mpp40", "mpp41";
+ marvell,function = "ge";
+ };
+ };
+};
+
+&cp0_sdhci0 {
+ status = "disabled";
+};
+
+&cp0_spi1 {
+ status = "okay";
+};
+
+&cp0_usb3_0 {
+ status = "okay";
+ usb-phy = <&cp0_usb3_0_phy0>;
+ phy-names = "usb";
+ /delete-property/ phys;
+};
+
+&cp0_usb3_1 {
+ status = "okay";
+ usb-phy = <&cp0_usb3_0_phy1>;
+ phy-names = "usb";
+ /delete-property/ phys;
+};
+
+&cp1_usb3_1 {
+ status = "okay";
+ usb-phy = <&cp1_usb3_0_phy0>;
+ /* Generic PHY, providing serdes lanes */
+ phys = <&cp1_comphy3 1>;
+ phy-names = "usb";
+};
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
@ 2023-11-19 13:34 ` Conor Dooley
2023-11-19 16:30 ` Andrew Lunn
2023-11-19 13:43 ` Rob Herring
2023-11-20 9:08 ` kernel test robot
2 siblings, 1 reply; 9+ messages in thread
From: Conor Dooley @ 2023-11-19 13:34 UTC (permalink / raw)
To: Elad Nachman
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
chris.packham, devicetree, linux-kernel, linux-arm-kernel, cyuval
[-- Attachment #1: Type: text/plain, Size: 1688 bytes --]
On Sun, Nov 19, 2023 at 12:26:37PM +0200, Elad Nachman wrote:
> From: Elad Nachman <enachman@marvell.com>
>
> Add dt bindings for:
> CN9130 COM Express CPU module
Did you forget this one? There's only the CN9191 system here.
> CN9131 COM Express CPU module
> AC5X RD COM Express Type 7 carrier board.
> AC5X RD COM Express board with a CN9131 COM Express Type 7 CPU module.
>
> Signed-off-by: Elad Nachman <enachman@marvell.com>
> ---
> .../bindings/arm/marvell/armada-7k-8k.yaml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> index 52d78521e412..6b19f23bcf63 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> @@ -60,4 +60,16 @@ properties:
> - const: marvell,armada-ap807-quad
> - const: marvell,armada-ap807
>
> + - description:
> + Alleycat5X (98DX35xx) Reference Design as COM Express Carrier plus
> + Armada CN9131 COM Express CPU module
> + items:
> + - enum:
> + - marvell,ac5x-car-with-cn9131
Why not just marvell,cn9131-ac5x-carrier or similar? The "with" isn't
particularly helpful.
Cheers,
Conor.
> + - const: marvell,rd-ac5x-carrier
> + - const: marvell,cn9131-cpu-module
> + - const: marvell,cn9131
> + - const: marvell,armada-ap807-quad
> + - const: marvell,armada-ap807
> +
> additionalProperties: true
> --
> 2.25.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
2023-11-19 13:34 ` Conor Dooley
@ 2023-11-19 13:43 ` Rob Herring
2023-11-20 9:08 ` kernel test robot
2 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2023-11-19 13:43 UTC (permalink / raw)
To: Elad Nachman
Cc: conor+dt, gregory.clement, devicetree, andrew,
krzysztof.kozlowski+dt, chris.packham, cyuval, pali,
sebastian.hesselbarth, linux-arm-kernel, linux-kernel, robh+dt,
mrkiko.rs
On Sun, 19 Nov 2023 12:26:37 +0200, Elad Nachman wrote:
> From: Elad Nachman <enachman@marvell.com>
>
> Add dt bindings for:
> CN9130 COM Express CPU module
> CN9131 COM Express CPU module
> AC5X RD COM Express Type 7 carrier board.
> AC5X RD COM Express board with a CN9131 COM Express Type 7 CPU module.
>
> Signed-off-by: Elad Nachman <enachman@marvell.com>
> ---
> .../bindings/arm/marvell/armada-7k-8k.yaml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
./Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml:64:13: [warning] wrong indentation: expected 10 but found 12 (indentation)
dtschema/dtc warnings/errors:
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20231119102638.1041978-3-enachman@marvell.com
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
2023-11-19 13:34 ` Conor Dooley
@ 2023-11-19 16:30 ` Andrew Lunn
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Lunn @ 2023-11-19 16:30 UTC (permalink / raw)
To: Conor Dooley
Cc: Elad Nachman, robh+dt, krzysztof.kozlowski+dt, conor+dt,
gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
chris.packham, devicetree, linux-kernel, linux-arm-kernel, cyuval
> > + - description:
> > + Alleycat5X (98DX35xx) Reference Design as COM Express Carrier plus
> > + Armada CN9131 COM Express CPU module
> > + items:
> > + - enum:
> > + - marvell,ac5x-car-with-cn9131
>
> Why not just marvell,cn9131-ac5x-carrier or similar? The "with" isn't
> particularly helpful.
As the description says, this is the combination of the SOM plus the
carrier to give a complete RDK.
Elad, do the RDK as a whole have a name? You can use that here for the
compatible.
Andrew
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files
2023-11-19 10:26 ` [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files Elad Nachman
@ 2023-11-19 16:32 ` Andrew Lunn
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Lunn @ 2023-11-19 16:32 UTC (permalink / raw)
To: Elad Nachman
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, gregory.clement,
sebastian.hesselbarth, pali, mrkiko.rs, chris.packham, devicetree,
linux-kernel, linux-arm-kernel, cyuval
On Sun, Nov 19, 2023 at 12:26:36PM +0200, Elad Nachman wrote:
> From: Elad Nachman <enachman@marvell.com>
>
> Add ac5 dts files to the list of maintained Marvell Armada dts files
>
> Signed-off-by: Elad Nachman <enachman@marvell.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Andrew
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
2023-11-19 13:34 ` Conor Dooley
2023-11-19 13:43 ` Rob Herring
@ 2023-11-20 9:08 ` kernel test robot
2 siblings, 0 replies; 9+ messages in thread
From: kernel test robot @ 2023-11-20 9:08 UTC (permalink / raw)
To: Elad Nachman, robh+dt, krzysztof.kozlowski+dt, conor+dt, andrew,
gregory.clement, sebastian.hesselbarth, pali, mrkiko.rs,
chris.packham, devicetree, linux-kernel, linux-arm-kernel
Cc: oe-kbuild-all, enachman, cyuval
Hi Elad,
kernel test robot noticed the following build warnings:
[auto build test WARNING on robh/for-next]
[also build test WARNING on linus/master v6.7-rc2 next-20231120]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Elad-Nachman/MAINTAINERS-add-ac5-to-list-of-maintained-Marvell-dts-files/20231119-182821
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link: https://lore.kernel.org/r/20231119102638.1041978-3-enachman%40marvell.com
patch subject: [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards
compiler: loongarch64-linux-gcc (GCC) 13.2.0
reproduce: (https://download.01.org/0day-ci/archive/20231120/202311201610.9d8EdO2Y-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311201610.9d8EdO2Y-lkp@intel.com/
dtcheck warnings: (new ones prefixed by >>)
>> Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml:64:13: [warning] wrong indentation: expected 10 but found 12 (indentation)
vim +64 Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
8
9 maintainers:
10 - Gregory CLEMENT <gregory.clement@bootlin.com>
11
12 properties:
13 $nodename:
14 const: '/'
15 compatible:
16 oneOf:
17
18 - description: Armada 7020 SoC
19 items:
20 - const: marvell,armada7020
21 - const: marvell,armada-ap806-dual
22 - const: marvell,armada-ap806
23
24 - description: Armada 7040 SoC
25 items:
26 - const: marvell,armada7040
27 - const: marvell,armada-ap806-quad
28 - const: marvell,armada-ap806
29
30 - description: Armada 8020 SoC
31 items:
32 - const: marvell,armada8020
33 - const: marvell,armada-ap806-dual
34 - const: marvell,armada-ap806
35
36 - description: Armada 8040 SoC
37 items:
38 - const: marvell,armada8040
39 - const: marvell,armada-ap806-quad
40 - const: marvell,armada-ap806
41
42 - description: Armada CN9130 SoC with no external CP
43 items:
44 - const: marvell,cn9130
45 - const: marvell,armada-ap807-quad
46 - const: marvell,armada-ap807
47
48 - description: Armada CN9131 SoC with one external CP
49 items:
50 - const: marvell,cn9131
51 - const: marvell,cn9130
52 - const: marvell,armada-ap807-quad
53 - const: marvell,armada-ap807
54
55 - description: Armada CN9132 SoC with two external CPs
56 items:
57 - const: marvell,cn9132
58 - const: marvell,cn9131
59 - const: marvell,cn9130
60 - const: marvell,armada-ap807-quad
61 - const: marvell,armada-ap807
62
63 - description:
> 64 Alleycat5X (98DX35xx) Reference Design as COM Express Carrier plus
65 Armada CN9131 COM Express CPU module
66 items:
67 - enum:
68 - marvell,ac5x-car-with-cn9131
69 - const: marvell,rd-ac5x-carrier
70 - const: marvell,cn9131-cpu-module
71 - const: marvell,cn9131
72 - const: marvell,armada-ap807-quad
73 - const: marvell,armada-ap807
74
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2023-11-20 9:09 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-19 10:26 [PATCH v5 0/3] arm64: dts: cn913x: add COM Express boards Elad Nachman
2023-11-19 10:26 ` [PATCH v5 1/3] MAINTAINERS: add ac5 to list of maintained Marvell dts files Elad Nachman
2023-11-19 16:32 ` Andrew Lunn
2023-11-19 10:26 ` [PATCH v5 2/3] dt-bindings: arm64: add Marvell COM Express boards Elad Nachman
2023-11-19 13:34 ` Conor Dooley
2023-11-19 16:30 ` Andrew Lunn
2023-11-19 13:43 ` Rob Herring
2023-11-20 9:08 ` kernel test robot
2023-11-19 10:26 ` [PATCH v5 3/3] arm64: dts: cn913x: add device trees for " Elad Nachman
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).