* [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
@ 2026-05-07 9:26 Linus Walleij
2026-05-07 11:40 ` Herve Codina
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Linus Walleij @ 2026-05-07 9:26 UTC (permalink / raw)
To: Herve Codina, Andrew Lunn, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Horatiu Vultur
Cc: netdev, devicetree, Linus Walleij
The dsa.yaml and ethernet-switch.yaml bindings recommend
prefixing ethernet switches and ports with "ethernet-" so
make the LAN966x do the same.
Reported-by: Herve Codina <herve.codina@bootlin.com>
Signed-off-by: Linus Walleij <linusw@kernel.org>
---
.../devicetree/bindings/net/microchip,lan966x-switch.yaml | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
index 306ef9ecf2b9..0f0f35865ef4 100644
--- a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
+++ b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
@@ -17,7 +17,7 @@ description: |
properties:
$nodename:
- pattern: "^switch@[0-9a-f]+$"
+ pattern: "^(ethernet-)?switch@[0-9a-f]+$"
compatible:
const: microchip,lan966x-switch
@@ -70,7 +70,7 @@ properties:
additionalProperties: false
patternProperties:
- "^port@[0-9a-f]+$":
+ "^(ethernet-)?port@[0-9a-f]+$":
type: object
$ref: /schemas/net/ethernet-controller.yaml#
@@ -138,7 +138,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
- switch: switch@e0000000 {
+ switch: ethernet-switch@e0000000 {
compatible = "microchip,lan966x-switch";
reg = <0xe0000000 0x0100000>,
<0xe2000000 0x0800000>;
@@ -151,14 +151,14 @@ examples:
#address-cells = <1>;
#size-cells = <0>;
- port0: port@0 {
+ port0: ethernet-port@0 {
reg = <0>;
phy-handle = <&phy0>;
phys = <&serdes 0 0>;
phy-mode = "gmii";
};
- port1: port@1 {
+ port1: ethernet-port@1 {
reg = <1>;
sfp = <&sfp_eth1>;
managed = "in-band-status";
---
base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731
change-id: 20260507-lan966-binding-0df62a018509
Best regards,
--
Linus Walleij <linusw@kernel.org>
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
2026-05-07 9:26 [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes Linus Walleij
@ 2026-05-07 11:40 ` Herve Codina
2026-05-07 11:59 ` Rob Herring
2026-05-07 15:41 ` Rob Herring (Arm)
2026-05-08 22:20 ` patchwork-bot+netdevbpf
2 siblings, 1 reply; 6+ messages in thread
From: Herve Codina @ 2026-05-07 11:40 UTC (permalink / raw)
To: Linus Walleij
Cc: Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Horatiu Vultur, netdev, devicetree
Hi Linus,
On Thu, 07 May 2026 11:26:01 +0200
Linus Walleij <linusw@kernel.org> wrote:
> The dsa.yaml and ethernet-switch.yaml bindings recommend
> prefixing ethernet switches and ports with "ethernet-" so
> make the LAN966x do the same.
>
> Reported-by: Herve Codina <herve.codina@bootlin.com>
> Signed-off-by: Linus Walleij <linusw@kernel.org>
> ---
> .../devicetree/bindings/net/microchip,lan966x-switch.yaml | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> index 306ef9ecf2b9..0f0f35865ef4 100644
> --- a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> +++ b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> @@ -17,7 +17,7 @@ description: |
>
> properties:
> $nodename:
> - pattern: "^switch@[0-9a-f]+$"
> + pattern: "^(ethernet-)?switch@[0-9a-f]+$"
>
> compatible:
> const: microchip,lan966x-switch
> @@ -70,7 +70,7 @@ properties:
> additionalProperties: false
>
> patternProperties:
> - "^port@[0-9a-f]+$":
> + "^(ethernet-)?port@[0-9a-f]+$":
> type: object
>
> $ref: /schemas/net/ethernet-controller.yaml#
> @@ -138,7 +138,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> - switch: switch@e0000000 {
> + switch: ethernet-switch@e0000000 {
> compatible = "microchip,lan966x-switch";
> reg = <0xe0000000 0x0100000>,
> <0xe2000000 0x0800000>;
> @@ -151,14 +151,14 @@ examples:
> #address-cells = <1>;
> #size-cells = <0>;
>
> - port0: port@0 {
> + port0: ethernet-port@0 {
> reg = <0>;
> phy-handle = <&phy0>;
> phys = <&serdes 0 0>;
> phy-mode = "gmii";
> };
>
> - port1: port@1 {
> + port1: ethernet-port@1 {
> reg = <1>;
> sfp = <&sfp_eth1>;
> managed = "in-band-status";
>
> ---
> base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731
> change-id: 20260507-lan966-binding-0df62a018509
>
> Best regards,
> --
> Linus Walleij <linusw@kernel.org>
>
With those changes, dtb_check will not be happy when following dtsi/dts are
involved:
- arch/arm/boot/dts/microchip/lan966x.dtsi
- arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt.dtsi
- arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
- arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
- arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-8g.dts
IMHO they should be updated as part of the series.
Best regards,
Hervé
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
2026-05-07 11:40 ` Herve Codina
@ 2026-05-07 11:59 ` Rob Herring
2026-05-07 12:13 ` Herve Codina
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2026-05-07 11:59 UTC (permalink / raw)
To: Herve Codina
Cc: Linus Walleij, Andrew Lunn, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni, Krzysztof Kozlowski, Conor Dooley,
Horatiu Vultur, netdev, devicetree
On Thu, May 07, 2026 at 01:40:43PM +0200, Herve Codina wrote:
> Hi Linus,
>
> On Thu, 07 May 2026 11:26:01 +0200
> Linus Walleij <linusw@kernel.org> wrote:
>
> > The dsa.yaml and ethernet-switch.yaml bindings recommend
> > prefixing ethernet switches and ports with "ethernet-" so
> > make the LAN966x do the same.
> >
> > Reported-by: Herve Codina <herve.codina@bootlin.com>
> > Signed-off-by: Linus Walleij <linusw@kernel.org>
> > ---
> > .../devicetree/bindings/net/microchip,lan966x-switch.yaml | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > index 306ef9ecf2b9..0f0f35865ef4 100644
> > --- a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > +++ b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > @@ -17,7 +17,7 @@ description: |
> >
> > properties:
> > $nodename:
> > - pattern: "^switch@[0-9a-f]+$"
> > + pattern: "^(ethernet-)?switch@[0-9a-f]+$"
> >
> > compatible:
> > const: microchip,lan966x-switch
> > @@ -70,7 +70,7 @@ properties:
> > additionalProperties: false
> >
> > patternProperties:
> > - "^port@[0-9a-f]+$":
> > + "^(ethernet-)?port@[0-9a-f]+$":
> > type: object
> >
> > $ref: /schemas/net/ethernet-controller.yaml#
> > @@ -138,7 +138,7 @@ additionalProperties: false
> > examples:
> > - |
> > #include <dt-bindings/interrupt-controller/arm-gic.h>
> > - switch: switch@e0000000 {
> > + switch: ethernet-switch@e0000000 {
> > compatible = "microchip,lan966x-switch";
> > reg = <0xe0000000 0x0100000>,
> > <0xe2000000 0x0800000>;
> > @@ -151,14 +151,14 @@ examples:
> > #address-cells = <1>;
> > #size-cells = <0>;
> >
> > - port0: port@0 {
> > + port0: ethernet-port@0 {
> > reg = <0>;
> > phy-handle = <&phy0>;
> > phys = <&serdes 0 0>;
> > phy-mode = "gmii";
> > };
> >
> > - port1: port@1 {
> > + port1: ethernet-port@1 {
> > reg = <1>;
> > sfp = <&sfp_eth1>;
> > managed = "in-band-status";
> >
> > ---
> > base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731
> > change-id: 20260507-lan966-binding-0df62a018509
> >
> > Best regards,
> > --
> > Linus Walleij <linusw@kernel.org>
> >
>
> With those changes, dtb_check will not be happy when following dtsi/dts are
> involved:
> - arch/arm/boot/dts/microchip/lan966x.dtsi
> - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt.dtsi
> - arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
> - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
> - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-8g.dts
How so? the added prefix is optional.
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
2026-05-07 11:59 ` Rob Herring
@ 2026-05-07 12:13 ` Herve Codina
0 siblings, 0 replies; 6+ messages in thread
From: Herve Codina @ 2026-05-07 12:13 UTC (permalink / raw)
To: Rob Herring
Cc: Linus Walleij, Andrew Lunn, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni, Krzysztof Kozlowski, Conor Dooley,
Horatiu Vultur, netdev, devicetree
Hi Rob, Linus,
On Thu, 7 May 2026 06:59:35 -0500
Rob Herring <robh@kernel.org> wrote:
> On Thu, May 07, 2026 at 01:40:43PM +0200, Herve Codina wrote:
> > Hi Linus,
> >
> > On Thu, 07 May 2026 11:26:01 +0200
> > Linus Walleij <linusw@kernel.org> wrote:
> >
> > > The dsa.yaml and ethernet-switch.yaml bindings recommend
> > > prefixing ethernet switches and ports with "ethernet-" so
> > > make the LAN966x do the same.
> > >
> > > Reported-by: Herve Codina <herve.codina@bootlin.com>
> > > Signed-off-by: Linus Walleij <linusw@kernel.org>
> > > ---
> > > .../devicetree/bindings/net/microchip,lan966x-switch.yaml | 10 +++++-----
> > > 1 file changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > > index 306ef9ecf2b9..0f0f35865ef4 100644
> > > --- a/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > > +++ b/Documentation/devicetree/bindings/net/microchip,lan966x-switch.yaml
> > > @@ -17,7 +17,7 @@ description: |
> > >
> > > properties:
> > > $nodename:
> > > - pattern: "^switch@[0-9a-f]+$"
> > > + pattern: "^(ethernet-)?switch@[0-9a-f]+$"
> > >
> > > compatible:
> > > const: microchip,lan966x-switch
> > > @@ -70,7 +70,7 @@ properties:
> > > additionalProperties: false
> > >
> > > patternProperties:
> > > - "^port@[0-9a-f]+$":
> > > + "^(ethernet-)?port@[0-9a-f]+$":
> > > type: object
> > >
> > > $ref: /schemas/net/ethernet-controller.yaml#
> > > @@ -138,7 +138,7 @@ additionalProperties: false
> > > examples:
> > > - |
> > > #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > - switch: switch@e0000000 {
> > > + switch: ethernet-switch@e0000000 {
> > > compatible = "microchip,lan966x-switch";
> > > reg = <0xe0000000 0x0100000>,
> > > <0xe2000000 0x0800000>;
> > > @@ -151,14 +151,14 @@ examples:
> > > #address-cells = <1>;
> > > #size-cells = <0>;
> > >
> > > - port0: port@0 {
> > > + port0: ethernet-port@0 {
> > > reg = <0>;
> > > phy-handle = <&phy0>;
> > > phys = <&serdes 0 0>;
> > > phy-mode = "gmii";
> > > };
> > >
> > > - port1: port@1 {
> > > + port1: ethernet-port@1 {
> > > reg = <1>;
> > > sfp = <&sfp_eth1>;
> > > managed = "in-band-status";
> > >
> > > ---
> > > base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731
> > > change-id: 20260507-lan966-binding-0df62a018509
> > >
> > > Best regards,
> > > --
> > > Linus Walleij <linusw@kernel.org>
> > >
> >
> > With those changes, dtb_check will not be happy when following dtsi/dts are
> > involved:
> > - arch/arm/boot/dts/microchip/lan966x.dtsi
> > - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt.dtsi
> > - arch/arm/boot/dts/microchip/lan966x-pcb8290.dts
> > - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
> > - arch/arm/boot/dts/microchip/lan966x-kontron-kswitch-d10-mmt-8g.dts
>
> How so? the added prefix is optional.
Oups, I've missed that. My bad.
Reviewed-by: Herve Codina <herve.codina@bootlin.com>
Sorry for the noise.
Best regards,
Hervé
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
2026-05-07 9:26 [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes Linus Walleij
2026-05-07 11:40 ` Herve Codina
@ 2026-05-07 15:41 ` Rob Herring (Arm)
2026-05-08 22:20 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 6+ messages in thread
From: Rob Herring (Arm) @ 2026-05-07 15:41 UTC (permalink / raw)
To: Linus Walleij
Cc: Andrew Lunn, Jakub Kicinski, Eric Dumazet, Krzysztof Kozlowski,
Herve Codina, Paolo Abeni, Horatiu Vultur, devicetree,
Conor Dooley, David S. Miller, netdev
On Thu, 07 May 2026 11:26:01 +0200, Linus Walleij wrote:
> The dsa.yaml and ethernet-switch.yaml bindings recommend
> prefixing ethernet switches and ports with "ethernet-" so
> make the LAN966x do the same.
>
> Reported-by: Herve Codina <herve.codina@bootlin.com>
> Signed-off-by: Linus Walleij <linusw@kernel.org>
> ---
> .../devicetree/bindings/net/microchip,lan966x-switch.yaml | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes
2026-05-07 9:26 [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes Linus Walleij
2026-05-07 11:40 ` Herve Codina
2026-05-07 15:41 ` Rob Herring (Arm)
@ 2026-05-08 22:20 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 6+ messages in thread
From: patchwork-bot+netdevbpf @ 2026-05-08 22:20 UTC (permalink / raw)
To: Linus Walleij
Cc: herve.codina, andrew+netdev, davem, edumazet, kuba, pabeni, robh,
krzk+dt, conor+dt, horatiu.vultur, netdev, devicetree
Hello:
This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:
On Thu, 07 May 2026 11:26:01 +0200 you wrote:
> The dsa.yaml and ethernet-switch.yaml bindings recommend
> prefixing ethernet switches and ports with "ethernet-" so
> make the LAN966x do the same.
>
> Reported-by: Herve Codina <herve.codina@bootlin.com>
> Signed-off-by: Linus Walleij <linusw@kernel.org>
>
> [...]
Here is the summary with links:
- dt-bindings: net: lan966x: Accept standard ethernet prefixes
https://git.kernel.org/netdev/net-next/c/4374fc0b89ff
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-05-08 22:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-07 9:26 [PATCH] dt-bindings: net: lan966x: Accept standard ethernet prefixes Linus Walleij
2026-05-07 11:40 ` Herve Codina
2026-05-07 11:59 ` Rob Herring
2026-05-07 12:13 ` Herve Codina
2026-05-07 15:41 ` Rob Herring (Arm)
2026-05-08 22:20 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox