Hello! You forgot to CC me. :-/
On 09/21/2018 04:52 PM, Andrew Lunn wrote: > The previous commit to ravb had the side effect of making the PHY > advertise Pause and Asym Pause, which previously did not happen. By > default, phydev->supported has both forms of pause enabled, but > phydev->advertising does not. The new phy_remove_link_mode() copies > phydev->supported to phydev->advertising after removing the requested > link mode. These Pause configuration bits appears it stops the PHY > from completing Auto-Neg and the link remains down. Be explicit and > remove the Pause and Asym Pause modes, so restoring the old behavior. > > Reported-by: Simon Horman <[email protected]> > Fixes: 41124fa64d4b ("net: ethernet: Add helper to remove a supported link > mode") > Signed-off-by: Andrew Lunn <[email protected]> [...] Reviewed-by: Sergei Shtylyov <[email protected]> MBR, Sergei
