Commit 97a2ae0f authored by Amit Cohen's avatar Amit Cohen Committed by David S. Miller

mlxsw: spectrum_fid: Pass FID structure to mlxsw_sp_fid_op()

The function gets several arguments derived from the FID structure
itself. In the future, it will need to be extended to configure
additional FID attributes.

Prepare for that change and reduce the arguments list by passing the FID
structure itself.
Signed-off-by: default avatarAmit Cohen <amcohen@nvidia.com>
Signed-off-by: default avatarIdo Schimmel <idosch@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 893b5c30
...@@ -418,13 +418,13 @@ static enum mlxsw_reg_sfmr_op mlxsw_sp_sfmr_op(bool valid) ...@@ -418,13 +418,13 @@ static enum mlxsw_reg_sfmr_op mlxsw_sp_sfmr_op(bool valid)
MLXSW_REG_SFMR_OP_DESTROY_FID; MLXSW_REG_SFMR_OP_DESTROY_FID;
} }
static int mlxsw_sp_fid_op(struct mlxsw_sp *mlxsw_sp, u16 fid_index, static int mlxsw_sp_fid_op(const struct mlxsw_sp_fid *fid, bool valid)
u16 fid_offset, bool valid)
{ {
struct mlxsw_sp *mlxsw_sp = fid->fid_family->mlxsw_sp;
char sfmr_pl[MLXSW_REG_SFMR_LEN]; char sfmr_pl[MLXSW_REG_SFMR_LEN];
mlxsw_reg_sfmr_pack(sfmr_pl, mlxsw_sp_sfmr_op(valid), fid_index, mlxsw_reg_sfmr_pack(sfmr_pl, mlxsw_sp_sfmr_op(valid), fid->fid_index,
fid_offset); fid->fid_offset);
return mlxsw_reg_write(mlxsw_sp->core, MLXSW_REG(sfmr), sfmr_pl); return mlxsw_reg_write(mlxsw_sp->core, MLXSW_REG(sfmr), sfmr_pl);
} }
...@@ -468,18 +468,14 @@ static void mlxsw_sp_fid_8021d_setup(struct mlxsw_sp_fid *fid, const void *arg) ...@@ -468,18 +468,14 @@ static void mlxsw_sp_fid_8021d_setup(struct mlxsw_sp_fid *fid, const void *arg)
static int mlxsw_sp_fid_8021d_configure(struct mlxsw_sp_fid *fid) static int mlxsw_sp_fid_8021d_configure(struct mlxsw_sp_fid *fid)
{ {
struct mlxsw_sp_fid_family *fid_family = fid->fid_family; return mlxsw_sp_fid_op(fid, true);
return mlxsw_sp_fid_op(fid_family->mlxsw_sp, fid->fid_index,
fid->fid_offset, true);
} }
static void mlxsw_sp_fid_8021d_deconfigure(struct mlxsw_sp_fid *fid) static void mlxsw_sp_fid_8021d_deconfigure(struct mlxsw_sp_fid *fid)
{ {
if (fid->vni_valid) if (fid->vni_valid)
mlxsw_sp_nve_fid_disable(fid->fid_family->mlxsw_sp, fid); mlxsw_sp_nve_fid_disable(fid->fid_family->mlxsw_sp, fid);
mlxsw_sp_fid_op(fid->fid_family->mlxsw_sp, fid->fid_index, mlxsw_sp_fid_op(fid, false);
fid->fid_offset, false);
} }
static int mlxsw_sp_fid_8021d_index_alloc(struct mlxsw_sp_fid *fid, static int mlxsw_sp_fid_8021d_index_alloc(struct mlxsw_sp_fid *fid,
...@@ -916,15 +912,12 @@ static void mlxsw_sp_fid_dummy_setup(struct mlxsw_sp_fid *fid, const void *arg) ...@@ -916,15 +912,12 @@ static void mlxsw_sp_fid_dummy_setup(struct mlxsw_sp_fid *fid, const void *arg)
static int mlxsw_sp_fid_dummy_configure(struct mlxsw_sp_fid *fid) static int mlxsw_sp_fid_dummy_configure(struct mlxsw_sp_fid *fid)
{ {
struct mlxsw_sp *mlxsw_sp = fid->fid_family->mlxsw_sp; return mlxsw_sp_fid_op(fid, true);
return mlxsw_sp_fid_op(mlxsw_sp, fid->fid_index, fid->fid_offset, true);
} }
static void mlxsw_sp_fid_dummy_deconfigure(struct mlxsw_sp_fid *fid) static void mlxsw_sp_fid_dummy_deconfigure(struct mlxsw_sp_fid *fid)
{ {
mlxsw_sp_fid_op(fid->fid_family->mlxsw_sp, fid->fid_index, mlxsw_sp_fid_op(fid, false);
fid->fid_offset, false);
} }
static int mlxsw_sp_fid_dummy_index_alloc(struct mlxsw_sp_fid *fid, static int mlxsw_sp_fid_dummy_index_alloc(struct mlxsw_sp_fid *fid,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment