lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 22 Dec 2022 21:04:11 +0100 From: Piotr Raczynski <piotr.raczynski@...el.com> To: Clark Wang <xiaoning.wang@....com> CC: <linux@...linux.org.uk>, <peppe.cavallaro@...com>, <alexandre.torgue@...s.st.com>, <joabreu@...opsys.com>, <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>, <mcoquelin.stm32@...il.com>, <andrew@...n.ch>, <hkallweit1@...il.com>, <netdev@...r.kernel.org>, <linux-stm32@...md-mailman.stormreply.com>, <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 2/2] net: stmmac: resume phy separately before calling stmmac_hw_setup() On Wed, Dec 21, 2022 at 04:01:44PM +0800, Clark Wang wrote: > On some platforms, mac cannot work after resumed from the suspend with WoL > enabled. > > We found the stmmac_hw_setup() when system resumes will called after the > stmmac_mac_link_up(). So the correct values set in stmmac_mac_link_up() > are overwritten by stmmac_core_init() in phylink_resume(). > > So call the new added function in phylink to resume phy fristly. first. > Then can call the stmmac_hw_setup() before calling phy_resume(). It'd be nice to add Fixes tag with appropriate commit id. Other than that looks fine. Reviewed-by: Piotr Raczynski <piotr.raczynski@...el.com> > > Signed-off-by: Clark Wang <xiaoning.wang@....com> > --- > .../net/ethernet/stmicro/stmmac/stmmac_main.c | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > index c6951c976f5d..d0bdc9b6dbe8 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > @@ -7532,16 +7532,9 @@ int stmmac_resume(struct device *dev) > } > > rtnl_lock(); > - if (device_may_wakeup(priv->device) && priv->plat->pmt) { > - phylink_resume(priv->phylink); > - } else { > - phylink_resume(priv->phylink); > - if (device_may_wakeup(priv->device)) > - phylink_speed_up(priv->phylink); > - } > - rtnl_unlock(); > > - rtnl_lock(); > + phylink_phy_resume(priv->phylink); > + > mutex_lock(&priv->lock); > > stmmac_reset_queues_param(priv); > @@ -7559,6 +7552,11 @@ int stmmac_resume(struct device *dev) > stmmac_enable_all_dma_irq(priv); > > mutex_unlock(&priv->lock); > + > + phylink_resume(priv->phylink); > + if (device_may_wakeup(priv->device) && !priv->plat->pmt) > + phylink_speed_up(priv->phylink); > + > rtnl_unlock(); > > netif_device_attach(ndev); > -- > 2.34.1 >
Powered by blists - more mailing lists