mlxsw: spectrum_switchdev: Remove redundant check
authorIdo Schimmel <idosch@mellanox.com>
Tue, 16 May 2017 17:38:30 +0000 (19:38 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 17 May 2017 18:06:54 +0000 (14:06 -0400)
Since commit 97c242902c20 ("switchdev: Execute bridge ndos only for
bridge ports") switchdev code checks that port is bridged, so no need to
perform the same check in the driver.

Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c

index 85790d38d2f0d7b6c3967dd5b3724dead9239d39..d9393f7ff79f93b762da4e34a5fe3a931214eea6 100644 (file)
@@ -374,9 +374,6 @@ static int mlxsw_sp_port_attr_br_flags_set(struct mlxsw_sp_port *mlxsw_sp_port,
        unsigned long uc_flood = mlxsw_sp_port->uc_flood ? BR_FLOOD : 0;
        int err;
 
-       if (!mlxsw_sp_port->bridged)
-               return -EINVAL;
-
        if (switchdev_trans_ph_prepare(trans))
                return 0;
 
@@ -796,9 +793,6 @@ static int __mlxsw_sp_port_vlans_add(struct mlxsw_sp_port *mlxsw_sp_port,
        u16 vid, old_pvid;
        int err;
 
-       if (!mlxsw_sp_port->bridged)
-               return -EINVAL;
-
        err = mlxsw_sp_port_fid_join(mlxsw_sp_port, vid_begin, vid_end);
        if (err) {
                netdev_err(dev, "Failed to join FIDs\n");
@@ -1162,9 +1156,6 @@ static int __mlxsw_sp_port_vlans_del(struct mlxsw_sp_port *mlxsw_sp_port,
 {
        u16 vid, pvid;
 
-       if (!mlxsw_sp_port->bridged)
-               return -EINVAL;
-
        mlxsw_sp_port_vid_learning_set(mlxsw_sp_port, vid_begin, vid_end,
                                       false);