Re: [PATCH net-next v3 3/8] dt-bindings: net: dsa: Add SoC-e SWIP switch
From: Vasilij Strassheim
Date: Wed Sep 30 2026 - 14:01:06 EST
On Sat, 2026-09-26 at 01:05 +0200, Andrew Lunn wrote:
> > + mdio@204:
> > + $ref: /schemas/net/soce,swip-mdio.yaml#
> > + unevaluatedProperties: false
> > + description:
> > + Integrated MDIO controller bus on the master side of the mux.
> > +
> > + mdio-mux@202:
> > + $ref: /schemas/net/mdio-mux-mmioreg.yaml#
> > + unevaluatedProperties: false
>
> > + mdio_parent: mdio@204 {
> > + compatible = "soce,swip-mdio-23-02";
> > + reg = <0x204 0xc>, <0x200 0x4>;
> > + reg-names = "data", "control";
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + };
> > +
> > + mdio-mux@202 {
> > + compatible = "mdio-mux-mmioreg", "mdio-mux";
> > + reg = <0x202 0x2>;
>
> So these two overlap? That is pretty unusual, so might be worth a comment somewhere.
No, they don't. I accidentally included the entire register in the
documentation. However, this register can be split, since only 2 bytes
are used.
I will fix this in both bindings and their examples:
"reg = <0x204 0xc>, <0x200 0x2>;"
>
> Andrew
Thanks,
Vasilij