On Tue, Dec 06, 2022 at 05:18:51PM +0200, Vladimir Oltean wrote: > On Fri, Dec 02, 2022 at 12:45:53PM -0800, Colin Foster wrote: > > DSA switches can fall into one of two categories: switches where all ports > > follow standard '(ethernet-)?port' properties, and switches that have > > additional properties for the ports. > > > > The scenario where DSA ports are all standardized can be handled by > > swtiches with a reference to the new 'dsa.yaml#/$defs/ethernet-ports'. > > > > The scenario where DSA ports require additional properties can reference > > '$dsa.yaml#' directly. This will allow switches to reference these standard > > defitions of the DSA switch, but add additional properties under the port > > nodes. > > --- > > diff --git a/Documentation/devicetree/bindings/net/dsa/dsa.yaml b/Documentation/devicetree/bindings/net/dsa/dsa.yaml > > index b9d48e357e77..b9e366e46aed 100644 > > --- a/Documentation/devicetree/bindings/net/dsa/dsa.yaml > > +++ b/Documentation/devicetree/bindings/net/dsa/dsa.yaml > > @@ -19,9 +19,6 @@ description: > > select: false > > > > properties: > > - $nodename: > > - pattern: "^(ethernet-)?switch(@.*)?$" > > - > > Does this deletion belong to this patch or to "dt-bindings: net: add > generic ethernet-switch"? Good catch. I'll move it. > > > dsa,member: > > minItems: 2 > > maxItems: 2 > > @@ -58,4 +55,26 @@ oneOf: > > > > additionalProperties: true