The only PHYs that are used with phylink which change their interface
are the BCM84881 and MV88X3310 family, both of which only change their
interface modes on link-up events.  This will break when drivers are
converted to split-PCS.  Fix this.

Reviewed-by: Florian Fainelli <f.faine...@gmail.com>
Signed-off-by: Russell King <rmk+ker...@armlinux.org.uk>
---
 drivers/net/phy/phylink.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/net/phy/phylink.c b/drivers/net/phy/phylink.c
index 1507ea8a9385..f1693ec63366 100644
--- a/drivers/net/phy/phylink.c
+++ b/drivers/net/phy/phylink.c
@@ -629,8 +629,15 @@ static void phylink_resolve(struct work_struct *w)
                                phylink_link_down(pl);
                                cur_link_state = false;
                        }
+                       phylink_pcs_config(pl, false, &link_state);
+                       pl->link_config.interface = link_state.interface;
+               } else {
+                       /* The interface remains unchanged, only the speed,
+                        * duplex or pause settings have changed. Call the
+                        * old mac_config() method to configure the MAC/PCS.
+                        */
+                       phylink_mac_config(pl, &link_state);
                }
-               phylink_mac_config(pl, &link_state);
        }
 
        if (link_state.link != cur_link_state) {
-- 
2.20.1

Reply via email to