Re: [PATCH 16/20] dt-bindings: marvell: Declare the CN913x SoC compatibles
From: Rob Herring <robh@kernel.org>
Date: 2019-08-21 19:37:26
Also in:
linux-arm-kernel
On Tue, Aug 06, 2019 at 04:54:56PM +0200, Miquel Raynal wrote:
From: Grzegorz Jaszczyk <redacted> Describe the compatible properties for the new Marvell SoCs: * CN9130: 1x AP807-quad + 1x CP115 (1x embedded) * CN9131: 1x AP807-quad + 2x CP115 (1x embedded + 1x modular) * CN9132: 1x AP807-quad + 3x CP115 (1x embedded + 2x modular) CP115 are similar to CP110 in terms of features. There are three development boards based on these SoCs: * CN9130-DB: comes as a single mother board (with the CP115 bundled) * CN9131-DB: same as CN9130-DB with one additional modular CP115 * CN9132-DB: same as CN9130-DB with two additional modular CP115 Signed-off-by: Grzegorz Jaszczyk <redacted> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> --- .../bindings/arm/marvell/armada-7k-8k.txt | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-)
Please convert this to DT schema first.
quoted hunk ↗ jump to hunk
diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.txt b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.txt index df98a9c82a8c..8eb34ca4c4f0 100644 --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.txt +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.txt@@ -1,7 +1,7 @@ Marvell Armada 7K/8K Platforms Device Tree Bindings --------------------------------------------------- -Boards using a SoC of the Marvell Armada 7K or 8K families must carry +Boards using a SoC of the Marvell Armada 7K/8K or CN913x families must carry the following root node property: - compatible, with one of the following values:@@ -18,6 +18,17 @@ the following root node property: - "marvell,armada8040", "marvell,armada-ap806-quad", "marvell,armada-ap806" when the SoC being used is the Armada 8040 + - "marvell,cn9130", "marvell,armada-ap807-quad", "marvell,armada-ap807" + when the SoC being used is the Armada CN9130 with no external CP. + + - "marvell,cn9131", "marvell,cn9130", + "marvell,armada-ap807-quad", "marvell,armada-ap807" + when the SoC being used is the Armada CN9130 with one external CP. + + - "marvell,cn9132", "marvell,cn9131", "marvell,cn9130", + "marvell,armada-ap807-quad", "marvell,armada-ap807"
It's generally not all that useful to have all these compatibles.
+ when the SoC being used is the Armada CN9130 with two external CPs.
Is the number of external CPs not discoverable somewhere else in the DT?
+ Example: compatible = "marvell,armada7040-db", "marvell,armada7040", -- 2.20.1