drivers: net: cpsw: Add default vlan for dual emac case also
authorMugunthan V N <mugunthanvnm@ti.com>
Wed, 9 Apr 2014 06:04:40 +0000 (11:34 +0530)
committerDavid S. Miller <davem@davemloft.net>
Wed, 9 Apr 2014 18:55:37 +0000 (14:55 -0400)
Dual EMAC works with VLAN segregation of the ports, so default vlan needs
to be added in dual EMAC case else default vlan will be tagged for all
egress packets and vlan unaware switches/servers will drop packets
from the EVM.

Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
Tested-by: Yegor Yefremov <yegorslists@googlemail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ti/cpsw.c

index 5d5fec6c4eb04bbdacf851635394893055a27866..6133e4af997519293f648d512b0150d7da9e712b 100644 (file)
@@ -1201,8 +1201,7 @@ static int cpsw_ndo_open(struct net_device *ndev)
        for_each_slave(priv, cpsw_slave_open, priv);
 
        /* Add default VLAN */
-       if (!priv->data.dual_emac)
-               cpsw_add_default_vlan(priv);
+       cpsw_add_default_vlan(priv);
 
        if (!cpsw_common_res_usage_state(priv)) {
                /* setup tx dma to fixed prio and zero offset */