diff options
| author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2022-04-15 18:46:24 +0300 |
|---|---|---|
| committer | Paolo Abeni <pabeni@redhat.com> | 2022-04-19 11:31:31 +0200 |
| commit | 88abd89c9e1520c3a98ccbc4cc69779ffbad12de (patch) | |
| tree | 99707474fa08ab53b1979564d1795e04b6f52fbd | |
| parent | 0a218c7b7fa3628bead8f78900131f4ef4f784b2 (diff) | |
| download | net-next-88abd89c9e1520c3a98ccbc4cc69779ffbad12de.tar.gz | |
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: Paolo Abeni <pabeni@redhat.com>
| -rw-r--r-- | net/dsa/slave.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c index da234c4b7daad..3ff8647be6193 100644 --- 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); |