Re: [PATCH net] net: phy: Run state machine to completion

2017-07-28 Thread Florian Fainelli
On 07/28/2017 03:57 AM, Marc Gonzalez wrote: > On 26/07/2017 21:24, Florian Fainelli wrote: > >> Marc reported that he was not getting the PHY library adjust_link() >> callback function to run when calling phy_stop() + phy_disconnect() >> which does not indeed happen because we don't make sure we

Re: [PATCH net] net: phy: Run state machine to completion

2017-07-28 Thread Marc Gonzalez
On 26/07/2017 21:24, Florian Fainelli wrote: > Marc reported that he was not getting the PHY library adjust_link() > callback function to run when calling phy_stop() + phy_disconnect() > which does not indeed happen because we don't make sure we flush the > PHYLIB delayed work and let it run to

Re: [PATCH net] net: phy: Run state machine to completion

2017-07-26 Thread Florian Fainelli
On 07/26/2017 12:34 PM, Florian Fainelli wrote: > On 07/26/2017 12:24 PM, Florian Fainelli wrote: >> Marc reported that he was not getting the PHY library adjust_link() >> callback function to run when calling phy_stop() + phy_disconnect() >> which does not indeed happen because we don't make sure

Re: [PATCH net] net: phy: Run state machine to completion

2017-07-26 Thread Florian Fainelli
On 07/26/2017 12:24 PM, Florian Fainelli wrote: > Marc reported that he was not getting the PHY library adjust_link() > callback function to run when calling phy_stop() + phy_disconnect() > which does not indeed happen because we don't make sure we flush the > PHYLIB delayed work and let it run to

[PATCH net] net: phy: Run state machine to completion

2017-07-26 Thread Florian Fainelli
Marc reported that he was not getting the PHY library adjust_link() callback function to run when calling phy_stop() + phy_disconnect() which does not indeed happen because we don't make sure we flush the PHYLIB delayed work and let it run to completion. We also need to have a synchronous call to