Subject: Re: [RFC v3 1/4] dt-bindings: net: dsa: add MT7628 ESW

From: Daniel Golle <[email-masked]>
Date: Sat, 21 Mar 2026 19:58:13 +0000

On Sat, Mar 21, 2026 at 09:43:37PM +0200, Joris Vaisvila wrote:
> Add bindings for MT7628 SoC's Embedded Switch.
> 
> Signed-off-by: Joris Vaisvila <[email-masked]>
> [...]
> diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7628-esw.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7628-esw.yaml
> new file mode 100644
> index 000000000000..5955c07a1be7
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7628-esw.yaml
> [...]
> +
> +  mdio:
> +    $ref: /schemas/net/mdio.yaml#

Why would anyone define the MDIO bus or PHYs therein?
 - PHY addresses match port addresses
 - Only C22 operations are used
 - No reasonable way to connect any external PHYs
=> relying on ds->user_mii_bus, or even just defining phy_read and phy_read ops
in struct dsa_switch_ops should be sufficient? Unless you need the PHYs in DT,
eg. to define PHY LEDs... But that can then come later once you actually add
support for that.
(netdev maintainer folks: correct me if I'm wrong)

> [...]
> +        ports {
> +            #address-cells = <1>;
> +            #size-cells = <0>;
> +
> +            port@0 {
> +                reg = <0>;
> +                label = "swp0";

Do not add labels in binding example, remove them from
all ports.