RDMA/bnxt_re: Remove incorrect return check from slow path
authorKashyap Desai <kashyap.desai@broadcom.com>
Fri, 16 Jun 2023 06:16:59 +0000 (11:46 +0530)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 26 Jun 2023 12:44:06 +0000 (09:44 -0300)
commitc8dce4e7438be24be7a5b8477555ba03c0fb16ae
treee3b270eb422c7c520487b2224fe4a0e46dc248af
parent360da60d6c6edb9740de7a8e6d8969d62ceff956
RDMA/bnxt_re: Remove incorrect return check from slow path

The commit 691eb7c6110f ("RDMA/bnxt_re: handle command completions after
driver detect a timedout") introduced code resulting in below warning
issued by the smatch static checker.

        drivers/infiniband/hw/bnxt_re/qplib_rcfw.c:513 __bnxt_qplib_rcfw_send_message()
        warn: duplicate check 'rc' (previous on line 506)

Fix the warning by removing incorrect code block.

Fixes: 691eb7c6110f ("RDMA/bnxt_re: handle command completions after driver detect a timedout")
Link: https://lore.kernel.org/r/20230616061700.741769-1-kashyap.desai@broadcom.com
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Kashyap Desai <kashyap.desai@broadcom.com>
Signed-off-by: Selvin Xavier <selvin.xavier@broadcom.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/bnxt_re/qplib_rcfw.c