net: dsa: remove useless args of dsa_cpu_dsa_setup
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Sat, 5 Aug 2017 20:20:18 +0000 (16:20 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Aug 2017 21:22:42 +0000 (14:22 -0700)
dsa_cpu_dsa_setup currently takes 4 arguments but they are all available
from the dsa_port argument. Remove all others.

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dsa/dsa.c
net/dsa/dsa2.c
net/dsa/dsa_priv.h
net/dsa/legacy.c

index a91e520e735feea054ffbdee59a7a173b7d691ea..4118848b8e58728d1bf8d471a755aef051cb6fcd 100644 (file)
@@ -67,17 +67,17 @@ const struct dsa_device_ops *dsa_device_ops[DSA_TAG_LAST] = {
        [DSA_TAG_PROTO_NONE] = &none_ops,
 };
 
-int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,
-                     struct dsa_port *dport, int port)
+int dsa_cpu_dsa_setup(struct dsa_port *port)
 {
-       struct device_node *port_dn = dport->dn;
+       struct device_node *port_dn = port->dn;
+       struct dsa_switch *ds = port->ds;
        struct phy_device *phydev;
        int ret, mode;
 
        if (of_phy_is_fixed_link(port_dn)) {
                ret = of_phy_register_fixed_link(port_dn);
                if (ret) {
-                       dev_err(dev, "failed to register fixed PHY\n");
+                       dev_err(ds->dev, "failed to register fixed PHY\n");
                        return ret;
                }
                phydev = of_phy_find_device(port_dn);
@@ -90,7 +90,7 @@ int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,
                genphy_config_init(phydev);
                genphy_read_status(phydev);
                if (ds->ops->adjust_link)
-                       ds->ops->adjust_link(ds, port, phydev);
+                       ds->ops->adjust_link(ds, port->index, phydev);
 
                put_device(&phydev->mdio.dev);
        }
index c442051d5a55732d37ddc18187389d66d8d08bd9..2a0120493cf1b277c861c184219fe5ae3524b71c 100644 (file)
@@ -219,7 +219,7 @@ static int dsa_dsa_port_apply(struct dsa_port *port)
        struct dsa_switch *ds = port->ds;
        int err;
 
-       err = dsa_cpu_dsa_setup(ds, ds->dev, port, port->index);
+       err = dsa_cpu_dsa_setup(port);
        if (err) {
                dev_warn(ds->dev, "Failed to setup dsa port %d: %d\n",
                         port->index, err);
@@ -243,7 +243,7 @@ static int dsa_cpu_port_apply(struct dsa_port *port)
        struct dsa_switch *ds = port->ds;
        int err;
 
-       err = dsa_cpu_dsa_setup(ds, ds->dev, port, port->index);
+       err = dsa_cpu_dsa_setup(port);
        if (err) {
                dev_warn(ds->dev, "Failed to setup cpu port %d: %d\n",
                         port->index, err);
index 306cff229def9d67480508d2eef63518698035cc..2873d912b12ecb9d63fd5d1127f13895bd1f2743 100644 (file)
@@ -101,8 +101,7 @@ struct dsa_slave_priv {
 };
 
 /* dsa.c */
-int dsa_cpu_dsa_setup(struct dsa_switch *ds, struct device *dev,
-                     struct dsa_port *dport, int port);
+int dsa_cpu_dsa_setup(struct dsa_port *port);
 void dsa_cpu_dsa_destroy(struct dsa_port *dport);
 const struct dsa_device_ops *dsa_resolve_tag_protocol(int tag_protocol);
 int dsa_cpu_port_ethtool_setup(struct dsa_port *cpu_dp);
index fa162030a69c430689f0fa3582e9bebdc1007c33..903e8d70c7926de0884152196be69b533d119e16 100644 (file)
@@ -80,15 +80,13 @@ dsa_switch_probe(struct device *parent, struct device *host_dev, int sw_addr,
 /* basic switch operations **************************************************/
 static int dsa_cpu_dsa_setups(struct dsa_switch *ds)
 {
-       struct dsa_port *dport;
        int ret, port;
 
        for (port = 0; port < ds->num_ports; port++) {
                if (!(dsa_is_cpu_port(ds, port) || dsa_is_dsa_port(ds, port)))
                        continue;
 
-               dport = &ds->ports[port];
-               ret = dsa_cpu_dsa_setup(ds, ds->dev, dport, port);
+               ret = dsa_cpu_dsa_setup(&ds->ports[port]);
                if (ret)
                        return ret;
        }