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

Krzysztof Kozlowski
2025-07-01T06:26:41Z

On 01/07/2025 03:23, Inochi Amaoto wrote:
> On Tue, Jul 01, 2025 at 09:17:25AM +0800, Inochi Amaoto wrote:
>> Add device binding and dts for CV18XX series SoC, this dts change series
>> required the reset patch [1] for the dts, which is already taken.
>>
>> [1] https://lore.kernel.org/all/20250617070144.1149926-1-inochiama@gmail.com
>>
>> The patch is marked as RFC as it require reset dts.
>>
>> Change from RFC v3:
>> - https://lore.kernel.org/all/20250626080056.325496-1-inochiama@gmail.com
>> 1. patch 3: change internal phy id from 0 to 1
>>
>> Change from RFC v2:
>> - https://lore.kernel.org/all/20250623003049.574821-1-inochiama@gmail.com
>> 1. patch 1: fix wrong binding title
>> 2. patch 3: fix unmatched mdio bus number
>> 3. patch 4: remove setting phy-mode and phy-handle in board dts and move
>> them into patch 3.
>>
>> Change from RFC v1:
>> - https://lore.kernel.org/all/20250611080709.1182183-1-inochiama@gmail.com
>> 1. patch 3: switch to mdio-mux-mmioreg
>> 2. patch 4: add configuration for Huashan Pi
>>
>> Inochi Amaoto (4):
>> 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
>> riscv: dts: sophgo: Enable ethernet device for Huashan Pi
>>
>> .../bindings/net/sophgo,cv1800b-dwmac.yaml | 113 ++++++++++++++++++
>> arch/riscv/boot/dts/sophgo/cv180x.dtsi | 73 +++++++++++
>> .../boot/dts/sophgo/cv1812h-huashan-pi.dts | 8 ++
>> 3 files changed, 194 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/net/sophgo,cv1800b-dwmac.yaml
>>
>> --
>> 2.50.0
>>
>
> As this is mark as RFC due to the reset dependency, now it is OK
> to merge it as the reset patch is taken and this patch is a minor
> change . I hopeif anyone can take the binding patch so I can take
> the devicetree patches.

I don't understand why you target net-next with your DTS patches. The
subject prefix is here not correct and probably this should be split.
Anyway, bindings have issues, so no, it cannot be merged.

Best regards,
Krzysztof