diff options
author | Florian Fainelli <f.fainelli@gmail.com> | 2020-03-30 23:38:47 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-03-31 04:55:42 +0200 |
commit | d7a0b1f7652f9f6b7ba0c9d8ad8edd6b8c0c1511 (patch) | |
tree | 14a6e50e49688cdea6c1730f2aa076fdabcccf52 /drivers/net | |
parent | net: dsa: bcm_sf2: Fix overflow checks (diff) | |
download | linux-d7a0b1f7652f9f6b7ba0c9d8ad8edd6b8c0c1511.tar.xz linux-d7a0b1f7652f9f6b7ba0c9d8ad8edd6b8c0c1511.zip |
net: dsa: b53: Restore VLAN entries upon (re)configuration
The first time b53_configure_vlan() is called we have not configured any
VLAN entries yet, since that happens later when interfaces get brought
up. When b53_configure_vlan() is called again from suspend/resume we
need to restore all VLAN entries though.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/dsa/b53/b53_common.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c index 39ae4ed87d1d..5cb678e8b9cd 100644 --- a/drivers/net/dsa/b53/b53_common.c +++ b/drivers/net/dsa/b53/b53_common.c @@ -681,7 +681,9 @@ int b53_configure_vlan(struct dsa_switch *ds) { struct b53_device *dev = ds->priv; struct b53_vlan vl = { 0 }; + struct b53_vlan *v; int i, def_vid; + u16 vid; def_vid = b53_default_pvid(dev); @@ -699,6 +701,19 @@ int b53_configure_vlan(struct dsa_switch *ds) b53_write16(dev, B53_VLAN_PAGE, B53_VLAN_PORT_DEF_TAG(i), def_vid); + /* Upon initial call we have not set-up any VLANs, but upon + * system resume, we need to restore all VLAN entries. + */ + for (vid = def_vid; vid < dev->num_vlans; vid++) { + v = &dev->vlans[vid]; + + if (!v->members) + continue; + + b53_set_vlan_entry(dev, vid, v); + b53_fast_age_vlan(dev, vid); + } + return 0; } EXPORT_SYMBOL(b53_configure_vlan); |