net: dsa: fix dsa_port_vlan_filtering when global
authorVladimir Oltean <vladimir.oltean@nxp.com>
Fri, 15 Jul 2022 15:16:58 +0000 (18:16 +0300)
committerJakub Kicinski <kuba@kernel.org>
Tue, 19 Jul 2022 03:14:23 +0000 (20:14 -0700)
commit4db2a5ef4ccbe6d138828284cfab241b434b5d95
treefd16065cdf104ed7e2991d2ead1622fcbd8c4f1a
parent1e53834ce541d4fe271cdcca7703e50be0a44f8a
net: dsa: fix dsa_port_vlan_filtering when global

The blamed refactoring commit changed a "port" iterator with "other_dp",
but still looked at the slave_dev of the dp outside the loop, instead of
other_dp->slave from the loop.

As a result, dsa_port_vlan_filtering() would not call
dsa_slave_manage_vlan_filtering() except for the port in cause, and not
for all switch ports as expected.

Fixes: d0004a020bb5 ("net: dsa: remove the "dsa_to_port in a loop" antipattern from the core")
Reported-by: Lucian Banu <Lucian.Banu@westermo.com>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/dsa/port.c