net: dsa: mv88e6xxx: call vtu_getnext directly in vlan_add
authorVivien Didelot <vivien.didelot@gmail.com>
Thu, 1 Aug 2019 18:36:37 +0000 (14:36 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 1 Aug 2019 20:43:09 +0000 (16:43 -0400)
commitb1ac6fb440d6abf77ae82667b57126c6059c7bd6
tree999b497b6dbe89b138c9d63ec6fba230a78a2ff4
parent5210989283c28b909e13efb61c0e298c178bb76f
net: dsa: mv88e6xxx: call vtu_getnext directly in vlan_add

Wrapping mv88e6xxx_vtu_getnext makes the code less easy to read and
_mv88e6xxx_port_vlan_add is the only function requiring the preparation
of a new VLAN entry.

To simplify things up, remove the mv88e6xxx_vtu_get wrapper and
explicit the VLAN lookup in _mv88e6xxx_port_vlan_add. This rework
also avoids programming the broadcast entries again when changing a
port's membership, e.g. from tagged to untagged.

At the same time, rename the helper using an old underscore convention.

Signed-off-by: Vivien Didelot <vivien.didelot@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mv88e6xxx/chip.c