net: dsa: avoid one dsa_to_port() in dsa_slave_change_mtu

We could retrieve the cpu_dp pointer directly from the "dp" we already
have, no need to resort to dsa_to_port(ds, port).

This change also removes the need for an "int port", so that is also
deleted.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index da234c4b7daadfa65f82b546ebbf881256e875db..3ff8647be619326a52e86fd2de4870f0e886ef0d 100644 (file)
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -1807,10 +1807,9 @@ int dsa_slave_change_mtu(struct net_device *dev, int new_mtu)
        struct net_device *master = dsa_slave_to_master(dev);
        struct dsa_port *dp = dsa_slave_to_port(dev);
        struct dsa_slave_priv *p = netdev_priv(dev);
+       struct dsa_port *cpu_dp = dp->cpu_dp;
        struct dsa_switch *ds = p->dp->ds;
        struct dsa_port *other_dp;
-       struct dsa_port *cpu_dp;
-       int port = p->dp->index;
        int largest_mtu = 0;
        int new_master_mtu;
        int old_master_mtu;
@@ -1843,8 +1842,6 @@ int dsa_slave_change_mtu(struct net_device *dev, int new_mtu)
                        largest_mtu = slave_mtu;
        }
 
-       cpu_dp = dsa_to_port(ds, port)->cpu_dp;
-
        mtu_limit = min_t(int, master->max_mtu, dev->max_mtu);
        old_master_mtu = master->mtu;
        new_master_mtu = largest_mtu + dsa_tag_protocol_overhead(cpu_dp->tag_ops);