dsa: Remove phydev parameter from disable_port call
authorAndrew Lunn <andrew@lunn.ch>
Sun, 24 Feb 2019 19:44:43 +0000 (20:44 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 25 Feb 2019 06:30:34 +0000 (22:30 -0800)
No current DSA driver makes use of the phydev parameter passed to the
disable_port call. Remove it.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
16 files changed:
drivers/net/dsa/b53/b53_common.c
drivers/net/dsa/b53/b53_priv.h
drivers/net/dsa/bcm_sf2.c
drivers/net/dsa/lan9303-core.c
drivers/net/dsa/lantiq_gswip.c
drivers/net/dsa/microchip/ksz_common.c
drivers/net/dsa/microchip/ksz_common.h
drivers/net/dsa/mt7530.c
drivers/net/dsa/mv88e6xxx/chip.c
drivers/net/dsa/qca8k.c
drivers/net/dsa/rtl8366rb.c
drivers/net/dsa/vitesse-vsc73xx.c
include/net/dsa.h
net/dsa/dsa_priv.h
net/dsa/port.c
net/dsa/slave.c

index c76892ac4e699cf083c74330c398e36b91b86a8c..0852e5e0817721448c5cab9de89423a45181d6f6 100644 (file)
@@ -543,7 +543,7 @@ int b53_enable_port(struct dsa_switch *ds, int port, struct phy_device *phy)
 }
 EXPORT_SYMBOL(b53_enable_port);
 
-void b53_disable_port(struct dsa_switch *ds, int port, struct phy_device *phy)
+void b53_disable_port(struct dsa_switch *ds, int port)
 {
        struct b53_device *dev = ds->priv;
        u8 reg;
@@ -963,7 +963,7 @@ static int b53_setup(struct dsa_switch *ds)
                if (dsa_is_cpu_port(ds, port))
                        b53_enable_cpu_port(dev, port);
                else if (dsa_is_unused_port(ds, port))
-                       b53_disable_port(ds, port, NULL);
+                       b53_disable_port(ds, port);
        }
 
        return ret;
index 4dc7ee38b2580fe8fee4e22e31f505559cd53dd8..e3441dcf2d2132a68b36b7e11f7e9da7f7dc8f10 100644 (file)
@@ -356,7 +356,7 @@ enum dsa_tag_protocol b53_get_tag_protocol(struct dsa_switch *ds, int port);
 void b53_mirror_del(struct dsa_switch *ds, int port,
                    struct dsa_mall_mirror_tc_entry *mirror);
 int b53_enable_port(struct dsa_switch *ds, int port, struct phy_device *phy);
-void b53_disable_port(struct dsa_switch *ds, int port, struct phy_device *phy);
+void b53_disable_port(struct dsa_switch *ds, int port);
 void b53_brcm_hdr_setup(struct dsa_switch *ds, int port);
 void b53_eee_enable_set(struct dsa_switch *ds, int port, bool enable);
 int b53_eee_init(struct dsa_switch *ds, int port, struct phy_device *phy);
index f91b8e77d5439896f91114d789abef7aafcb52bf..c8e3f05e1d729816eb9709611da1db6942c66bb4 100644 (file)
@@ -221,8 +221,7 @@ static int bcm_sf2_port_setup(struct dsa_switch *ds, int port,
        return b53_enable_port(ds, port, phy);
 }
 
-static void bcm_sf2_port_disable(struct dsa_switch *ds, int port,
-                                struct phy_device *phy)
+static void bcm_sf2_port_disable(struct dsa_switch *ds, int port)
 {
        struct bcm_sf2_priv *priv = bcm_sf2_to_priv(ds);
        u32 reg;
@@ -241,7 +240,7 @@ static void bcm_sf2_port_disable(struct dsa_switch *ds, int port,
        if (priv->int_phy_mask & 1 << port && priv->hw_params.num_gphy == 1)
                bcm_sf2_gphy_enable_set(ds, false);
 
-       b53_disable_port(ds, port, phy);
+       b53_disable_port(ds, port);
 
        /* Power down the port memory */
        reg = core_readl(priv, CORE_MEM_PSM_VDD_CTRL);
@@ -692,7 +691,7 @@ static int bcm_sf2_sw_suspend(struct dsa_switch *ds)
         */
        for (port = 0; port < ds->num_ports; port++) {
                if (dsa_is_user_port(ds, port) || dsa_is_cpu_port(ds, port))
-                       bcm_sf2_port_disable(ds, port, NULL);
+                       bcm_sf2_port_disable(ds, port);
        }
 
        return 0;
@@ -788,7 +787,7 @@ static int bcm_sf2_sw_setup(struct dsa_switch *ds)
                else if (dsa_is_cpu_port(ds, port))
                        bcm_sf2_imp_setup(ds, port);
                else
-                       bcm_sf2_port_disable(ds, port, NULL);
+                       bcm_sf2_port_disable(ds, port);
        }
 
        b53_configure_vlan(ds);
index b4f6e1a67dd9a1197f16b72ae0e412ba79c9a99c..2ffab7ee3d80d2b776d593bc4189686107f7a642 100644 (file)
@@ -1091,8 +1091,7 @@ static int lan9303_port_enable(struct dsa_switch *ds, int port,
        return lan9303_enable_processing_port(chip, port);
 }
 
-static void lan9303_port_disable(struct dsa_switch *ds, int port,
-                                struct phy_device *phy)
+static void lan9303_port_disable(struct dsa_switch *ds, int port)
 {
        struct lan9303 *chip = ds->priv;
 
index 27d092cab40eff58acd572336bb0cae12461b438..ee1455758764bc5b81c5f99589c72ac63850a729 100644 (file)
@@ -480,8 +480,7 @@ static int gswip_port_enable(struct dsa_switch *ds, int port,
        return 0;
 }
 
-static void gswip_port_disable(struct dsa_switch *ds, int port,
-                              struct phy_device *phy)
+static void gswip_port_disable(struct dsa_switch *ds, int port)
 {
        struct gswip_priv *priv = ds->priv;
 
@@ -549,7 +548,7 @@ static int gswip_setup(struct dsa_switch *ds)
 
        /* disable port fetch/store dma on all ports */
        for (i = 0; i < priv->hw_info->max_ports; i++)
-               gswip_port_disable(ds, i, NULL);
+               gswip_port_disable(ds, i);
 
        /* enable Switch */
        gswip_mdio_mask(priv, 0, GSWIP_MDIO_GLOB_ENABLE, GSWIP_MDIO_GLOB);
index d89c97724c35d45e63b8490a021f9782280ea89a..9328b88849d27fa24fb796838a1f783634e5fa71 100644 (file)
@@ -380,7 +380,7 @@ int ksz_enable_port(struct dsa_switch *ds, int port, struct phy_device *phy)
 }
 EXPORT_SYMBOL_GPL(ksz_enable_port);
 
-void ksz_disable_port(struct dsa_switch *ds, int port, struct phy_device *phy)
+void ksz_disable_port(struct dsa_switch *ds, int port)
 {
        struct ksz_device *dev = ds->priv;
 
index 0d25bc44f4537744700e892943fc673ef64dc1cc..21cd794e18f1f83bc5ed00a3f7648b9fdf6b0c4d 100644 (file)
@@ -35,7 +35,7 @@ void ksz_port_mdb_add(struct dsa_switch *ds, int port,
 int ksz_port_mdb_del(struct dsa_switch *ds, int port,
                     const struct switchdev_obj_port_mdb *mdb);
 int ksz_enable_port(struct dsa_switch *ds, int port, struct phy_device *phy);
-void ksz_disable_port(struct dsa_switch *ds, int port, struct phy_device *phy);
+void ksz_disable_port(struct dsa_switch *ds, int port);
 
 /* Common register access functions */
 
index c2b61500f958a6e71a7aa4b37b72381b9a28d74b..0390c5e8271abf6e1e87d1f7d4d42c56dd5cb800 100644 (file)
@@ -729,8 +729,7 @@ mt7530_port_enable(struct dsa_switch *ds, int port,
 }
 
 static void
-mt7530_port_disable(struct dsa_switch *ds, int port,
-                   struct phy_device *phy)
+mt7530_port_disable(struct dsa_switch *ds, int port)
 {
        struct mt7530_priv *priv = ds->priv;
 
@@ -1301,7 +1300,7 @@ mt7530_setup(struct dsa_switch *ds)
                if (dsa_is_cpu_port(ds, i))
                        mt7530_cpu_port_enable(priv, i);
                else
-                       mt7530_port_disable(ds, i, NULL);
+                       mt7530_port_disable(ds, i);
        }
 
        /* Flush the FDB table */
index d30336f259cef9e306f38d55457021131db5880d..0c944ed2173e3933b4721975738a16f657b2dfdc 100644 (file)
@@ -2388,8 +2388,7 @@ static int mv88e6xxx_port_enable(struct dsa_switch *ds, int port,
        return err;
 }
 
-static void mv88e6xxx_port_disable(struct dsa_switch *ds, int port,
-                                  struct phy_device *phydev)
+static void mv88e6xxx_port_disable(struct dsa_switch *ds, int port)
 {
        struct mv88e6xxx_chip *chip = ds->priv;
 
index 195a8a87b98411ef06d11a29575962fe0ce03830..576b37d12a63ca4ea5064cd568194ec25fa22ee5 100644 (file)
@@ -801,8 +801,7 @@ qca8k_port_enable(struct dsa_switch *ds, int port,
 }
 
 static void
-qca8k_port_disable(struct dsa_switch *ds, int port,
-                  struct phy_device *phy)
+qca8k_port_disable(struct dsa_switch *ds, int port)
 {
        struct qca8k_priv *priv = (struct qca8k_priv *)ds->priv;
 
index a4d5049df6928281edaddb198457c064989e3657..40b3974970c686388f9505e08ccd6fa949b9350c 100644 (file)
@@ -1073,8 +1073,7 @@ rtl8366rb_port_enable(struct dsa_switch *ds, int port,
 }
 
 static void
-rtl8366rb_port_disable(struct dsa_switch *ds, int port,
-                      struct phy_device *phy)
+rtl8366rb_port_disable(struct dsa_switch *ds, int port)
 {
        struct realtek_smi *smi = ds->priv;
        int ret;
index 9f1b5f2e8a6453ad8773915368fda0e0efb00ad2..d4780610ea8a320696d13d3928156eac4915e9ae 100644 (file)
@@ -1013,8 +1013,7 @@ static int vsc73xx_port_enable(struct dsa_switch *ds, int port,
        return 0;
 }
 
-static void vsc73xx_port_disable(struct dsa_switch *ds, int port,
-                                struct phy_device *phy)
+static void vsc73xx_port_disable(struct dsa_switch *ds, int port)
 {
        struct vsc73xx *vsc = ds->priv;
 
index 2c2c10812814e7d400f60c9c865b0f554195c278..e8ac5b35ac4aaaf506d86c74364c817494e2baf8 100644 (file)
@@ -364,8 +364,7 @@ struct dsa_switch_ops {
         */
        int     (*port_enable)(struct dsa_switch *ds, int port,
                               struct phy_device *phy);
-       void    (*port_disable)(struct dsa_switch *ds, int port,
-                               struct phy_device *phy);
+       void    (*port_disable)(struct dsa_switch *ds, int port);
 
        /*
         * Port's MAC EEE settings
index 47a1d1379d159800712e5efa675a20cea5356b8a..c6caa58c5c7176559228af58a35f93ec589c9b10 100644 (file)
@@ -143,7 +143,7 @@ static inline struct net_device *dsa_master_find_slave(struct net_device *dev,
 int dsa_port_set_state(struct dsa_port *dp, u8 state,
                       struct switchdev_trans *trans);
 int dsa_port_enable(struct dsa_port *dp, struct phy_device *phy);
-void dsa_port_disable(struct dsa_port *dp, struct phy_device *phy);
+void dsa_port_disable(struct dsa_port *dp);
 int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br);
 void dsa_port_bridge_leave(struct dsa_port *dp, struct net_device *br);
 int dsa_port_vlan_filtering(struct dsa_port *dp, bool vlan_filtering,
index 6b114b045162cf8a890b47e34d4d5d83982ba67c..a2dad10646cb6b420681578cd87686eb7354aee1 100644 (file)
@@ -85,7 +85,7 @@ int dsa_port_enable(struct dsa_port *dp, struct phy_device *phy)
        return 0;
 }
 
-void dsa_port_disable(struct dsa_port *dp, struct phy_device *phy)
+void dsa_port_disable(struct dsa_port *dp)
 {
        struct dsa_switch *ds = dp->ds;
        int port = dp->index;
@@ -94,7 +94,7 @@ void dsa_port_disable(struct dsa_port *dp, struct phy_device *phy)
                dsa_port_set_state_now(dp, BR_STATE_DISABLED);
 
        if (ds->ops->port_disable)
-               ds->ops->port_disable(ds, port, phy);
+               ds->ops->port_disable(ds, port);
 }
 
 int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br)
index 90629b12beaf7a2e7e6d165af205d64c57a1b54e..7274499293c960ae4bbdec97015c20242c223e44 100644 (file)
@@ -122,7 +122,7 @@ static int dsa_slave_close(struct net_device *dev)
 
        phylink_stop(dp->pl);
 
-       dsa_port_disable(dp, dev->phydev);
+       dsa_port_disable(dp);
 
        dev_mc_unsync(master, dev);
        dev_uc_unsync(master, dev);