[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250331090033.281807b2@wsk>
Date: Mon, 31 Mar 2025 09:00:33 +0200
From: Lukasz Majewski <lukma@...x.de>
To: Andrew Lunn <andrew@...n.ch>
Cc: Andrew Lunn <andrew+netdev@...n.ch>, davem@...emloft.net, Eric Dumazet
<edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni
<pabeni@...hat.com>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Shawn Guo
<shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix
Kernel Team <kernel@...gutronix.de>, Fabio Estevam <festevam@...il.com>,
Richard Cochran <richardcochran@...il.com>, netdev@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2 4/4] net: mtip: The L2 switch driver for imx287
Hi Andrew.
> > > > + /* Prevent a state halted on mii error */
> > > > + if (fep->mii_timeout && phy_dev->state == PHY_HALTED) {
> > > > + phy_dev->state = PHY_UP;
> > > > + goto spin_unlock;
> > > > + }
> > >
> > > A MAC driver should not be playing around with the internal state
> > > of phylib.
> >
> > Ok, I've replaced it with PHY API calls (phy_start() and
> > phy_is_started()).
>
> phy_start() and phy_stop() should be used in pairs. It is not good to
> call start more often than stop.
>
> What exactly is going on here? Why would there be MII errors?
>
Exactly.
I've double check it - this can be safely dropped.
> Andrew
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@...x.de
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists