Commit fd28c8a8 authored by Kalesh AP's avatar Kalesh AP Committed by Leon Romanovsky
Browse files

RDMA/bnxt_re: Remove a redundant flag 



After the cited commit, BNXT_RE_FLAG_GOT_MSIX is redundant.
Remove it.

Fixes: 30343221 ("bnxt_en: Remove runtime interrupt vector allocation")
Signed-off-by: default avatarKalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: default avatarSelvin Xavier <selvin.xavier@broadcom.com>
Link: https://lore.kernel.org/r/1691052326-32143-3-git-send-email-selvin.xavier@broadcom.com


Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent f19fba1f

drivers/infiniband/hw/bnxt_re/bnxt_re.h

+0 −1
Original line number Diff line number Diff line
@@ -140,7 +140,6 @@ struct bnxt_re_dev { 
	struct list_head		list;

	unsigned long			flags;

#define BNXT_RE_FLAG_NETDEV_REGISTERED		0

#define BNXT_RE_FLAG_GOT_MSIX			2

#define BNXT_RE_FLAG_HAVE_L2_REF		3

#define BNXT_RE_FLAG_RCFW_CHANNEL_EN		4

#define BNXT_RE_FLAG_QOS_WORK_REG		5

drivers/infiniband/hw/bnxt_re/main.c

+2 −3
Original line number Diff line number Diff line
@@ -1422,7 +1422,7 @@ static void bnxt_re_dev_uninit(struct bnxt_re_dev *rdev) 
		bnxt_re_net_ring_free(rdev, rdev->rcfw.creq.ring_id, type);

		bnxt_qplib_free_rcfw_channel(&rdev->rcfw);

	}

	if (test_and_clear_bit(BNXT_RE_FLAG_GOT_MSIX, &rdev->flags))



	rdev->num_msix = 0;



	if (rdev->pacing.dbr_pacing)
@@ -1480,7 +1480,6 @@ static int bnxt_re_dev_init(struct bnxt_re_dev *rdev, u8 wqe_mode) 
	ibdev_dbg(&rdev->ibdev, "Got %d MSI-X vectors\n",

		  rdev->en_dev->ulp_tbl->msix_requested);

	rdev->num_msix = rdev->en_dev->ulp_tbl->msix_requested;

	set_bit(BNXT_RE_FLAG_GOT_MSIX, &rdev->flags);



	bnxt_re_query_hwrm_intf_version(rdev);