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

From: Alexander Sverdlin <alexander.sverdlin@gmail.com>
To: Inochi Amaoto <inochiama@gmail.com>,
	Andrew Lunn <andrew+netdev@lunn.ch>,
	 "David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski	 <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Chen Wang	 <unicorn_wang@outlook.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	 Richard Cochran <richardcochran@gmail.com>,
	Yixun Lan <dlan@gentoo.org>,
	Thomas Bonnefille	 <thomas.bonnefille@bootlin.com>,
	Ze Huang <huangze@whut.edu.cn>
Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org,
	sophgo@lists.linux.dev, 	linux-kernel@vger.kernel.org,
	linux-riscv@lists.infradead.org,
	Longbin Li	 <looong.bin@gmail.com>
Subject: Re: [PATCH net-next RFC 0/3] riscv: dts: sophgo: Add ethernet support for cv18xx
Date: Sat, 14 Jun 2025 23:19:34 +0200	[thread overview]
Message-ID: <d3b20a9ce58fa296034fe3aa8b60ecde4c4192f4.camel@gmail.com> (raw)
In-Reply-To: <20250611080709.1182183-1-inochiama@gmail.com>

Hi 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

Have you noticed any problems on the board you are testing on?
I've added the patchset + pre-requisited + the following into my board DT
for Milk-V Duo Module 01 EVB:

&mdio {
       status = "okay";
};

&gmac0 {
       phy-mode = "internal";
       phy-handle = <&internal_ephy>;
       status = "okay";
};

And the PHY is being detected and the Ethernet controller is being instantiated,
but the PHY behaves really strange: LEDs blinking wildly, link status is bogus
100FULL UP even without cable insterted and the real traffic starts to travel
only roughly a minute after the cable has been plugged in.

I could look into it, if Sophgo has documented the internal PHY, just wanted to
sync with you for the case you've seen this already...

> 
>  .../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.

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv