Re: [PATCH next-next] net: phy: realtek: add support for rtl8224 2.5Gbps PHY

From: Andrew Lunn
Date: Tue Jun 11 2024 - 09:21:32 EST


On Tue, Jun 11, 2024 at 05:34:14PM +1200, Chris Packham wrote:
> The Realtek RTL8224 PHY is a 2.5Gbps capable PHY. It only uses the
> clause 45 MDIO interface and can leverage the support that has already
> been added for the other 822x PHYs.
>
> Signed-off-by: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx>

You probably should Cc: Eric Woudstra and Marek Behún who have both
worked on 2.5G variants of this PHY.

> Notes:
> I'm currently testing this on an older kernel because the board I'm
> using has a SOC/DSA switch that has a driver in openwrt for Linux 5.15.
> I have tried to selectively back port the bits I need from the other
> rtl822x work so this should be all that is required for the rtl8224.
>
> There's quite a lot that would need forward porting get a working system
> against a current kernel so hopefully this is small enough that it can
> land while I'm trying to figure out how to untangle all the other bits.

I don't see this as being a problem. It should not be possible to
cause regressions by adding a new device like this. If it turns out to
be broken, you can fix it up later.

Andrew