Re: [PATCH 1/2] net: phy: realtek: add combo mode support for RTL8211FS
From: 万致远
Date: Mon Dec 02 2024 - 23:44:20 EST
On 2024/12/3 11:53, Andrew Lunn wrote:
> No, it needs a lot more work than just that. Spend some time to really
> understand how the marvell driver handles either copper or fibre, and
> assume the Rockchip SDK is poor quality code.
>
I'm currently busy in a new IC project so I might not have much time
working on this patch, would you like to accept another patch about
broadcast PHY address?
> It might also be that the marvell scheme does not work. It will depend
> on how the PHY actually works.
In fact I don't familiar with how this code works and which
function will handle link-state change callback. But yeh, Rockchip SDK
is indeed filled with low-quality code,, although it works.
Maybe I'll get back to this patch several weeks later. I'm sorry for that.
Sincerely,
Zhiyuan Wan