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>
Subject: [PATCH v3 net-next 10/10] dt-bindings: net: mscc,vsc7514-switch: utilize generic ethernet-switch.yaml
Date:   Sun, 27 Nov 2022 14:47:34 -0800
Message-Id: <20221127224734.885526-11-colin.foster@in-advantage.com>
In-Reply-To: <20221127224734.885526-1-colin.foster@in-advantage.com>
References: <20221127224734.885526-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?ccBOsnLOby3ojQT/UKtb19nOfC+7CqDhR5dOuacVM+c8m8T2tT/sVSIYRcUJ?=
 =?us-ascii?Q?ZQWvRYW6nHzzFpQ4CD0J4V+rfJrqkaaExYChph1IYu0nDxD84jEmRb16L5tX?=
 =?us-ascii?Q?A6NFz1DMbqRtmf16U2C93y6eJC3nzLcDUr8pWXYB532lmks0jKtTTGOooaPN?=
 =?us-ascii?Q?YGi2McrgHYN2yScmsIJGHNiK+KPBlCmsVsCKAQzDNZ7/S04AIoCg4PLXkFxt?=
 =?us-ascii?Q?vuX8Hepsu2+keUQwg2bAocvWatv9e36aZAOCMJyiT330/TbOygno0sTLBd8V?=
 =?us-ascii?Q?716aGWrs1Zgg2OQ0sJIfm6oUAKzSSvR7UI5NGlpf5N1fkBckH9Eciui2lKEM?=
 =?us-ascii?Q?ePgxsRTwL54SSTKOhc9Z4isIvgrcM5J24s049D6DGC9w+qA8I8io0VTvbQB9?=
 =?us-ascii?Q?rT1FSpQF53gxVGXC0GhUz736H37VUKNDBtZQiNnY0w/ZlracU7UufmGGsgo/?=
 =?us-ascii?Q?dwO5t0p/O2tCC/pTVz+/MMGCJfiY4n/1s4DobDwVIfM0KdsxpUjfZ7TQtnSf?=
 =?us-ascii?Q?Di/gLBBMOTi0A9i+56BjnC5nU4nojJUTyAxxkfNNUBpjw39S45JN9yTXboaB?=
 =?us-ascii?Q?f8YTWRzv9Y+hf0Zm1lEB1AQh2+NG/7GPleyDj5oSa1VExZCEfkvBqFI7ROBJ?=
 =?us-ascii?Q?Mj+kOzOGSJjI2q26rtAlkv5nD6Warx0+ybGGKld0NeweP4f/2cUBXU0oRAg3?=
 =?us-ascii?Q?ry7zMaD6Lnmw6EGlkbfUZe1l1dexrgmrlRh+qXvyLda13/hRCWrDLM1n5u//?=
 =?us-ascii?Q?Wh4KueVhnKw+QY8syRX1FcNWqnRGfbn6Myb+OBj+XWQgVOggcbll2tf5hnoC?=
 =?us-ascii?Q?sT44FSy4W6IvtLGMR6e3TfK8Plho/i5LrKy6WaC1P08dDzZSuWX2sd1xbS+r?=
 =?us-ascii?Q?IjnKhXScAyLEWMJoXeJWngJUpEMu2+dzh7aD4L6r6lOnqglEmlSxtwRCtDq5?=
 =?us-ascii?Q?+mrRnWvpEx49kcY7tcZ7KyUi72YSGri3X8gioFnOEBZ/Hu6WJali4fgKhBSt?=
 =?us-ascii?Q?mvbinHpqCFTlxXrfnwMcVHE/y26fmCQOLzk/nPFJUG+ImVG5a6hhcCqbMkB0?=
 =?us-ascii?Q?gfdGN1Ouw/cFBkb+5WXGXchT5HolKojXruftZkCz2phLCJOEfNnuPoySR3ED?=
 =?us-ascii?Q?/WKoAyE6PFUoMqMQ0cdzstIQTa9sT+m04biMh/t9Mh4sRWdqG4q0wNooTpBs?=
 =?us-ascii?Q?inQKfw7cU78PZpsctgRmQVR6qkQ25iNEAVy6jtXGOJu2PDm3Cznt04wsGSN4?=
 =?us-ascii?Q?IMmexXwso5t2k5IYB9yFPXOVJYGhscXldmluJRV4+ql4toNpG3TlFnRsN6AN?=
 =?us-ascii?Q?4m5M6V8rhWNH0nBO+BEVNgBJK6yF5Jepu0dtDcsFlMRGESK0ltd7vb9aDLHP?=
 =?us-ascii?Q?PUxruL4qLtMH2hNBO2QXAdqELBo7avX+Pfx5AZH/7xmxocRvk2rYxU5HXgjP?=
 =?us-ascii?Q?e1qIlBnZLUDVknUHcjKInSDQ9JgSLDLWq2Ohe6S/Q2vHQB2Wo58sQFUWHzsb?=
 =?us-ascii?Q?Q7CA+nQT/001Np48U01DfKe1rwuLImuBF6BOyoC2HMLB+okN11q1HVd2GfIq?=
 =?us-ascii?Q?NcXj3IVEg9XdnBOA44O+LtUSCNM2sudkjg6yWboBcJGs5Y1DtMcAxjGIj+Kd?=
 =?us-ascii?Q?35ESJXYpDZD2LnoZ+jXj0ik=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e67f645e-530c-4688-461f-08dad0c97359
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Nov 2022 22:48:07.0612
 (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: M+ZEczLNcB2cNSoccORefcknYEv8iLPMkME3EyMAGPSYv3v8u26lMtzjxFrBD0ZqJP+6A/QGw2ifNe0HrEH16oR4eH0Nwulmx/YJksrWnjY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN0PR10MB4998
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>
---

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