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 v4 net-next 9/9] dt-bindings: net: mscc,vsc7514-switch: utilize generic ethernet-switch.yaml
Date:   Fri,  2 Dec 2022 12:45:59 -0800
Message-Id: <20221202204559.162619-10-colin.foster@in-advantage.com>
In-Reply-To: <20221202204559.162619-1-colin.foster@in-advantage.com>
References: <20221202204559.162619-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?3xPXRfOQ396/HteBWVq0m+Srfu4p9gvxn0uwRBF+5C8+eglPsL1fbkbPtlM8?=
 =?us-ascii?Q?XiYMdc0A7nBLHfGgWUJ4umfCfA+HQfDfcoOMbsocpgI7c0oPqgDwGuJkuMQv?=
 =?us-ascii?Q?rX1Qho1q0FXFWgT+7cjtfjohOAwB9+ZH/yhZFaQS1M4wsdxfkAcQEOznJQ0N?=
 =?us-ascii?Q?AjrZfkYSYDLKcxYgME6qOeGTUBLBGv9Cf8SMaYW/rJ6jg5T9VaX6iWiZ+D37?=
 =?us-ascii?Q?pyQzhtD63uMc7APtuxPyPcFietfzsyIhtC3UpiBS3W71W4NnYCCJFeWtVp1r?=
 =?us-ascii?Q?dvOWRz4LDGCTk5qUCiQN+lArHMFKqnqUkW0Qg4T1eVPw5qfRLD7xWqQXzbVF?=
 =?us-ascii?Q?+xgel7A+sxId74GMAkmQnvkAGnAWQH8KXcb2kAvRnviTQd2Cs1FBD94xPPtH?=
 =?us-ascii?Q?pbkaKQVT6itEn391LaqSEePsQqF92ou1X0JZEBnMG314sihzy3xYrXKiecph?=
 =?us-ascii?Q?5lTOCRgXJg2hoZTPkAqqrCvvhyoFiFM1rEcuwBm1uqehuuLgTUPoZIipxJOA?=
 =?us-ascii?Q?rQR7dUQx0JE7VFl7r06WaP/Fz5amZwbeWp2rYg9u229z71d9mavWZyak2i2e?=
 =?us-ascii?Q?skU4XhhtSTl026KfNMvRDtFy3ujz2YPL2mPsJ/yoKwVx/80zI3HF0Okkz8Av?=
 =?us-ascii?Q?IGRmkwF/0r4Qw9T/cZPXvOL+bn4ufeenpsBVt/RlDUifez6s00reVfNQhoI5?=
 =?us-ascii?Q?2L2KnQ6RwH3G/dhQkSYWS2YZNIsr6rYvUC897HKwa8A1abmXCxzidI5H4Pud?=
 =?us-ascii?Q?lYHHyjTkqS0LYE7AE98L+STq307ckSVz6FpiR3j6BtSHe9eVlELg09HyLPrB?=
 =?us-ascii?Q?T+W//GRS/w0plUkI0NTLpQlW/LbhO8rIRJIXGl/YL9qwhZ+hT+2ZxVaAYj7L?=
 =?us-ascii?Q?Hihyv8nU6UAP1zngp0QF05i3tRZE5nvffi3DGjn6y7FAF5s/kmoIHWDC0ErK?=
 =?us-ascii?Q?GTE7bRMoutqBdXGZFwPgeqp1UezM17ACFo44vgXB8B3dsEtZu5oruvisVwHk?=
 =?us-ascii?Q?bsp0iDlVins+MFfcfZS8XQAprYOUYUfsa6DRLZTcCzTI3xXG6PwCl5Tb2C0I?=
 =?us-ascii?Q?f7/5rTAyjTF03b78SCd33xmACH/1GaABalcFVC2cUuDx6wL+DwQcUC6AeBpG?=
 =?us-ascii?Q?kd0tUKji4r9q/YIopkUDn5MX0awYzW7KSv7muI5rCDiLx1xHbNOOZoXo8j6O?=
 =?us-ascii?Q?Y/1W3ILxD8XN1qA/VuztN2rqL54lhkZRbybz67xyNS9UZgdoZlOG50J4EH7v?=
 =?us-ascii?Q?vkS6VNL9IGiv+mGo7zGQoEmzm9Ci5yJm0h5nfGVM0nWQYItpJDrNBu68Zq3x?=
 =?us-ascii?Q?Skgkqewj8Ykodja0sMPei+WT/Y4zPOwgSappngd4NmkE+THAjnbxJLCNPCuN?=
 =?us-ascii?Q?iR+qus9tUlZucLBVubNnMzW0eGanOAfpbgNIHh46uXTHJwaUHkoVVG9J8BDH?=
 =?us-ascii?Q?Mo/ROg6swJ9TwG/qco/WuM5yWljiUMXLb1Kx9Vkmne8IlVk76Bv/OTdBzU3p?=
 =?us-ascii?Q?HwsuTUgGJE1H3PwNnF2iRk/SpQOJN1ncsB8zotBM+wRL3cTOx1uUlrz6ZtVE?=
 =?us-ascii?Q?USMX5Qyqd6QRQD73KzSE4LmAu0S1PAsiIbXMICZA+ueBVdasZGhbxziOzHzJ?=
 =?us-ascii?Q?tdgpV7TmoFJY22MUUtBjtEw=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9f1f5cb0-a5a4-4587-86a4-08dad4a64426
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Dec 2022 20:46:20.1256
 (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: cpyacNXEjnsAmnr567+cFjoyBeaX1nGX3Ni9u5GN7QZ4DDY7GRZgJLih8tV/8jblUwC8oLMtVBeQu9O0P8ZhCMCFlhREDm8y6Eb1al0gCKs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR10MB4944
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>
---

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