Commit 62f3b70e authored by Xinhao Liu's avatar Xinhao Liu Committed by Jason Gunthorpe
Browse files

RDMA/hns: Clear redundant variable initialization 

There is no need to initialize some variable because they will be assigned
with a value later.

Link: https://lore.kernel.org/r/1607650657-35992-9-git-send-email-liweihang@huawei.com


Signed-off-by: default avatarXinhao Liu <liuxinhao5@hisilicon.com>
Signed-off-by: default avatarWeihang Li <liweihang@huawei.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent dc93a0d9

drivers/infiniband/hw/hns/hns_roce_hem.c

+1 −1
Original line number Diff line number Diff line
@@ -887,7 +887,7 @@ int hns_roce_init_hem_table(struct hns_roce_dev *hr_dev, 
		unsigned long buf_chunk_size;

		unsigned long bt_chunk_size;

		unsigned long bt_chunk_num;

		unsigned long num_bt_l0 = 0;

		unsigned long num_bt_l0;

		u32 hop_num;



		if (get_hem_table_config(hr_dev, &mhop, type))

drivers/infiniband/hw/hns/hns_roce_hw_v1.c

+4 −4
Original line number Diff line number Diff line
@@ -353,8 +353,8 @@ static int hns_roce_v1_post_recv(struct ib_qp *ibqp, 
	unsigned long flags = 0;

	unsigned int wqe_idx;

	int ret = 0;

	int nreq = 0;

	int i = 0;

	int nreq;

	int i;

	u32 reg_val;



	spin_lock_irqsave(&hr_qp->rq.lock, flags);
@@ -2300,7 +2300,7 @@ int hns_roce_v1_poll_cq(struct ib_cq *ibcq, int num_entries, struct ib_wc *wc) 
	struct hns_roce_qp *cur_qp = NULL;

	unsigned long flags;

	int npolled;

	int ret = 0;

	int ret;



	spin_lock_irqsave(&hr_cq->lock, flags);
@@ -4123,7 +4123,7 @@ static int hns_roce_v1_create_eq(struct hns_roce_dev *hr_dev, 
	void __iomem *eqc = hr_dev->eq_table.eqc_base[eq->eqn];

	struct device *dev = &hr_dev->pdev->dev;

	dma_addr_t tmp_dma_addr;

	u32 eqcuridx_val = 0;

	u32 eqcuridx_val;

	u32 eqconsindx_val;

	u32 eqshift_val;

	__le32 tmp2 = 0;

CRA Git | Maintained and supported by SUSTech CRA and CCSE