net: dsa: mv88e6xxx: drop double ds assignment
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Sun, 17 Apr 2016 17:23:56 +0000 (13:23 -0400)
committerDavid S. Miller <davem@davemloft.net>
Sun, 17 Apr 2016 22:54:15 +0000 (18:54 -0400)
Every driver assigns ps->ds even though it gets assigned in the shared
mv88e6xxx_setup_common function. Kill redundancy.

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mv88e6123.c
drivers/net/dsa/mv88e6131.c
drivers/net/dsa/mv88e6171.c
drivers/net/dsa/mv88e6352.c

index 9701c0f..85537eb 100644 (file)
@@ -79,8 +79,6 @@ static int mv88e6123_setup(struct dsa_switch *ds)
        struct mv88e6xxx_priv_state *ps = ds_to_priv(ds);
        int ret;
 
-       ps->ds = ds;
-
        ret = mv88e6xxx_setup_common(ds);
        if (ret < 0)
                return ret;
index fa3a354..4117c9b 100644 (file)
@@ -101,8 +101,6 @@ static int mv88e6131_setup(struct dsa_switch *ds)
        struct mv88e6xxx_priv_state *ps = ds_to_priv(ds);
        int ret;
 
-       ps->ds = ds;
-
        ret = mv88e6xxx_setup_common(ds);
        if (ret < 0)
                return ret;
index 8d86c9e..ae32875 100644 (file)
@@ -76,8 +76,6 @@ static int mv88e6171_setup(struct dsa_switch *ds)
        struct mv88e6xxx_priv_state *ps = ds_to_priv(ds);
        int ret;
 
-       ps->ds = ds;
-
        ret = mv88e6xxx_setup_common(ds);
        if (ret < 0)
                return ret;
index c7fa69c..10c36ab 100644 (file)
@@ -87,8 +87,6 @@ static int mv88e6352_setup(struct dsa_switch *ds)
        struct mv88e6xxx_priv_state *ps = ds_to_priv(ds);
        int ret;
 
-       ps->ds = ds;
-
        ret = mv88e6xxx_setup_common(ds);
        if (ret < 0)
                return ret;