net: phy: simplify phy_link_change arguments

This function was introduced to allow for different handling of
link up and link down events particularly with regard to the
netif_carrier. The third argument do_carrier allowed the flag to
be left unchanged.

Since then the phylink has introduced an implementation that
completely ignores the third parameter since it never wants to
change the flag and the phylib always sets the third parameter
to true so the flag is always changed.

Therefore the third argument (i.e. do_carrier) is no longer
necessary and can be removed. This also means that the phylib
phy_link_down() function no longer needs its second argument.

Signed-off-by: Doug Berger <opendmb@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index d4bbf79dab6c1864c5a0e04ec1eaac604655738a..d584701187dbd7c0b013ad39e10d8cb208a17b65 100644 (file)
--- a/drivers/net/phy/phy.c
+++ b/drivers/net/phy/phy.c
@@ -58,13 +58,13 @@ static const char *phy_state_to_str(enum phy_state st)
 
 static void phy_link_up(struct phy_device *phydev)
 {
-       phydev->phy_link_change(phydev, true, true);
+       phydev->phy_link_change(phydev, true);
        phy_led_trigger_change_speed(phydev);
 }
 
-static void phy_link_down(struct phy_device *phydev, bool do_carrier)
+static void phy_link_down(struct phy_device *phydev)
 {
-       phydev->phy_link_change(phydev, false, do_carrier);
+       phydev->phy_link_change(phydev, false);
        phy_led_trigger_change_speed(phydev);
 }
 
@@ -524,7 +524,7 @@ int phy_start_cable_test(struct phy_device *phydev,
                goto out;
 
        /* Mark the carrier down until the test is complete */
-       phy_link_down(phydev, true);
+       phy_link_down(phydev);
 
        netif_testing_on(dev);
        err = phydev->drv->cable_test_start(phydev);
@@ -595,7 +595,7 @@ static int phy_check_link_status(struct phy_device *phydev)
                phy_link_up(phydev);
        } else if (!phydev->link && phydev->state != PHY_NOLINK) {
                phydev->state = PHY_NOLINK;
-               phy_link_down(phydev, true);
+               phy_link_down(phydev);
        }
 
        return 0;
@@ -999,7 +999,7 @@ void phy_state_machine(struct work_struct *work)
        case PHY_HALTED:
                if (phydev->link) {
                        phydev->link = 0;
-                       phy_link_down(phydev, true);
+                       phy_link_down(phydev);
                }
                do_suspend = true;
                break;

diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c
index c3a107cf578e142ddd632a58c780d1dfb7ef922e..7481135d27abbf8c382caee96c2c2fc5992347df 100644 (file)
--- a/drivers/net/phy/phy_device.c
+++ b/drivers/net/phy/phy_device.c
@@ -916,16 +916,14 @@ struct phy_device *phy_find_first(struct mii_bus *bus)
 }
 EXPORT_SYMBOL(phy_find_first);
 
-static void phy_link_change(struct phy_device *phydev, bool up, bool do_carrier)
+static void phy_link_change(struct phy_device *phydev, bool up)
 {
        struct net_device *netdev = phydev->attached_dev;
 
-       if (do_carrier) {
-               if (up)
-                       netif_carrier_on(netdev);
-               else
-                       netif_carrier_off(netdev);
-       }
+       if (up)
+               netif_carrier_on(netdev);
+       else
+               netif_carrier_off(netdev);
        phydev->adjust_link(netdev);
        if (phydev->mii_ts && phydev->mii_ts->link_state)
                phydev->mii_ts->link_state(phydev->mii_ts, phydev);

diff --git a/drivers/net/phy/phylink.c b/drivers/net/phy/phylink.c
index 0f23bec431c1c3be8c6181cc04bb5b3a857df37c..b6b1f77bba58016f5f361790ac244f6e6c4ce262 100644 (file)
--- a/drivers/net/phy/phylink.c
+++ b/drivers/net/phy/phylink.c
@@ -803,8 +803,7 @@ void phylink_destroy(struct phylink *pl)
 }
 EXPORT_SYMBOL_GPL(phylink_destroy);
 
-static void phylink_phy_change(struct phy_device *phydev, bool up,
-                              bool do_carrier)
+static void phylink_phy_change(struct phy_device *phydev, bool up)
 {
        struct phylink *pl = phydev->phylink;
        bool tx_pause, rx_pause;

diff --git a/include/linux/phy.h b/include/linux/phy.h
index 5d8ff542801027e587d3e05ddaeaee86ba797d0c..467aa8bf9f6436e06087d4653d4f5b51616dbc65 100644 (file)
--- a/include/linux/phy.h
+++ b/include/linux/phy.h
@@ -543,7 +543,7 @@ struct phy_device {
        u8 mdix;
        u8 mdix_ctrl;
 
-       void (*phy_link_change)(struct phy_device *, bool up, bool do_carrier);
+       void (*phy_link_change)(struct phy_device *phydev, bool up);
        void (*adjust_link)(struct net_device *dev);
 
 #if IS_ENABLED(CONFIG_MACSEC)