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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <e84c95fa52ead5d6099950400aac9fd38ee1574e.camel@gmail.com>
Date: Sat, 14 Jun 2025 23:52:30 +0200
From: Alexander Sverdlin <alexander.sverdlin@...il.com>
To: Inochi Amaoto <inochiama@...il.com>, Andrew Lunn
 <andrew+netdev@...n.ch>,  "David S. Miller" <davem@...emloft.net>, Eric
 Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo
 Abeni <pabeni@...hat.com>, Rob Herring <robh@...nel.org>, Krzysztof
 Kozlowski	 <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Chen
 Wang	 <unicorn_wang@...look.com>, Paul Walmsley <paul.walmsley@...ive.com>,
 Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>,
 Alexandre Ghiti <alex@...ti.fr>,  Richard Cochran
 <richardcochran@...il.com>, Yixun Lan <dlan@...too.org>, Thomas Bonnefille	
 <thomas.bonnefille@...tlin.com>, Ze Huang <huangze@...t.edu.cn>
Cc: netdev@...r.kernel.org, devicetree@...r.kernel.org,
 sophgo@...ts.linux.dev, 	linux-kernel@...r.kernel.org,
 linux-riscv@...ts.infradead.org, Longbin Li	 <looong.bin@...il.com>
Subject: Re: [PATCH net-next RFC 0/3] riscv: dts: sophgo: Add ethernet
 support for cv18xx

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@gmail.com
> > [2] https://lore.kernel.org/all/20250611075321.1160973-1-inochiama@gmail.com
> > 
> > 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.