Re: [PATCH v2 0/5] net: stmmac: dwmac-rk: Add GMAC support for RK3528
From: Simon Horman
Date: Tue Mar 18 2025 - 11:31:33 EST
On Mon, Mar 17, 2025 at 08:50:34PM +0100, Jonas Karlman wrote:
> Hi Simon,
>
> On 2025-03-17 20:43, Simon Horman wrote:
> > On Sun, Mar 09, 2025 at 11:26:10PM +0000, Jonas Karlman wrote:
> >> The Rockchip RK3528 has two Ethernet controllers, one 100/10 MAC to be
> >> used with the integrated PHY and a second 1000/100/10 MAC to be used
> >> with an external Ethernet PHY.
> >>
> >> This series add initial support for the Ethernet controllers found in
> >> RK3528 and initial support to power up/down the integrated PHY.
> >>
> >> This series depends on v2 of the "net: stmmac: dwmac-rk: Validate GRF
> >> and peripheral GRF during probe" [1] cleanup series.
> >>
> >>
> >> Changes in v2:
> >> - Restrict the minItems: 4 change to rockchip,rk3528-gmac
> >> - Add initial support to power up/down the integrated PHY in RK3528
> >> - Split device tree changes into a separate series
> >>
> >> [1] https://lore.kernel.org/r/20250308213720.2517944-1-jonas@xxxxxxxxx/
> >
> > Hi Jonas,
> >
> > This patchset looks reasonable to me. However it will need
> > to be reposted once it's dependencies ([1]) are present in net-next.
>
> The dependent series ([1]) has already been merged into net-next [2].
Thanks, and sorry for not noticing that.
> Do I still need to repost this series?
Yes, I think that would be best so there is a CI run over the series
(the CI doesn't understand dependencies).
> [2] https://lore.kernel.org/r/174186063226.1446759.12026198009173732573.git-patchwork-notify@xxxxxxxxxx/
>
> >
> > And on the topic of process:
> >
> > * As this is a patch-set for net-next it would be best to
> > target it accordingly:
> >
> > Subject: [PATCH net-next] ...
> >
> > * Please post patches for net/net-next which have dependencies as RFCs.
> >
> > For more information on Netdev processes please take a look at
> > https://docs.kernel.org/process/maintainer-netdev.html
> >
>
> Thanks, I see, netdev seem to use a slight different process than what
> I am familiar with compared to other Linux subsystems and U-Boot :-)
>
> Regards,
> Jonas
>