* [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml
@ 2021-11-12 16:57 Ansuel Smith
2021-11-12 16:57 ` [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation Ansuel Smith
2021-11-29 21:18 ` [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Rob Herring
0 siblings, 2 replies; 5+ messages in thread
From: Ansuel Smith @ 2021-11-12 16:57 UTC (permalink / raw)
To: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Jakub Kicinski, Rob Herring, John Crispin,
netdev, devicetree, linux-kernel
Cc: Ansuel Smith
Some switch may require to add additional binding to the node port.
Move DSA generic port definition to a dedicated yaml to permit this.
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
---
.../devicetree/bindings/net/dsa/dsa-port.yaml | 70 +++++++++++++++++++
.../devicetree/bindings/net/dsa/dsa.yaml | 54 +-------------
2 files changed, 72 insertions(+), 52 deletions(-)
create mode 100644 Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
diff --git a/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
new file mode 100644
index 000000000000..258df41c9133
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
@@ -0,0 +1,70 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/dsa/dsa-port.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Ethernet Switch port Device Tree Bindings
+
+maintainers:
+ - Andrew Lunn <andrew@lunn.ch>
+ - Florian Fainelli <f.fainelli@gmail.com>
+ - Vivien Didelot <vivien.didelot@gmail.com>
+
+description:
+ Ethernet switch port Description
+
+properties:
+ reg:
+ description: Port number
+
+ label:
+ description:
+ Describes the label associated with this port, which will become
+ the netdev name
+ $ref: /schemas/types.yaml#/definitions/string
+
+ link:
+ description:
+ Should be a list of phandles to other switch's DSA port. This
+ port is used as the outgoing port towards the phandle ports. The
+ full routing information must be given, not just the one hop
+ routes to neighbouring switches
+ $ref: /schemas/types.yaml#/definitions/phandle-array
+
+ ethernet:
+ description:
+ Should be a phandle to a valid Ethernet device node. This host
+ device is what the switch port is connected to
+ $ref: /schemas/types.yaml#/definitions/phandle
+
+ dsa-tag-protocol:
+ description:
+ Instead of the default, the switch will use this tag protocol if
+ possible. Useful when a device supports multiple protocols and
+ the default is incompatible with the Ethernet device.
+ enum:
+ - dsa
+ - edsa
+ - ocelot
+ - ocelot-8021q
+ - seville
+
+ phy-handle: true
+
+ phy-mode: true
+
+ fixed-link: true
+
+ mac-address: true
+
+ sfp: true
+
+ managed: true
+
+required:
+ - reg
+
+additionalProperties: true
+
+...
diff --git a/Documentation/devicetree/bindings/net/dsa/dsa.yaml b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
index 224cfa45de9a..15ea9ef3def9 100644
--- a/Documentation/devicetree/bindings/net/dsa/dsa.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
@@ -46,58 +46,8 @@ patternProperties:
type: object
description: Ethernet switch ports
- properties:
- reg:
- description: Port number
-
- label:
- description:
- Describes the label associated with this port, which will become
- the netdev name
- $ref: /schemas/types.yaml#/definitions/string
-
- link:
- description:
- Should be a list of phandles to other switch's DSA port. This
- port is used as the outgoing port towards the phandle ports. The
- full routing information must be given, not just the one hop
- routes to neighbouring switches
- $ref: /schemas/types.yaml#/definitions/phandle-array
-
- ethernet:
- description:
- Should be a phandle to a valid Ethernet device node. This host
- device is what the switch port is connected to
- $ref: /schemas/types.yaml#/definitions/phandle
-
- dsa-tag-protocol:
- description:
- Instead of the default, the switch will use this tag protocol if
- possible. Useful when a device supports multiple protocols and
- the default is incompatible with the Ethernet device.
- enum:
- - dsa
- - edsa
- - ocelot
- - ocelot-8021q
- - seville
-
- phy-handle: true
-
- phy-mode: true
-
- fixed-link: true
-
- mac-address: true
-
- sfp: true
-
- managed: true
-
- required:
- - reg
-
- additionalProperties: false
+ allOf:
+ - $ref: dsa-port.yaml#
oneOf:
- required:
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation
2021-11-12 16:57 [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Ansuel Smith
@ 2021-11-12 16:57 ` Ansuel Smith
2021-11-29 21:20 ` Rob Herring
2021-11-29 21:18 ` [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Ansuel Smith @ 2021-11-12 16:57 UTC (permalink / raw)
To: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Jakub Kicinski, Rob Herring, John Crispin,
netdev, devicetree, linux-kernel
Cc: Ansuel Smith
Clean and improve port definition for qca8k documentation by referencing
the dsa generic port definition and adding the additional specific port
definition.
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
---
.../devicetree/bindings/net/dsa/qca8k.yaml | 82 ++++++-------------
1 file changed, 23 insertions(+), 59 deletions(-)
diff --git a/Documentation/devicetree/bindings/net/dsa/qca8k.yaml b/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
index 48de0ace265d..9eb24cdf6cd4 100644
--- a/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
@@ -99,65 +99,29 @@ patternProperties:
type: object
description: Ethernet switch ports
- properties:
- reg:
- description: Port number
-
- label:
- description:
- Describes the label associated with this port, which will become
- the netdev name
- $ref: /schemas/types.yaml#/definitions/string
-
- link:
- description:
- Should be a list of phandles to other switch's DSA port. This
- port is used as the outgoing port towards the phandle ports. The
- full routing information must be given, not just the one hop
- routes to neighbouring switches
- $ref: /schemas/types.yaml#/definitions/phandle-array
-
- ethernet:
- description:
- Should be a phandle to a valid Ethernet device node. This host
- device is what the switch port is connected to
- $ref: /schemas/types.yaml#/definitions/phandle
-
- phy-handle: true
-
- phy-mode: true
-
- fixed-link: true
-
- mac-address: true
-
- sfp: true
-
- qca,sgmii-rxclk-falling-edge:
- $ref: /schemas/types.yaml#/definitions/flag
- description:
- Set the receive clock phase to falling edge. Mostly commonly used on
- the QCA8327 with CPU port 0 set to SGMII.
-
- qca,sgmii-txclk-falling-edge:
- $ref: /schemas/types.yaml#/definitions/flag
- description:
- Set the transmit clock phase to falling edge.
-
- qca,sgmii-enable-pll:
- $ref: /schemas/types.yaml#/definitions/flag
- description:
- For SGMII CPU port, explicitly enable PLL, TX and RX chain along with
- Signal Detection. On the QCA8327 this should not be enabled, otherwise
- the SGMII port will not initialize. When used on the QCA8337, revision 3
- or greater, a warning will be displayed. When the CPU port is set to
- SGMII on the QCA8337, it is advised to set this unless a communication
- issue is observed.
-
- required:
- - reg
-
- additionalProperties: false
+ allOf:
+ - $ref: dsa-port.yaml#
+ - properties:
+ qca,sgmii-rxclk-falling-edge:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description:
+ Set the receive clock phase to falling edge. Mostly commonly used on
+ the QCA8327 with CPU port 0 set to SGMII.
+
+ qca,sgmii-txclk-falling-edge:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description:
+ Set the transmit clock phase to falling edge.
+
+ qca,sgmii-enable-pll:
+ $ref: /schemas/types.yaml#/definitions/flag
+ description:
+ For SGMII CPU port, explicitly enable PLL, TX and RX chain along with
+ Signal Detection. On the QCA8327 this should not be enabled, otherwise
+ the SGMII port will not initialize. When used on the QCA8337, revision 3
+ or greater, a warning will be displayed. When the CPU port is set to
+ SGMII on the QCA8337, it is advised to set this unless a communication
+ issue is observed.
oneOf:
- required:
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml
2021-11-12 16:57 [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Ansuel Smith
2021-11-12 16:57 ` [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation Ansuel Smith
@ 2021-11-29 21:18 ` Rob Herring
2021-11-29 22:24 ` Ansuel Smith
1 sibling, 1 reply; 5+ messages in thread
From: Rob Herring @ 2021-11-29 21:18 UTC (permalink / raw)
To: Ansuel Smith
Cc: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Jakub Kicinski, John Crispin, netdev,
devicetree, linux-kernel
On Fri, Nov 12, 2021 at 05:57:51PM +0100, Ansuel Smith wrote:
> Some switch may require to add additional binding to the node port.
> Move DSA generic port definition to a dedicated yaml to permit this.
>
> Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> ---
> .../devicetree/bindings/net/dsa/dsa-port.yaml | 70 +++++++++++++++++++
> .../devicetree/bindings/net/dsa/dsa.yaml | 54 +-------------
> 2 files changed, 72 insertions(+), 52 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
>
> diff --git a/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
> new file mode 100644
> index 000000000000..258df41c9133
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
> @@ -0,0 +1,70 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/net/dsa/dsa-port.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Ethernet Switch port Device Tree Bindings
> +
> +maintainers:
> + - Andrew Lunn <andrew@lunn.ch>
> + - Florian Fainelli <f.fainelli@gmail.com>
> + - Vivien Didelot <vivien.didelot@gmail.com>
> +
> +description:
> + Ethernet switch port Description
> +
> +properties:
> + reg:
> + description: Port number
> +
> + label:
> + description:
> + Describes the label associated with this port, which will become
> + the netdev name
> + $ref: /schemas/types.yaml#/definitions/string
> +
> + link:
> + description:
> + Should be a list of phandles to other switch's DSA port. This
> + port is used as the outgoing port towards the phandle ports. The
> + full routing information must be given, not just the one hop
> + routes to neighbouring switches
> + $ref: /schemas/types.yaml#/definitions/phandle-array
> +
> + ethernet:
> + description:
> + Should be a phandle to a valid Ethernet device node. This host
> + device is what the switch port is connected to
> + $ref: /schemas/types.yaml#/definitions/phandle
> +
> + dsa-tag-protocol:
> + description:
> + Instead of the default, the switch will use this tag protocol if
> + possible. Useful when a device supports multiple protocols and
> + the default is incompatible with the Ethernet device.
> + enum:
> + - dsa
> + - edsa
> + - ocelot
> + - ocelot-8021q
> + - seville
> +
> + phy-handle: true
> +
> + phy-mode: true
> +
> + fixed-link: true
> +
> + mac-address: true
> +
> + sfp: true
> +
> + managed: true
> +
> +required:
> + - reg
> +
> +additionalProperties: true
> +
> +...
> diff --git a/Documentation/devicetree/bindings/net/dsa/dsa.yaml b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> index 224cfa45de9a..15ea9ef3def9 100644
> --- a/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> @@ -46,58 +46,8 @@ patternProperties:
> type: object
> description: Ethernet switch ports
>
> - properties:
> - reg:
> - description: Port number
> -
> - label:
> - description:
> - Describes the label associated with this port, which will become
> - the netdev name
> - $ref: /schemas/types.yaml#/definitions/string
> -
> - link:
> - description:
> - Should be a list of phandles to other switch's DSA port. This
> - port is used as the outgoing port towards the phandle ports. The
> - full routing information must be given, not just the one hop
> - routes to neighbouring switches
> - $ref: /schemas/types.yaml#/definitions/phandle-array
> -
> - ethernet:
> - description:
> - Should be a phandle to a valid Ethernet device node. This host
> - device is what the switch port is connected to
> - $ref: /schemas/types.yaml#/definitions/phandle
> -
> - dsa-tag-protocol:
> - description:
> - Instead of the default, the switch will use this tag protocol if
> - possible. Useful when a device supports multiple protocols and
> - the default is incompatible with the Ethernet device.
> - enum:
> - - dsa
> - - edsa
> - - ocelot
> - - ocelot-8021q
> - - seville
> -
> - phy-handle: true
> -
> - phy-mode: true
> -
> - fixed-link: true
> -
> - mac-address: true
> -
> - sfp: true
> -
> - managed: true
> -
> - required:
> - - reg
> -
> - additionalProperties: false
> + allOf:
> + - $ref: dsa-port.yaml#
Don't need 'allOf' here. And you need to add 'unevaluatedProperties:
false'. With that,
Reviewed-by: Rob Herring <robh@kernel.org>
(This needs to go in net-next to avoid conflicts, but given the
maintainers didn't apply it already unreviewed they probably expect I
will apply it.)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation
2021-11-12 16:57 ` [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation Ansuel Smith
@ 2021-11-29 21:20 ` Rob Herring
0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2021-11-29 21:20 UTC (permalink / raw)
To: Ansuel Smith
Cc: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Jakub Kicinski, John Crispin, netdev,
devicetree, linux-kernel
On Fri, Nov 12, 2021 at 05:57:52PM +0100, Ansuel Smith wrote:
> Clean and improve port definition for qca8k documentation by referencing
> the dsa generic port definition and adding the additional specific port
> definition.
>
> Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> ---
> .../devicetree/bindings/net/dsa/qca8k.yaml | 82 ++++++-------------
> 1 file changed, 23 insertions(+), 59 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/dsa/qca8k.yaml b/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
> index 48de0ace265d..9eb24cdf6cd4 100644
> --- a/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/qca8k.yaml
> @@ -99,65 +99,29 @@ patternProperties:
> type: object
> description: Ethernet switch ports
>
> - properties:
> - reg:
> - description: Port number
> -
> - label:
> - description:
> - Describes the label associated with this port, which will become
> - the netdev name
> - $ref: /schemas/types.yaml#/definitions/string
> -
> - link:
> - description:
> - Should be a list of phandles to other switch's DSA port. This
> - port is used as the outgoing port towards the phandle ports. The
> - full routing information must be given, not just the one hop
> - routes to neighbouring switches
> - $ref: /schemas/types.yaml#/definitions/phandle-array
> -
> - ethernet:
> - description:
> - Should be a phandle to a valid Ethernet device node. This host
> - device is what the switch port is connected to
> - $ref: /schemas/types.yaml#/definitions/phandle
> -
> - phy-handle: true
> -
> - phy-mode: true
> -
> - fixed-link: true
> -
> - mac-address: true
> -
> - sfp: true
> -
> - qca,sgmii-rxclk-falling-edge:
> - $ref: /schemas/types.yaml#/definitions/flag
> - description:
> - Set the receive clock phase to falling edge. Mostly commonly used on
> - the QCA8327 with CPU port 0 set to SGMII.
> -
> - qca,sgmii-txclk-falling-edge:
> - $ref: /schemas/types.yaml#/definitions/flag
> - description:
> - Set the transmit clock phase to falling edge.
> -
> - qca,sgmii-enable-pll:
> - $ref: /schemas/types.yaml#/definitions/flag
> - description:
> - For SGMII CPU port, explicitly enable PLL, TX and RX chain along with
> - Signal Detection. On the QCA8327 this should not be enabled, otherwise
> - the SGMII port will not initialize. When used on the QCA8337, revision 3
> - or greater, a warning will be displayed. When the CPU port is set to
> - SGMII on the QCA8337, it is advised to set this unless a communication
> - issue is observed.
> -
> - required:
> - - reg
> -
> - additionalProperties: false
> + allOf:
> + - $ref: dsa-port.yaml#
> + - properties:
You can drop 'allOf' here too. And add 'unevaluatedProperties: false'.
With that,
Reviewed-by: Rob Herring <robh@kernel.org>
> + qca,sgmii-rxclk-falling-edge:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description:
> + Set the receive clock phase to falling edge. Mostly commonly used on
> + the QCA8327 with CPU port 0 set to SGMII.
> +
> + qca,sgmii-txclk-falling-edge:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description:
> + Set the transmit clock phase to falling edge.
> +
> + qca,sgmii-enable-pll:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description:
> + For SGMII CPU port, explicitly enable PLL, TX and RX chain along with
> + Signal Detection. On the QCA8327 this should not be enabled, otherwise
> + the SGMII port will not initialize. When used on the QCA8337, revision 3
> + or greater, a warning will be displayed. When the CPU port is set to
> + SGMII on the QCA8337, it is advised to set this unless a communication
> + issue is observed.
>
> oneOf:
> - required:
> --
> 2.32.0
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml
2021-11-29 21:18 ` [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Rob Herring
@ 2021-11-29 22:24 ` Ansuel Smith
0 siblings, 0 replies; 5+ messages in thread
From: Ansuel Smith @ 2021-11-29 22:24 UTC (permalink / raw)
To: Rob Herring
Cc: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Jakub Kicinski, John Crispin, netdev,
devicetree, linux-kernel
On Mon, Nov 29, 2021 at 03:18:54PM -0600, Rob Herring wrote:
> On Fri, Nov 12, 2021 at 05:57:51PM +0100, Ansuel Smith wrote:
> > Some switch may require to add additional binding to the node port.
> > Move DSA generic port definition to a dedicated yaml to permit this.
> >
> > Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> > ---
> > .../devicetree/bindings/net/dsa/dsa-port.yaml | 70 +++++++++++++++++++
> > .../devicetree/bindings/net/dsa/dsa.yaml | 54 +-------------
> > 2 files changed, 72 insertions(+), 52 deletions(-)
> > create mode 100644 Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
> > new file mode 100644
> > index 000000000000..258df41c9133
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/net/dsa/dsa-port.yaml
> > @@ -0,0 +1,70 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/net/dsa/dsa-port.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Ethernet Switch port Device Tree Bindings
> > +
> > +maintainers:
> > + - Andrew Lunn <andrew@lunn.ch>
> > + - Florian Fainelli <f.fainelli@gmail.com>
> > + - Vivien Didelot <vivien.didelot@gmail.com>
> > +
> > +description:
> > + Ethernet switch port Description
> > +
> > +properties:
> > + reg:
> > + description: Port number
> > +
> > + label:
> > + description:
> > + Describes the label associated with this port, which will become
> > + the netdev name
> > + $ref: /schemas/types.yaml#/definitions/string
> > +
> > + link:
> > + description:
> > + Should be a list of phandles to other switch's DSA port. This
> > + port is used as the outgoing port towards the phandle ports. The
> > + full routing information must be given, not just the one hop
> > + routes to neighbouring switches
> > + $ref: /schemas/types.yaml#/definitions/phandle-array
> > +
> > + ethernet:
> > + description:
> > + Should be a phandle to a valid Ethernet device node. This host
> > + device is what the switch port is connected to
> > + $ref: /schemas/types.yaml#/definitions/phandle
> > +
> > + dsa-tag-protocol:
> > + description:
> > + Instead of the default, the switch will use this tag protocol if
> > + possible. Useful when a device supports multiple protocols and
> > + the default is incompatible with the Ethernet device.
> > + enum:
> > + - dsa
> > + - edsa
> > + - ocelot
> > + - ocelot-8021q
> > + - seville
> > +
> > + phy-handle: true
> > +
> > + phy-mode: true
> > +
> > + fixed-link: true
> > +
> > + mac-address: true
> > +
> > + sfp: true
> > +
> > + managed: true
> > +
> > +required:
> > + - reg
> > +
> > +additionalProperties: true
> > +
> > +...
> > diff --git a/Documentation/devicetree/bindings/net/dsa/dsa.yaml b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> > index 224cfa45de9a..15ea9ef3def9 100644
> > --- a/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> > +++ b/Documentation/devicetree/bindings/net/dsa/dsa.yaml
> > @@ -46,58 +46,8 @@ patternProperties:
> > type: object
> > description: Ethernet switch ports
> >
> > - properties:
> > - reg:
> > - description: Port number
> > -
> > - label:
> > - description:
> > - Describes the label associated with this port, which will become
> > - the netdev name
> > - $ref: /schemas/types.yaml#/definitions/string
> > -
> > - link:
> > - description:
> > - Should be a list of phandles to other switch's DSA port. This
> > - port is used as the outgoing port towards the phandle ports. The
> > - full routing information must be given, not just the one hop
> > - routes to neighbouring switches
> > - $ref: /schemas/types.yaml#/definitions/phandle-array
> > -
> > - ethernet:
> > - description:
> > - Should be a phandle to a valid Ethernet device node. This host
> > - device is what the switch port is connected to
> > - $ref: /schemas/types.yaml#/definitions/phandle
> > -
> > - dsa-tag-protocol:
> > - description:
> > - Instead of the default, the switch will use this tag protocol if
> > - possible. Useful when a device supports multiple protocols and
> > - the default is incompatible with the Ethernet device.
> > - enum:
> > - - dsa
> > - - edsa
> > - - ocelot
> > - - ocelot-8021q
> > - - seville
> > -
> > - phy-handle: true
> > -
> > - phy-mode: true
> > -
> > - fixed-link: true
> > -
> > - mac-address: true
> > -
> > - sfp: true
> > -
> > - managed: true
> > -
> > - required:
> > - - reg
> > -
> > - additionalProperties: false
> > + allOf:
> > + - $ref: dsa-port.yaml#
>
> Don't need 'allOf' here. And you need to add 'unevaluatedProperties:
> false'. With that,
>
> Reviewed-by: Rob Herring <robh@kernel.org>
>
> (This needs to go in net-next to avoid conflicts, but given the
> maintainers didn't apply it already unreviewed they probably expect I
> will apply it.)
If you want I can push a v2 with the net-next tag.
--
Ansuel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-11-29 22:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-12 16:57 [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Ansuel Smith
2021-11-12 16:57 ` [PATCH 2/2] dt-bindings: net: dsa: qca8k: improve port definition documentation Ansuel Smith
2021-11-29 21:20 ` Rob Herring
2021-11-29 21:18 ` [PATCH 1/2] dt-bindings: net: dsa: split generic port definition from dsa.yaml Rob Herring
2021-11-29 22:24 ` Ansuel Smith
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).