net: dsa: rtl8366: Check VLAN ID and not ports
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 1 Oct 2019 14:28:43 +0000 (16:28 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 7 Oct 2019 16:57:23 +0000 (18:57 +0200)
commit5c08d7e4f1ea4d1288e677e7d43f06c2952a804e
treed3e06463c971bf445cb83686ba0a3edcd8a0feb9
parent3c1f0704401cfa4a6e77fd7f28e9a4460470dbda
net: dsa: rtl8366: Check VLAN ID and not ports

[ Upstream commit e8521e53cca584ddf8ec4584d3c550a6c65f88c4 ]

There has been some confusion between the port number and
the VLAN ID in this driver. What we need to check for
validity is the VLAN ID, nothing else.

The current confusion came from assigning a few default
VLANs for default routing and we need to rewrite that
properly.

Instead of checking if the port number is a valid VLAN
ID, check the actual VLAN IDs passed in to the callback
one by one as expected.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/net/dsa/rtl8366.c