net: dsa: rtl8366: Fix VLAN semantics
authorLinus Walleij <linus.walleij@linaro.org>
Sun, 26 Jul 2020 23:34:39 +0000 (01:34 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 29 Jul 2020 00:44:23 +0000 (17:44 -0700)
commit15ab7906cc9290afb006df1bb1074907fbcc7061
tree47e7fa2c0f86b7c4db2313d0c398f9690bc4ce4e
parentb9aaec8f0be518096d1377082e0abe6a85e86ff3
net: dsa: rtl8366: Fix VLAN semantics

The RTL8366 would not handle adding new members (ports) to
a VLAN: the code assumed that ->port_vlan_add() was only
called once for a single port. When intializing the
switch with .configure_vlan_while_not_filtering set to
true, the function is called numerous times for adding
all ports to VLAN1, which was something the code could
not handle.

Alter rtl8366_set_vlan() to just |= new members and
untagged flags to 4k and MC VLAN table entries alike.
This makes it possible to just add new ports to a
VLAN.

Put in some helpful debug code that can be used to find
any further bugs here.

Cc: DENG Qingfang <dqfext@gmail.com>
Cc: Mauri Sandberg <sandberg@mailfence.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Fixes: d8652956cf37 ("net: dsa: realtek-smi: Add Realtek SMI driver")
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/rtl8366.c