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

Inochi Amaoto posted 4 patches 3 months, 1 week ago
.../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
[PATCH net-next RFC v4 0/4] riscv: dts: sophgo: Add ethernet support for cv18xx
Posted by Inochi Amaoto 3 months, 1 week ago
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
Re: [PATCH net-next RFC v4 0/4] riscv: dts: sophgo: Add ethernet support for cv18xx
Posted by Inochi Amaoto 3 months, 1 week ago
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.

Regards,
Inochi
Re: [PATCH net-next RFC v4 0/4] riscv: dts: sophgo: Add ethernet support for cv18xx
Posted by Krzysztof Kozlowski 3 months, 1 week ago
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
Re: [PATCH net-next RFC v4 0/4] riscv: dts: sophgo: Add ethernet support for cv18xx
Posted by Inochi Amaoto 3 months, 1 week ago
On Tue, Jul 01, 2025 at 08:26:31AM +0200, Krzysztof Kozlowski wrote:
> 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.

I just want to make them all get reviewed, as the dts patch is
related to the binding.
 
> The subject prefix is here not correct and probably this should be
> split. Anyway, bindings have issues, so no, it cannot be merged.
> 

Thanks, I will separate the binding patch and fix the issue.

Regards,
Inochi