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

From: Alexander Sverdlin
Date: Sat Jun 14 2025 - 17:01:49 EST


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.

>
>  .../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
>
>
> base-commit: 19272b37aa4f83ca52bdf9c16d5d81bdd1354494
> prerequisite-patch-id: d5162144180458b11587ebd4ad24e5e3f62b0caf
> prerequisite-patch-id: 9e1992d2ec3c81fbcc463ff7397168fc2acbbf1b
> prerequisite-patch-id: ab3ca8c9cda888f429945fb0283145122975b734
> prerequisite-patch-id: bd94f8bd3d4ce4f3b153cbb36a3896c5dc143c17
> prerequisite-patch-id: 1b73196566058718471def62bc215d2f319513c3
> prerequisite-patch-id: 54157303203826ccf91e985458c4ae7bcdd9b2ba
> --
> 2.49.0

--
Alexander Sverdlin.