ramips: Use doubletagging to disable ramips_esw vlan by default, it seems more reliable.
authorGabor Juhos <juhosg@openwrt.org>
Wed, 5 Sep 2012 20:08:42 +0000 (20:08 +0000)
committerGabor Juhos <juhosg@openwrt.org>
Wed, 5 Sep 2012 20:08:42 +0000 (20:08 +0000)
commit736245fe46f7d068e9f400d4b9715d531c93ba11
tree62092f6a6a31fc6365c91da132ef2c6f23e8645f
parent17ecf018587c6f5bb95c07773eb585aae7aab53a
ramips: Use doubletagging to disable ramips_esw vlan by default, it seems more reliable.

Use doubletagging to disable ramips_esw vlan by default, it seems
more reliable.

Daniel Golle found an issue where sometimes (possibly only for
RT3352) the default vlan disable method (clearing en_vlan, untag,
doubletag and putting all ports into vlan 0) doesn't work and the
packets get sent out vlan-tagged with vlan 0.

Instead switch to using the doubletagging method (allow doubletagged
packets, put all ports into vlan 0 with untag enabled) by default.

Unless someone figures out a way to really globally disable vlan for
this switch, this seems like the best (most reliable) option.

I did some tests regarding maximum packet size and did not see any

difference between the two methods, both allow for slightly bigger packets
than the ramips_main.c ethernet driver (ping stops going through
above "ping -s 1472" (1514 bytes), on the switch packets are recv_good until
"ping -s 1490", or about 1532 bytes).

Signed-off-by: Tobias Diedrich <ranma+openwrt@tdiedrich.de>
SVN-Revision: 33321
target/linux/ramips/files/drivers/net/ethernet/ramips/ramips_esw.c