OSDN Git Service

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 c76892a..0852e5e 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 4dc7ee3..e3441dc 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 f91b8e7..c8e3f05 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 b4f6e1a..2ffab7e 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 27d092c..ee14557 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 d89c977..9328b88 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 0d25bc4..21cd794 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 c2b6150..0390c5e 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 d30336f..0c944ed 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 195a8a8..576b37d 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 a4d5049..40b3974 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 9f1b5f2..d478061 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 2c2c108..e8ac5b3 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 47a1d13..c6caa58 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 6b114b0..a2dad10 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 90629b1..7274499 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);