diff options
author | Jack Morgenstein <jackm@dev.mellanox.co.il> | 2015-01-27 08:58:05 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-01-27 20:12:57 -0500 |
commit | ef0223e693d66bc1edae342e7fe829c1092a653c (patch) | |
tree | f91a24e07956ba6e567681fc42e44f95d2f32a82 | |
parent | dc7d500451277fde0a6fe46bd45249978c8e697e (diff) |
net/mlx4_core: Remove duplicate code line from procedure mlx4_bf_alloc
mlx4_bf_alloc had an unnecessary/duplicate code line. Did no harm,
but not good practice.
Reported by the Mellanox Beijing team.
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Amir Vadai <amirv@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx4/pd.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx4/pd.c b/drivers/net/ethernet/mellanox/mlx4/pd.c index a42b4c0a9ed9..609c59dc854e 100644 --- a/drivers/net/ethernet/mellanox/mlx4/pd.c +++ b/drivers/net/ethernet/mellanox/mlx4/pd.c | |||
@@ -214,7 +214,6 @@ int mlx4_bf_alloc(struct mlx4_dev *dev, struct mlx4_bf *bf, int node) | |||
214 | list_add(&uar->bf_list, &priv->bf_list); | 214 | list_add(&uar->bf_list, &priv->bf_list); |
215 | } | 215 | } |
216 | 216 | ||
217 | bf->uar = uar; | ||
218 | idx = ffz(uar->free_bf_bmap); | 217 | idx = ffz(uar->free_bf_bmap); |
219 | uar->free_bf_bmap |= 1 << idx; | 218 | uar->free_bf_bmap |= 1 << idx; |
220 | bf->uar = uar; | 219 | bf->uar = uar; |