]> git.baikalelectronics.ru Git - kernel.git/commit
net: dsa: sja1105: Make vid 1 the default pvid
authorVladimir Oltean <olteanv@gmail.com>
Tue, 25 Jun 2019 23:39:35 +0000 (02:39 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 27 Jun 2019 18:03:21 +0000 (11:03 -0700)
commit4649b9b2ed6aba3da02525074457fe27a72dda6b
tree021f0078eb80ac1acaeec7531a98cc71ea37595f
parentea8d6e3fda1a2b860bc63b04e98b147939107cf7
net: dsa: sja1105: Make vid 1 the default pvid

In SJA1105 there is no concept of 'default values' per se, everything
needs to be driver-supplied through the static configuration tables.

The issue is that the hardware manual says that 'at least the default
untagging VLAN' is mandatory to be provided through the static config.
But VLAN 0 isn't a very good initial pvid - its use is reserved for
priority-tagged frames, and the layers of the stack that care about
those already make sure that this VLAN is installed, as can be seen in
the message below:

  8021q: adding VLAN 0 to HW filter on device swp2

So change the pvid provided through the static configuration to 1, which
matches the bridge core's defaults.

Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/sja1105/sja1105_main.c