diff options
author | Ioana Ciornei <ioana.ciornei@nxp.com> | 2019-08-13 15:43:04 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-08-14 17:46:25 +0300 |
commit | e76d3d60e3bb918fffb1af3558fe34cd3ea2116b (patch) | |
tree | 7342844a1a550e45a77e9be37f8de7045c6e2c75 /drivers/staging/fsl-dpaa2 | |
parent | 36ec62b7e9e6d2096f01e2df38b309d24ade1abd (diff) | |
download | linux-e76d3d60e3bb918fffb1af3558fe34cd3ea2116b.tar.xz |
staging: fsl-dpaa2/ethsw: remove redundant VLAN check
The ethsw_add_vlan() function is already called only when the VLAN is
not yet configured on the switch. Remove the redundant check.
Reported-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Link: https://lore.kernel.org/r/1565700187-16048-8-git-send-email-ioana.ciornei@nxp.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/fsl-dpaa2')
-rw-r--r-- | drivers/staging/fsl-dpaa2/ethsw/ethsw.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/staging/fsl-dpaa2/ethsw/ethsw.c b/drivers/staging/fsl-dpaa2/ethsw/ethsw.c index 20519af4d804..b69b2b7be972 100644 --- a/drivers/staging/fsl-dpaa2/ethsw/ethsw.c +++ b/drivers/staging/fsl-dpaa2/ethsw/ethsw.c @@ -34,11 +34,6 @@ static int ethsw_add_vlan(struct ethsw_core *ethsw, u16 vid) .fdb_id = 0, }; - if (ethsw->vlans[vid]) { - dev_err(ethsw->dev, "VLAN already configured\n"); - return -EEXIST; - } - err = dpsw_vlan_add(ethsw->mc_io, 0, ethsw->dpsw_handle, vid, &vcfg); if (err) { |