Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-renesas-soc@vger.kernel.org,
        linux-mediatek@lists.infradead.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, netdev@vger.kernel.org
Cc:     John Crispin <john@phrozen.org>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Marek Vasut <marex@denx.de>,
        Sean Wang <sean.wang@mediatek.com>,
        DENG Qingfang <dqfext@gmail.com>,
        Landen Chao <Landen.Chao@mediatek.com>,
        =?UTF-8?q?n=C3=A7=20=C3=9CNAL?= <arinc.unal@arinc9.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        =?UTF-8?q?Cl=C3=A9ment=20L=C3=A9ger?= <clement.leger@bootlin.com>,
        =?UTF-8?q?Alvin=20=C5=A0ipraga?= <alsi@bang-olufsen.dk>,
        Linus Walleij <linus.walleij@linaro.org>,
        UNGLinuxDriver@microchip.com,
        Woojung Huh <woojung.huh@microchip.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Kurt Kanzenbach <kurt@linutronix.de>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Vladimir Oltean <olteanv@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        George McCollister <george.mccollister@gmail.com>,
        Rob Herring <robh@kernel.org>
Subject: [PATCH v5 net-next 10/10] dt-bindings: net: mscc,vsc7514-switch: utilize generic ethernet-switch.yaml
Date:   Fri,  9 Dec 2022 19:30:33 -0800
Message-Id: <20221210033033.662553-11-colin.foster@in-advantage.com>
In-Reply-To: <20221210033033.662553-1-colin.foster@in-advantage.com>
References: <20221210033033.662553-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?GQAztZu1tE9JYxPv3QwjdnOjHZGyVET+2ItAUMHTOd1qD87I9O5/CpfrqBMp?=
 =?us-ascii?Q?L2J0dn3Nk8ahwDRVuNkNugQPEl2YHKB18uumDXEGMjukP8fNm33NlZ6gjn8n?=
 =?us-ascii?Q?tDTiYkVz8B73BamWHvxSFiFI6RFimGYqUnydnWExE9VRTsysIuW+izKm8iiP?=
 =?us-ascii?Q?eNKCTX0WDBLzKP5NKz8vPzY5Aie8KG9EMQH1N5RbqrHs+TATQmWv9rNVhUg4?=
 =?us-ascii?Q?gWHgAw2jz1gnH3rptDZxEUiQ0TOvQtR0UDh3HFCKoINriRcFKuXonXiuEumE?=
 =?us-ascii?Q?gTB1OfwGohX1xi10/B/AlHklR7cZeVyEqDwIaqvvO4RUGbCtyulAQdopp2FS?=
 =?us-ascii?Q?zOWXydR/sXwJ3ZeYiUPONKqRxEN15+QlG8VgkQ0A/6eztEJ3KcjW2zTiVpMW?=
 =?us-ascii?Q?r/U7IaAbgecrDWNVMfE1kqFDFxMRYcyyXwM/qMXeI2EtPXi/dPUuf2Newdva?=
 =?us-ascii?Q?gSFBNb+c0FcBi5JGM+MYaYiSOGsHgyi83Muvl72dCE+UyVwD8qHqH80F7f4Z?=
 =?us-ascii?Q?E9KbfLJXfpLImDji98dEcUSoAJvG9aDneA6nHLDn6HznMX24Uc2htnle5zM3?=
 =?us-ascii?Q?c3me2ivtNMgcv2Xiuk4G7zLf1fvOo7ykkoS76RJ9gIEapvpysolVI7BJYJ5L?=
 =?us-ascii?Q?JrdEeXnY2Id/Uj98R5VLcxQJV2UcyAgr5/WFDHtvJxBUQbQC+NaEFPmyId/Z?=
 =?us-ascii?Q?ifJK/eKxcCoqMu8/d9jOtwYzdTBMgeuPE/tBxJeRUiCddZ2asS28V+7r5nR4?=
 =?us-ascii?Q?RQI1q3g9E7/g20I+79ypp1Niq+I2NmP0DCQGuMUYDC+qvcZiGlvU1mu3vCGg?=
 =?us-ascii?Q?S3arVaxK3svAgPthm3rCwvP6wqKDjXS4An3EgJ9Gqej22mpKkqb6J2uRSBVQ?=
 =?us-ascii?Q?9XL6NVOwW9/bFQ2O88kba8B8DZwaNEgfNWj1ugqjoWgirfGyFEwZBAzTEY73?=
 =?us-ascii?Q?iCGG4cVsC2ETfwbmFrVGkI4gRZqdjLOdHX6j+lDOe0PaJmrw53XgHwWnZSoO?=
 =?us-ascii?Q?SPqctfQrjLIk+gjg352NIf9PxHwkfteT1K8YlVVMlbNuD4xomcK8XDKIRPVA?=
 =?us-ascii?Q?qiFgYO8s7Te0liJhhqW4bCuBRK76LN+86wGKsY529ZmOSWoELUM95JUtToQP?=
 =?us-ascii?Q?Wf5p/PYHzm36C/OcZvQGw68ZqgYqtSnK3dZuuzl1MvlEIUdqPlpOVqJebegN?=
 =?us-ascii?Q?1pyHqvwA6sB+C4Eq7Wc9L9A28ZfUxdxX2PlyBUfcAfvvll3eb2GV5tc2vSlg?=
 =?us-ascii?Q?+v56CkPjcUIrHrVrVR9BXkmCBqqPgMWCPiXacToxu0m2MOOd4YIunQzg9OBc?=
 =?us-ascii?Q?aQwXZPP5XVPAd2RfjZ4XqjwI9vLvCnJw0vtSLJevdBmCgEB4qzj2pl7c1ruA?=
 =?us-ascii?Q?2VSXunsp+cbFQKJmWhgzGTeyjjrh6nLGvzEHjLaV/5VSiiZk5ndVtHYNDeov?=
 =?us-ascii?Q?huWh+nfHvMqIxU/3mV8aLek9ThhEH0jGO+8fszu06uVy/pFMfcFng5xzmBQ/?=
 =?us-ascii?Q?BOzjYRAtlFV3x72vSjw55waiokxwnUtl2RFXUXWIWl6AZvycPbiuP/ieAcpD?=
 =?us-ascii?Q?c1Kd9RDpz3+CjxCpzgV4wq+o/jUDFfL9naYL4oDZEMJmDkKvY+t3bnNtYcUW?=
 =?us-ascii?Q?6SLRaC8Iln5z11XcK9qnNWM=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f4efedc1-b5c2-41bb-7b96-08dada5efbd1
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Dec 2022 03:31:11.4499
 (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: o0JwQw5kEtAlB3nLVohMtu7etW1hv+Qe/+OEVBTivh9x6HcVwg10nPomBEhyG4r4FZ4toopMgYU5ue3ImKeuZ8jjkfPxhFlfY86Cfddvhk8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4779
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Several bindings for ethernet switches are available for non-dsa switches
by way of ethernet-switch.yaml. Remove these duplicate entries and utilize
the common bindings for the VSC7514.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
Suggested-by: Vladimir Oltean <olteanv@gmail.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---

v4 -> v5
  * Add Rob Reviewed tag

v3 -> v4
  * Add Florian Reviewed tag

v2 -> v3:
  * Reference ethernet-switch-port.yaml# instead of ethernet-controller
  * Undo the addition of "unevaluatedProperties: true" from v2. Those
    were only added because of my misunderstandings.
  * Keep #address-cells and #size-cells in the ports node.

v1 -> v2:
  * Fix "$ref: ethernet-switch.yaml" placement. Oops.
  * Add "unevaluatedProperties: true" to ethernet-ports layer so it
    can correctly read into ethernet-switch.yaml
  * Add "unevaluatedProperties: true" to ethernet-port layer so it can
    correctly read into ethernet-controller.yaml

---
 .../bindings/net/mscc,vsc7514-switch.yaml     | 31 ++-----------------
 1 file changed, 3 insertions(+), 28 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml b/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
index ee0a504bdb24..5ffe831e59e4 100644
--- a/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
+++ b/Documentation/devicetree/bindings/net/mscc,vsc7514-switch.yaml
@@ -18,10 +18,9 @@ description: |
   packets using CPU. Additionally, PTP is supported as well as FDMA for faster
   packet extraction/injection.
 
-properties:
-  $nodename:
-    pattern: "^switch@[0-9a-f]+$"
+$ref: ethernet-switch.yaml#
 
+properties:
   compatible:
     const: mscc,vsc7514-switch
 
@@ -100,35 +99,11 @@ properties:
 
     patternProperties:
       "^port@[0-9a-f]+$":
-        type: object
-        description: Ethernet ports handled by the switch
 
-        $ref: ethernet-controller.yaml#
+        $ref: ethernet-switch-port.yaml#
 
         unevaluatedProperties: false
 
-        properties:
-          reg:
-            description: Switch port number
-
-          phy-handle: true
-
-          phy-mode: true
-
-          fixed-link: true
-
-          mac-address: true
-
-        required:
-          - reg
-          - phy-mode
-
-        oneOf:
-          - required:
-              - phy-handle
-          - required:
-              - fixed-link
-
 required:
   - compatible
   - reg
-- 
2.25.1