Re: [PATCH 1/1] net: phy: Do not do CL22 phy reset before ethernet phy driver probe

[Paul Kocialkowski <paulk@sys-base.io>]

[-- Attachment #1: Type: text/plain, Size: 2113 bytes --]

Hi,

Le Mon 14 Oct 24, 10:43, Marek Vasut a écrit :
> On 10/14/24 9:06 AM, Sky Huang wrote:
> > From: "SkyLake.Huang" <skylake.huang@mediatek.com>
> > 
> > Remove unnecessary CL22 phy reset before ethernet phy driver
> > probe. Lots of ethernet phys requires driver to load firmware.
> > Before that, CL22 phy reset may lead to malfunction.
> > 
> > Signed-off-by: SkyLake.Huang <skylake.huang@mediatek.com>
> > ---
> >   drivers/net/phy/phy.c | 2 --
> >   1 file changed, 2 deletions(-)
> > 
> > diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
> > index 716a1d46111..e6fed8c41d7 100644
> > --- a/drivers/net/phy/phy.c
> > +++ b/drivers/net/phy/phy.c
> > @@ -839,8 +839,6 @@ struct phy_device *phy_find_by_mask(struct mii_dev *bus, uint phy_mask)
> >   static void phy_connect_dev(struct phy_device *phydev, struct udevice *dev,
> >   			    phy_interface_t interface)
> >   {
> > -	/* Soft Reset the PHY */
> > -	phy_reset(phydev);
> >   	if (phydev->dev && phydev->dev != dev) {
> >   		printf("%s:%d is connected to %s.  Reconnecting to %s\n",
> >   		       phydev->bus->name, phydev->addr,
> 
> This needs clarification and likely has to be handled differently. Removing
> the reset would leave the PHY in potentially undefined state.
> 
> Which PHY is affected by this ?

Good hunch. I just bisected down to this commit as I had some issues with my
Allwinner V3 board using the internal CL22 PHY. The symptom is that the LEDS's
polarity was inverted. Network still seems to work (although not tested beyond
ping).

This change can have very significant consequences in general, which were not
explored at all in the commit. This may break many boards that do rely on that
PHY reset, in various scenarios and for various reasons.

I think it should be reverted as soon as possible to restore previous behavior.

All the best,

Paul

-- 
Paul Kocialkowski,

Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/

Expert in multimedia, graphics and embedded hardware support with Linux.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]