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:   Sean Anderson <sean.anderson@seco.com>
To:     "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Madalin Bucur <madalin.bucur@nxp.com>,
        Camelia Alexandra Groza <camelia.groza@nxp.com>,
        netdev@vger.kernel.org
Cc:     Russell King <linux@armlinux.org.uk>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        "linuxppc-dev @ lists . ozlabs . org" <linuxppc-dev@lists.ozlabs.org>,
        Sean Anderson <sean.anderson@seco.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org
Subject: [PATCH net-next v6 1/9] dt-bindings: net: Expand pcs-handle to an array
Date:   Fri, 30 Sep 2022 16:09:25 -0400
Message-Id: <20220930200933.4111249-2-sean.anderson@seco.com>
In-Reply-To: <20220930200933.4111249-1-sean.anderson@seco.com>
References: <20220930200933.4111249-1-sean.anderson@seco.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?gLSdFyfSkBAU/E21cP9rWqTRm+3Yz7lenHMSk1UpMJ1Xrrv3+qb7qgcr26jB?=
 =?us-ascii?Q?66naE7300vzdJchUnH5RzcP9xK05lZQVp7tQ2p1F7vOyBqgnwgnICopBXxZh?=
 =?us-ascii?Q?JnLN8iI4c9pE5/2GFcaign3kFdPsZSxPaDB1HENaUM3nH+qgkv2szOPLMA8h?=
 =?us-ascii?Q?/ZTy9lXTApZxnkzivrtJcOhNEW1WckX7szZx7YzvxU6TDU94NfXF71fWcldg?=
 =?us-ascii?Q?MkHIgY2Winx2zMkT6Px5wEohO5VZKTmijD53w0+Xyxsin12mhbWp9V1UckK6?=
 =?us-ascii?Q?Al1gWKXYgaWXbK9/RG4nUyTdcr09Ma4WOzqv+dtmu4ss/a05OYflPgT+We9Q?=
 =?us-ascii?Q?CTkuLd3oCACoYI0ht248xKSZSqJhWNqnnmN7bVn0+SI7hcNyrmA2/wHU6hyd?=
 =?us-ascii?Q?prAyPmcckW8r1Ew6XxjJrJp+coD4Y8RTn2hpU42nJ+xGSjfap67u/3LxX0o1?=
 =?us-ascii?Q?QJPk2DmSQL9dX8c+ZQMlktOaON6Zv3hVuusqisPXQso623Mg9D34sHjGP0cK?=
 =?us-ascii?Q?X/4INViTEMaHjfgNU+9cZsgwzqTJAX+1K9/XJ96enUgZQFmaCLakATkcVUS6?=
 =?us-ascii?Q?8pH5B/OiakqOKbyuXh0RznjGOi3m3++gOeWf3PS407FpJ8U73uvSAIH551tV?=
 =?us-ascii?Q?+PpgxtSB9vJzfYlxPr/JKaU/K8xkH+U6D6cDE/I9lz2nPEvdD8l0ycqIcACT?=
 =?us-ascii?Q?dnxffMUrJExzUrZQAjteXQ3sl1e3G/lnm2fOAQdKKEeqVtcdHGR1HDZwF3Ys?=
 =?us-ascii?Q?CgqD2r8hkhM5VsDMhs0bqji7lRaFQmfEzty79lgngSSR69ay1Ha57lMVftLS?=
 =?us-ascii?Q?Hvgmi7ufX5AetcpJ21qF02qWtvaP2J0aKAfCLBwYxNv6Orc9/uo8c+Fj7/sk?=
 =?us-ascii?Q?rHpS9+bP6pQqNfFIIwIwi7DEmntSHN1iP9Hmuquxmhg+5L3ebyT7OC5LswLl?=
 =?us-ascii?Q?ErFemDBvHv3D3sM07fkdKB1wZMVf2e5iSJeU69q0XwM5RsuzQZSJ+gVCErCw?=
 =?us-ascii?Q?wHdzgjyBZk13rS2MG/pcGRO8pdmIDrVjasPo954Stq1qIhJh4IuXAz9oz+0e?=
 =?us-ascii?Q?hnuuuZdJPsNdm/4oCLx9xLFt/tBOMCEZLbaKXI2mDbEZW9OO4LH+24TOxpES?=
 =?us-ascii?Q?12vYbHtz1lnIPr+jqSqvnRsGuwkLXKjQPj+X8SfH1G/OlONT67un4bJafc9Q?=
 =?us-ascii?Q?9ml3s0ghVGW96zTMD1GGccBjyXsSLS9aI2FduOnMTaO8e1rvLVDyxwHVYE1n?=
 =?us-ascii?Q?hsMDny902vqTXoYOo8uknz9zLTEHlRFcWHiGRywlxQwzJjQVsQoCdCl/4ybs?=
 =?us-ascii?Q?/NnQzCMDwtfuFUUsexUbtX7B8PFveqzhj73BK4kiYTvH6xntxIgFfw6G991z?=
 =?us-ascii?Q?EiP5uqykwdUOZX2mvhGehzXhNQSPDl4htQfPpv5w3V27lBn4HSWyQkz2nj0e?=
 =?us-ascii?Q?3edRh3CtdhB+lFA8pfJHshQJqAKQOw9QI7naN43+GI03FGJpOcp2VZoOQ0km?=
 =?us-ascii?Q?ptcHTpVOWg2NBr7k3v3fHVpu7ubsLvGRZdijaeUJamct7Kn4gg3vQWJjePT0?=
 =?us-ascii?Q?dxLHyy/Mqyw0Uxh1i1qvppU9gECebf2YVH2eU2dFLKKCcIxshibE2s4I3OiM?=
 =?us-ascii?Q?Sg=3D=3D?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f3f05b0c-421b-4bbb-daea-08daa31fba2c
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Sep 2022 20:09:49.0724
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: r7ATmW9qE6+M9re+ssfqZZWQp54KJ+2IyMpOOuaCl2tIEYnybayTOP0x7GaN1oz7zzu6CvuAOBYuxBscpZCToA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR03MB7749
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,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

This allows multiple phandles to be specified for pcs-handle, such as
when multiple PCSs are present for a single MAC. To differentiate
between them, also add a pcs-handle-names property.

Signed-off-by: Sean Anderson <sean.anderson@seco.com>
---
This was previously submitted as [1]. I expect to update this series
more, so I have moved it here. Changes from that version include:
- Add maxItems to existing bindings
- Add a dependency from pcs-names to pcs-handle.

[1] https://lore.kernel.org/netdev/20220711160519.741990-3-sean.anderson@seco.com/

Changes in v6:
- Remove unnecessary $ref from renesas,rzn1-a5psw
- Remove unnecessary type from pcs-handle-names
- Add maxItems to pcs-handle

Changes in v4:
- Use pcs-handle-names instead of pcs-names, as discussed

Changes in v3:
- New

 .../bindings/net/dsa/renesas,rzn1-a5psw.yaml          |  2 +-
 .../devicetree/bindings/net/ethernet-controller.yaml  | 11 ++++++++++-
 .../devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml   |  2 +-
 3 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
index 7ca9c19a157c..0a0d62b6c00e 100644
--- a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
@@ -74,10 +74,10 @@ properties:
 
         properties:
           pcs-handle:
+            maxItems: 1
             description:
               phandle pointing to a PCS sub-node compatible with
               renesas,rzn1-miic.yaml#
-            $ref: /schemas/types.yaml#/definitions/phandle
 
 unevaluatedProperties: false
 
diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
index 4b3c590fcebf..3aef506fa158 100644
--- a/Documentation/devicetree/bindings/net/ethernet-controller.yaml
+++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
@@ -108,11 +108,17 @@ properties:
     $ref: "#/properties/phy-connection-type"
 
   pcs-handle:
-    $ref: /schemas/types.yaml#/definitions/phandle
+    $ref: /schemas/types.yaml#/definitions/phandle-array
+    items:
+      maxItems: 1
     description:
       Specifies a reference to a node representing a PCS PHY device on a MDIO
       bus to link with an external PHY (phy-handle) if exists.
 
+  pcs-handle-names:
+    description:
+      The name of each PCS in pcs-handle.
+
   phy-handle:
     $ref: /schemas/types.yaml#/definitions/phandle
     description:
@@ -216,6 +222,9 @@ properties:
         required:
           - speed
 
+dependencies:
+  pcs-handle-names: [pcs-handle]
+
 allOf:
   - if:
       properties:
diff --git a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
index 7f620a71a972..600240281e8c 100644
--- a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
+++ b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
@@ -31,7 +31,7 @@ properties:
   phy-mode: true
 
   pcs-handle:
-    $ref: /schemas/types.yaml#/definitions/phandle
+    maxItems: 1
     description:
       A reference to a node representing a PCS PHY device found on
       the internal MDIO bus.
-- 
2.35.1.1320.gc452695387.dirty