net: dsa: b53: fix flushing old pvid VLAN on pvid change
authorJonas Gorski <jonas.gorski@gmail.com>
Tue, 29 Apr 2025 20:17:03 +0000 (22:17 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 8 May 2025 02:30:34 +0000 (19:30 -0700)
Presumably the intention here was to flush the VLAN of the old pvid, not
the added VLAN again, which we already flushed before.

Fixes: a2482d2ce349 ("net: dsa: b53: Plug in VLAN support")
Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
Tested-by: Florian Fainelli <florian.fainelli@broadcom.com>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
Link: https://patch.msgid.link/20250429201710.330937-5-jonas.gorski@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/dsa/b53/b53_common.c

index 65d74c455c573a8be2fbe14201cac9743ff65025..c67c0b5fbc1bec28c12c645719a2e4afcc9f5d3f 100644 (file)
@@ -1574,7 +1574,7 @@ int b53_vlan_add(struct dsa_switch *ds, int port,
        if (!dsa_is_cpu_port(ds, port) && new_pvid != old_pvid) {
                b53_write16(dev, B53_VLAN_PAGE, B53_VLAN_PORT_DEF_TAG(port),
                            new_pvid);
-               b53_fast_age_vlan(dev, vlan->vid);
+               b53_fast_age_vlan(dev, old_pvid);
        }
 
        return 0;