From:   Colin Foster <colin.foster@in-advantage.com>
To:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org
Cc:     Russell King <linux@armlinux.org.uk>,
        Richard Cochran <richardcochran@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>, UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>, Lee Jones <lee@kernel.org>
Subject: [PATCH v5 net-next 10/13] dt-bindings: net: mscc,vsc7514-switch: add dsa binding for the vsc7512
Date:   Fri, 27 Jan 2023 11:35:56 -0800
Message-Id: <20230127193559.1001051-11-colin.foster@in-advantage.com>
In-Reply-To: <20230127193559.1001051-1-colin.foster@in-advantage.com>
References: <20230127193559.1001051-1-colin.foster@in-advantage.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?5fdmlUz3MnLqyvQ9gbY+GAFWzIOxYLIAtXvc7SeuQV18vXzuhhscpoOZdYya?=
 =?us-ascii?Q?++6cmvIOLLiWqtgf25W4xO27AtvIGrIMWFAYC6mqXAYGQ4iX7dpMpnJXq2eH?=
 =?us-ascii?Q?hu5/a+NTe4NwDukPp/8HK0yoTC3+kOF6DYftKz021WOIsY4L+ckDgzvsk9VY?=
 =?us-ascii?Q?UoSdl2+LKTybHGrHtsWZnbWl0F/bncBIzTvFtalg8kxTIv3ocwXP4LbfaQvA?=
 =?us-ascii?Q?FNBgpzVHJfr4TkMeMuRWs0HFMMFdBHfiNhKnhM2Fil7PJnBsfgXpWxqqXxod?=
 =?us-ascii?Q?H+FqAoZsjq9kc5yRiHAjtlYXuSBlDqAQqUwaDEFZl79NjUC//baK2ZxwSXZO?=
 =?us-ascii?Q?7hj/ye54IPDoZpGw65Nv/T2PVD2MphAinFk/DrmTH7gpvbnMErHMzwaazC/Y?=
 =?us-ascii?Q?bISkqYGZTs6G2Dkp/yzDyHTuFJZlhSmftVUPrqaFFhJpf40C+4JzcSfWieK+?=
 =?us-ascii?Q?rO0lKo7XOAE5IVDH2mu24b4q4guSnWSSyTpSTAw3syyYwIJHbckxiPsNQOei?=
 =?us-ascii?Q?5NjtOcp1MSIyXtYBKEAJ0+fZzTdQfZVC6215KWcmx6KiBw+z9QaIw6Ri83/K?=
 =?us-ascii?Q?L89n20wZ9qT5DGf1gKJyfpdvTTPb6mAiX576ZRa4zViE86PJ8Nw1SZUE2ORc?=
 =?us-ascii?Q?/nfNeB6holGXBNsovz8Zs8pbOBvienWEeRMYUr7IpvDnRtx3mN1NXuMmRVml?=
 =?us-ascii?Q?C27E8VjDqE+2kb3YtC6qP7rgb2RbaQCAobLV6zT17uarV0CTrFa3hwNrErrS?=
 =?us-ascii?Q?7hT2G9gFN7u0h+3cVfqaWp90Lk22aabPumm1hTurtNMOGdDOtYnL10SHLw1d?=
 =?us-ascii?Q?SVWY993EI9EHxli2DpleNm1UL7eiy7E6NH14dZeV6buf7d37+d3oPaOxjj22?=
 =?us-ascii?Q?R3clqoPFFYgc5vmT4n6E1Dq4gGjjusgB4Hpy1ZuT+9XmdWPfyI1YmC5XGf2A?=
 =?us-ascii?Q?86VgSvPkkhmgI3MkFABmd/mO4rODQ7RjFm+kq89fKzhge9hWrIHkHHzxblAX?=
 =?us-ascii?Q?Fd/aH8j6jhA8MyhsdMM2IrrQkpetxL0g60Se/Ov4x5OJNYa/QQWwI+fznAZ9?=
 =?us-ascii?Q?2wdUVOMf2FPHi85wq7kPHpV5OoB7Ju35gawEAyHcWnetMjcpltMDnLMxQz2k?=
 =?us-ascii?Q?2bTPEsGHT2LIlmWx88INNIED3eiwWbBhLNDhfp6fgDNJm9yl5wL0rLCsTdqY?=
 =?us-ascii?Q?zEFABR9oaTlPV0T0L0bNtrQBa7L8Aa7+y6AAS0cq1pKq+41R3nJb1n4u9Sz3?=
 =?us-ascii?Q?REQiCPEp2I5cHBiTYcFOVvPwc4GHBVNJmoX++r4Weni1zzcRII3/38ZKIsOB?=
 =?us-ascii?Q?hc3S9xDzYOfCxBaA7HLPBSV6150IpGo2ziH0+0gGxKHbCcWxjQ/vqUKbtbBD?=
 =?us-ascii?Q?AIEOc0MxEb6UfNRzoaF4sSc7m33w6qSOeXADergEaeiSxyK4DiXKX75e68yL?=
 =?us-ascii?Q?9Edt7BHmmVQpBMVrEj91hRlqX8b1FGIXPtSUQJsEnQ5PcF5M3MCO+a/wbcVd?=
 =?us-ascii?Q?Ok/SoBGbVs4BvBVzvVTP4hSSN7Cdlhaub3gkgP6pRBBVgcfz4uJE+fAoZvqJ?=
 =?us-ascii?Q?jVtLH11EM2kfZBhO5SZ8Nu8qRw3q9HIH+FOL4MeKinRuO6ybLsIU8lbnUq3y?=
 =?us-ascii?Q?iJi1OkpL+O9rukw9GE36L5M=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f9bb4e86-2fc0-4630-32d4-08db009dc8ce
X-MS-Exchange-CrossTenant-AuthSource: DM5PR1001MB2345.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jan 2023 19:36:28.5941
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: P5k/GgYaa4QH9kQAho1+YK1aKV5eV2ejYkYE4ciGpLRqcChW8ngLyCVcmBSgpxyscCCAnvdPPcaV1GNeAZT7hd/BfQtkjJXlH2EIxzYUh4M=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4636
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The VSC7511, VSC7512, VSC7513 and VSC7514 all have the ability to be
controlled either internally by a memory-mapped CPU, or externally via
interfaces like SPI and PCIe. The internal CPU of the VSC7511 and 7512
don't have the resources to run Linux, so must be controlled via these
external interfaces in a DSA configuration.

Add mscc,vsc7512-switch compatible string to indicate that the chips are
being controlled externally in a DSA configuration.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
---

v5
    * New patch after a documentation overhaul series

---
 .../bindings/net/mscc,vsc7514-switch.yaml     | 113 ++++++++++++++----
 1 file changed, 90 insertions(+), 23 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml b/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
index 5ffe831e59e4..8ee2c7d7ff42 100644
--- a/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
+++ b/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
@@ -18,13 +18,52 @@ description: |
   packets using CPU. Additionally, PTP is supported as well as FDMA for faster
   packet extraction/injection.
 
-$ref: ethernet-switch.yaml#
+allOf:
+  - if:
+      properties:
+        compatible:
+          const: mscc,vsc7514-switch
+    then:
+      $ref: ethernet-switch.yaml#
+      required:
+        - interrupts
+        - interrupt-names
+      properties:
+        reg:
+          minItems: 21
+        reg-names:
+          minItems: 21
+        ethernet-ports:
+          patternProperties:
+            "^port@[0-9a-f]+$":
+              $ref: ethernet-switch-port.yaml#
+              unevaluatedProperties: false
+
+  - if:
+      properties:
+        compatible:
+          const: mscc,vsc7512-switch
+    then:
+      $ref: /schemas/net/dsa/dsa.yaml#
+      properties:
+        reg:
+          maxItems: 20
+        reg-names:
+          maxItems: 20
+        ethernet-ports:
+          patternProperties:
+            "^port@[0-9a-f]+$":
+              $ref: /schemas/net/dsa/dsa-port.yaml#
+              unevaluatedProperties: false
 
 properties:
   compatible:
-    const: mscc,vsc7514-switch
+    enum:
+      - mscc,vsc7512-switch
+      - mscc,vsc7514-switch
 
   reg:
+    minItems: 20
     items:
       - description: system target
       - description: rewriter target
@@ -49,6 +88,7 @@ properties:
       - description: fdma target
 
   reg-names:
+    minItems: 20
     items:
       - const: sys
       - const: rew
@@ -86,35 +126,16 @@ properties:
       - const: xtr
       - const: fdma
 
-  ethernet-ports:
-    type: object
-
-    properties:
-      '#address-cells':
-        const: 1
-      '#size-cells':
-        const: 0
-
-    additionalProperties: false
-
-    patternProperties:
-      "^port@[0-9a-f]+$":
-
-        $ref: ethernet-switch-port.yaml#
-
-        unevaluatedProperties: false
-
 required:
   - compatible
   - reg
   - reg-names
-  - interrupts
-  - interrupt-names
   - ethernet-ports
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
+  # VSC7514 (Switchdev)
   - |
     switch@1010000 {
       compatible = "mscc,vsc7514-switch";
@@ -162,5 +183,51 @@ examples:
         };
       };
     };
+  # VSC7512 (DSA)
+  - |
+    ethernet-switch@1{
+      compatible = "mscc,vsc7512-switch";
+      reg = <0x71010000 0x10000>,
+            <0x71030000 0x10000>,
+            <0x71080000 0x100>,
+            <0x710e0000 0x10000>,
+            <0x711e0000 0x100>,
+            <0x711f0000 0x100>,
+            <0x71200000 0x100>,
+            <0x71210000 0x100>,
+            <0x71220000 0x100>,
+            <0x71230000 0x100>,
+            <0x71240000 0x100>,
+            <0x71250000 0x100>,
+            <0x71260000 0x100>,
+            <0x71270000 0x100>,
+            <0x71280000 0x100>,
+            <0x71800000 0x80000>,
+            <0x71880000 0x10000>,
+            <0x71040000 0x10000>,
+            <0x71050000 0x10000>,
+            <0x71060000 0x10000>;
+            reg-names = "sys", "rew", "qs", "ptp", "port0", "port1",
+            "port2", "port3", "port4", "port5", "port6",
+            "port7", "port8", "port9", "port10", "qsys",
+            "ana", "s0", "s1", "s2";
+
+            ethernet-ports {
+            #address-cells = <1>;
+            #size-cells = <0>;
+
+           port@0 {
+            reg = <0>;
+            ethernet = <&mac_sw>;
+            phy-handle = <&phy0>;
+            phy-mode = "internal";
+          };
+          port@1 {
+            reg = <1>;
+            phy-handle = <&phy1>;
+            phy-mode = "internal";
+          };
+        };
+      };
 
 ...
-- 
2.25.1