Re: [PATCH net-next RFC 0/3] riscv: dts: sophgo: Add ethernet support for cv18xx

[Alexander Sverdlin]

On Sat, 2025-06-14 at 23:01 +0200, Alexander Sverdlin wrote:
> Thanks for the series Inochi!
> 
> On Wed, 2025-06-11 at 16:07 +0800, Inochi Amaoto wrote:
> > Add device binding and dts for CV18XX series SoC, this dts change series
> > require both the mdio patch [1] and the reset patch [2].
> > 
> > [1] https://lore.kernel.org/all/20250611080228.1166090-1-inochiama@xxxxxxxxx
> > [2] https://lore.kernel.org/all/20250611075321.1160973-1-inochiama@xxxxxxxxx
> > 
> > Inochi Amaoto (3):
> >   dt-bindings: net: Add support for Sophgo CV1800 dwmac
> >   riscv: dts: sophgo: Add ethernet device for cv18xx
> >   riscv: dts: sophgo: Add mdio multiplexer device for cv18xx
> 
> Taking into account, whatever MII variation is implemented in the SoC
> is always internal (and only MDIO part is multiplexed), can we add
> 	phy-mode = "internal";
> and
> 	phy-handle = <&internal_ephy>;
> right into cv180x.dtsi?
> 
> Boards can then enable the corresponding nodes if they wire RJ45 connector,
> but I see no way how they could vary the MII connection.

I suppose the above proposal was wrong, though undocumented, there seems
to be an external RMII interface (at least on SG2000 SoC).

Unfortunately the internal PHY is also barely documented...
Also ethtool seems to be incompatible with mdio muxes :(

> >  .../bindings/net/sophgo,cv1800b-dwmac.yaml    | 113 ++++++++++++++++++
> >  arch/riscv/boot/dts/sophgo/cv180x.dtsi        |  70 +++++++++++
> >  2 files changed, 183 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/net/sophgo,cv1800b-dwmac.yaml

-- 
Alexander Sverdlin.