Commit 6f4f5cf9 authored by Haoyue Xu's avatar Haoyue Xu Committed by Jason Gunthorpe
Browse files

RDMA/hns: Init the variable at the suitable place 

Assigning a value to ret in the init statement of a for-loop makes the
code less readable.

Link: https://lore.kernel.org/r/20220409083254.9696-6-liangwenpeng@huawei.com


Signed-off-by: default avatarHaoyue Xu <xuhaoyue1@hisilicon.com>
Signed-off-by: default avatarWenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent ac88da75

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+2 −1
Original line number Diff line number Diff line
@@ -1296,7 +1296,8 @@ static int __hns_roce_cmq_send(struct hns_roce_dev *hr_dev, 
	} while (++timeout < priv->cmq.tx_timeout);



	if (hns_roce_cmq_csq_done(hr_dev)) {

		for (ret = 0, i = 0; i < num; i++) {

		ret = 0;

		for (i = 0; i < num; i++) {

			/* check the result of hardware write back */

			desc[i] = csq->desc[tail++];

			if (tail == csq->desc_num)