summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2010-08-31 19:04:02 +0000
committerFelix Fietkau <nbd@openwrt.org>2010-08-31 19:04:02 +0000
commit08755b8858df151970fa88f95ea459b54a5f5556 (patch)
tree53293d0d11da184d32901bff9e4d5323f18e5d8e
parentd23ca9f0f36705d63734cfbe3d47188069243c3f (diff)
downloadmtk-20170518-08755b8858df151970fa88f95ea459b54a5f5556.zip
mtk-20170518-08755b8858df151970fa88f95ea459b54a5f5556.tar.gz
mtk-20170518-08755b8858df151970fa88f95ea459b54a5f5556.tar.bz2
rtl8366_smi: when setting VLAN ports, always initialize the PVID to ensure that the VLAN MC entry gets allocated. Fixes problems with tagged-only ports (#7795)
SVN-Revision: 22856
-rw-r--r--target/linux/generic/files/drivers/net/phy/rtl8366_smi.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/target/linux/generic/files/drivers/net/phy/rtl8366_smi.c b/target/linux/generic/files/drivers/net/phy/rtl8366_smi.c
index 9b70a92..dc83ba7 100644
--- a/target/linux/generic/files/drivers/net/phy/rtl8366_smi.c
+++ b/target/linux/generic/files/drivers/net/phy/rtl8366_smi.c
@@ -946,6 +946,7 @@ int rtl8366_sw_set_vlan_ports(struct switch_dev *dev, struct switch_val *val)
struct switch_port *port;
u32 member = 0;
u32 untag = 0;
+ int err;
int i;
if (!smi->ops->is_vlan_valid(smi, val->port_vlan))
@@ -957,6 +958,14 @@ int rtl8366_sw_set_vlan_ports(struct switch_dev *dev, struct switch_val *val)
if (!(port->flags & BIT(SWITCH_PORT_FLAG_TAGGED)))
untag |= BIT(port->id);
+
+ /*
+ * To ensure that we have a valid MC entry for this VLAN,
+ * initialize the port VLAN ID here.
+ */
+ err = rtl8366_set_pvid(smi, port->id, val->port_vlan);
+ if (err < 0)
+ return err;
}
return rtl8366_set_vlan(smi, val->port_vlan, member, untag, 0);