Re: [PATCH net-next] net: ks8851: Connect and start/stop the internal PHY

On 1/11/21 3:47 PM, Andrew Lunn wrote:

On Mon, Jan 11, 2021 at 01:53:37PM +0100, Marek Vasut wrote:

quoted

Unless the internal PHY is connected and started, the phylib will not
poll the PHY for state and produce state updates. Connect the PHY and
start/stop it.

Hi Marek

Please continue the conversion and remove all mii_calls.

ks8851_set_link_ksettings() calling mii_ethtool_set_link_ksettings()
is not good, phylib will not know about changes which we made to the
PHY etc.

Hi,

I noticed a couple of drivers implement both the mii and mdiobus 
options, I was pondering why is that. Is there some legacy backward 
compatibility reason for keeping both or is it safe to remove the mii 
support completely from the driver?

Either way, I will do that in a separate patch, so it could be reverted 
if it breaks something.