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
This commit is contained in:
Felix Fietkau 2010-08-31 19:04:02 +00:00
parent d23ca9f0f3
commit 08755b8858

View file

@ -946,6 +946,7 @@ int rtl8366_sw_set_vlan_ports(struct switch_dev *dev, struct switch_val *val)
struct switch_port *port; struct switch_port *port;
u32 member = 0; u32 member = 0;
u32 untag = 0; u32 untag = 0;
int err;
int i; int i;
if (!smi->ops->is_vlan_valid(smi, val->port_vlan)) 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))) if (!(port->flags & BIT(SWITCH_PORT_FLAG_TAGGED)))
untag |= BIT(port->id); 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); return rtl8366_set_vlan(smi, val->port_vlan, member, untag, 0);