Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 47a0a9e6 authored by Ido Schimmel's avatar Ido Schimmel Committed by David S. Miller
Browse files

mlxsw: spectrum: Remove redundant function argument 



In all call sites 'only_uc' is set to false, so strip it.

Signed-off-by: default avatarIdo Schimmel <idosch@mellanox.com>
Signed-off-by: default avatarJiri Pirko <jiri@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent d8651fd8

drivers/net/ethernet/mellanox/mlxsw/spectrum.c

+5 −7
Original line number Diff line number Diff line
@@ -791,7 +791,7 @@ int mlxsw_sp_port_add_vid(struct net_device *dev, __be16 __always_unused proto, 


	if (!vfid->nr_vports) {

		err = mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid,

					       true, false);

					       true);

		if (err) {

			netdev_err(dev, "Failed to setup flooding for vFID=%d\n",

				   vfid->vfid);
@@ -859,8 +859,7 @@ int mlxsw_sp_port_add_vid(struct net_device *dev, __be16 __always_unused proto, 
		mlxsw_sp_port_vlan_mode_trans(mlxsw_sp_port);

err_port_vp_mode_trans:

	if (!vfid->nr_vports)

		mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false,

					 false);

		mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false);

err_vport_flood_set:

	mlxsw_sp_port_vport_destroy(mlxsw_sp_vport);

err_port_vport_create:
@@ -3267,8 +3266,7 @@ static void mlxsw_sp_vport_bridge_leave(struct mlxsw_sp_port *mlxsw_sp_vport, 
		goto err_port_vid_learning_set;

	}



	err = mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false,

				       false);

	err = mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false);

	if (err) {

		netdev_err(dev, "Failed clear to clear flooding\n");

		goto err_vport_flood_set;
@@ -3327,7 +3325,7 @@ static int mlxsw_sp_vport_bridge_join(struct mlxsw_sp_port *mlxsw_sp_vport, 
		}

	}



	err = mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, true, false);

	err = mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, true);

	if (err) {

		netdev_err(dev, "Failed to setup flooding for vFID=%d\n",

			   vfid->vfid);
@@ -3386,7 +3384,7 @@ static int mlxsw_sp_vport_bridge_join(struct mlxsw_sp_port *mlxsw_sp_vport, 
err_port_vid_to_fid_invalidate:

	mlxsw_sp_port_vid_learning_set(mlxsw_sp_vport, vid, false);

err_port_vid_learning_set:

	mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false, false);

	mlxsw_sp_vport_flood_set(mlxsw_sp_vport, vfid->vfid, false);

err_port_flood_set:

	if (!vfid->nr_vports)

		mlxsw_sp_br_vfid_destroy(mlxsw_sp, vfid);

drivers/net/ethernet/mellanox/mlxsw/spectrum.h

+1 −1
Original line number Diff line number Diff line
@@ -367,7 +367,7 @@ int mlxsw_sp_port_add_vid(struct net_device *dev, __be16 __always_unused proto, 
int mlxsw_sp_port_kill_vid(struct net_device *dev,

			   __be16 __always_unused proto, u16 vid);

int mlxsw_sp_vport_flood_set(struct mlxsw_sp_port *mlxsw_sp_vport, u16 vfid,

			     bool set, bool only_uc);

			     bool set);

void mlxsw_sp_port_active_vlans_del(struct mlxsw_sp_port *mlxsw_sp_port);

int mlxsw_sp_port_pvid_set(struct mlxsw_sp_port *mlxsw_sp_port, u16 vid);

int mlxsw_sp_port_ets_set(struct mlxsw_sp_port *mlxsw_sp_port,

drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c

+2 −2
Original line number Diff line number Diff line
@@ -261,13 +261,13 @@ static int mlxsw_sp_port_uc_flood_set(struct mlxsw_sp_port *mlxsw_sp_port, 
}



int mlxsw_sp_vport_flood_set(struct mlxsw_sp_port *mlxsw_sp_vport, u16 vfid,

			     bool set, bool only_uc)

			     bool set)

{

	/* In case of vFIDs, index into the flooding table is relative to

	 * the start of the vFIDs range.

	 */

	return __mlxsw_sp_port_flood_set(mlxsw_sp_vport, vfid, vfid, set,

					 only_uc);

					 false);

}



static int mlxsw_sp_port_attr_br_flags_set(struct mlxsw_sp_port *mlxsw_sp_port,