mlxsw: spectrum_switchdev: Remove unused variable
authorIdo Schimmel <idosch@mellanox.com>
Wed, 12 Jul 2017 07:12:54 +0000 (09:12 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 12 Jul 2017 15:15:52 +0000 (08:15 -0700)
Commit 10e23eb299fa ("mlxsw: spectrum: Remove support for bypass bridge
port attributes/vlan set") removed statements that used 'bridge_vlan',
but didn't remove the variable itself resulting in the following warning
with W=1:

warning: variable ‘bridge_vlan’ set but not used
[-Wunused-but-set-variable]

Remove the variable and suppress the warning.

Fixes: 10e23eb299fa ("mlxsw: spectrum: Remove support for bypass bridge port attributes/vlan set")
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 cd89a3e6cd8184c1b80037b8a86e3dc3bc0e3a6d..4733fe918315fadb596976677358a575ea9107a2 100644 (file)
@@ -979,7 +979,6 @@ mlxsw_sp_bridge_port_vlan_add(struct mlxsw_sp_port *mlxsw_sp_port,
 {
        u16 pvid = mlxsw_sp_port_pvid_determine(mlxsw_sp_port, vid, is_pvid);
        struct mlxsw_sp_port_vlan *mlxsw_sp_port_vlan;
-       struct mlxsw_sp_bridge_vlan *bridge_vlan;
        u16 old_pvid = mlxsw_sp_port->pvid;
        int err;
 
@@ -1000,8 +999,6 @@ mlxsw_sp_bridge_port_vlan_add(struct mlxsw_sp_port *mlxsw_sp_port,
        if (err)
                goto err_port_vlan_bridge_join;
 
-       bridge_vlan = mlxsw_sp_bridge_vlan_find(bridge_port, vid);
-
        return 0;
 
 err_port_vlan_bridge_join: