On 03/22/2017 04:02 AM, Roger Quadros wrote: > phy_suspend() doesn't get called as part of phy_stop() for PHYs > using interrupts because the phy state machine is never triggered > after a phy_stop(). > > Explicitly trigger the PHY state machine in phy_stop() so that it can > see the new PHY state (HALTED) and suspend the PHY. > > As most PHYLIB consumers will call phy_stop() with rtnl_lock() held > from ndo_close() so we use don't wait for workqueue cancellation in > phy_trigger_machine() by passing false for the 'sync' argument. Sorry for this long delay in responding. I am not exactly sure if this thing to do here. phy_stop() does not have a requirement to suspend the PHY as it is currently defined. You may want to manually suspend the PHY after a phy_stop() by explicitly calling phy_suspend(). Let me think about it some more. > > Fixes: 3c293f4e08b5 ("net: phy: Trigger state machine on state change and not polling.") > Cc: stable <stable@xxxxxxxxxxxxxxx> # v4.9+ > Signed-off-by: Roger Quadros <rogerq@xxxxxx> > --- > drivers/net/phy/phy.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c > index 49dedf8..ab14e7b 100644 > --- a/drivers/net/phy/phy.c > +++ b/drivers/net/phy/phy.c > @@ -907,6 +907,7 @@ void phy_stop(struct phy_device *phydev) > * of rtnl_lock(), but PHY_HALTED shall guarantee phy_change() > * will not reenable interrupts. > */ > + phy_trigger_machine(phydev, false); > } > EXPORT_SYMBOL(phy_stop); > > -- Florian