]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commitdiff
RDMA/bnxt_re: Remove set but not used variables 'pg' and 'idx'
authorYueHaibing <yuehaibing@huawei.com>
Thu, 27 Feb 2020 06:49:00 +0000 (06:49 +0000)
committerJason Gunthorpe <jgg@mellanox.com>
Mon, 2 Mar 2020 15:10:38 +0000 (11:10 -0400)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/infiniband/hw/bnxt_re/qplib_rcfw.c: In function '__send_message':
drivers/infiniband/hw/bnxt_re/qplib_rcfw.c:101:10: warning:
 variable 'idx' set but not used [-Wunused-but-set-variable]
drivers/infiniband/hw/bnxt_re/qplib_rcfw.c:101:6: warning:
 variable 'pg' set but not used [-Wunused-but-set-variable]

commit cee0c7bba486 ("RDMA/bnxt_re: Refactor command queue management
code") involved this, but not used.

Link: https://lore.kernel.org/r/20200227064900.92255-1-yuehaibing@huawei.com
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/hw/bnxt_re/qplib_rcfw.c

index b0b050e5cd128a83c9429837dcdba44b7c7147b7..f01e864bb6116f6ad4d2d9e29daa7c078dbda24d 100644 (file)
@@ -98,7 +98,6 @@ static int __send_message(struct bnxt_qplib_rcfw *rcfw, struct cmdq_base *req,
        unsigned long flags;
        u32 size, opcode;
        u16 cookie, cbit;
-       int pg, idx;
        u8 *preq;
 
        pdev = rcfw->pdev;
@@ -167,9 +166,6 @@ static int __send_message(struct bnxt_qplib_rcfw *rcfw, struct cmdq_base *req,
        hwq_ptr = (struct bnxt_qplib_cmdqe **)hwq->pbl_ptr;
        preq = (u8 *)req;
        do {
-               pg = 0;
-               idx = 0;
-
                /* Locate the next cmdq slot */
                sw_prod = HWQ_CMP(hwq->prod, hwq);
                cmdqe = &hwq_ptr[get_cmdq_pg(sw_prod, cmdq_depth)]