| Message ID | 20200905121624.32776-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Jason Gunthorpe |
| Headers | show |
| Series | [-next] RDMA/bnxt_re: Remove set but not used variable 'qplib_ctx' | expand |
On Sat, Sep 05, 2020 at 08:16:24PM +0800, YueHaibing wrote: > drivers/infiniband/hw/bnxt_re/main.c:1012:25: > warning: variable ‘qplib_ctx’ set but not used [-Wunused-but-set-variable] > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > drivers/infiniband/hw/bnxt_re/main.c | 2 -- > 1 file changed, 2 deletions(-) The patch is for rdma-rc and not for rdma-next. Fixes: f86b31c6a28f ("RDMA/bnxt_re: Static NQ depth allocation") Thanks, Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
On Sat, Sep 05, 2020 at 08:16:24PM +0800, YueHaibing wrote: > drivers/infiniband/hw/bnxt_re/main.c:1012:25: > warning: variable ‘qplib_ctx’ set but not used [-Wunused-but-set-variable] > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > Reviewed-by: Leon Romanovsky <leonro@nvidia.com> > --- > drivers/infiniband/hw/bnxt_re/main.c | 2 -- > 1 file changed, 2 deletions(-) Applied to for-rc, thanks Jason
diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/bnxt_re/main.c index 13bbeb42794f..53aee5a42ab8 100644 --- a/drivers/infiniband/hw/bnxt_re/main.c +++ b/drivers/infiniband/hw/bnxt_re/main.c @@ -1009,7 +1009,6 @@ static void bnxt_re_free_res(struct bnxt_re_dev *rdev) static int bnxt_re_alloc_res(struct bnxt_re_dev *rdev) { struct bnxt_re_ring_attr rattr = {}; - struct bnxt_qplib_ctx *qplib_ctx; int num_vec_created = 0; int rc = 0, i; u8 type; @@ -1032,7 +1031,6 @@ static int bnxt_re_alloc_res(struct bnxt_re_dev *rdev) if (rc) goto dealloc_res; - qplib_ctx = &rdev->qplib_ctx; for (i = 0; i < rdev->num_msix - 1; i++) { struct bnxt_qplib_nq *nq;
drivers/infiniband/hw/bnxt_re/main.c:1012:25: warning: variable ‘qplib_ctx’ set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/infiniband/hw/bnxt_re/main.c | 2 -- 1 file changed, 2 deletions(-)