Re: Re: realtek PHY commit bbc4d71d63549 causes regression

quoted

Sadly, there is one board - Pine64 Plus - where HW settings are wrong and it
actually needs SW override. Until this Realtek PHY driver fix was merged, it
was unclear what magic value provided by Realtek to board manufacturer does.

Reference:
https://lore.kernel.org/netdev/20191001082912.12905-3-icenowy@aosc.io/ (local)

I have merged the fixes from the allwinner tree now, but I still think we
need something better than this, as the current state breaks any existing
dtb file that has the incorrect values, and this really should not have been
considered for backporting to stable kernels.

Hi Arnd

This PHY driver bug hiding DT bug is always hard to handle. We have
been though it once before with the Atheros PHY. All the buggy DT
files were fixed in about one cycle.

Now that we know there is a board which really does want rgmii when it
says rgmii, we cannot simply ignore it in the PHY driver.

But the whole story is muddy because of the backport to stable.  It
might make sense to revert the stable change, and just leave HEAD
broken. That then gives people more time to fix DT blobs. But we have
to consider Pine64 Plus, are we happy breaking that for a while, when
it is actually doing everything correct, and is bug free?

	 Andrew