[net-next RFC PATCH 0/3] net: phy: detach PHY driver OPs from phy_driver struct
Andrew Lunn
andrew at lunn.ch
Sat Feb 17 15:48:24 PST 2024
> Yes, it was done to limit the patch delta, if I had to account for the
> tab for each new section we would be in the order of 2000+ changes I
> think.
>
> > > 64 files changed, 737 insertions(+), 291 deletions(-)
> >
> > These statistics are not good. If you had deleted more lines than you
> > added, then maybe it might be an O.K. idea.
> >
> > Sometimes KISS is best.
> >
>
> Well IMHO these stats are a bit flawed, the additional code is really
> just extra check if ops is defined and the new .ops variable in each
> phy_driver.
>
> If you check patch 2 and 3 you can already see some code is removed.
Yes, the problem is, it probably needs another 50 patches to remove
all the duplication. I have to question, is that really going to
happen? Are you going to keep working on this until every driver has
its duplicates removed?
It probably needs some tooling to help. Something which can decode the
object file, and tell you which ops structures are identical. That can
then guide you when editing all the PHY drivers.
Andrew
More information about the linux-amlogic
mailing list