Commit 60142049 authored Jul 05, 2022 by Haoyue Xu Committed by Zheng Zengkai Jul 05, 2022

RDMA/hns: Init the variable at the suitable place

mainline inclusion
from mainline-for-linus
commit 6f4f5cf9
category: feature
bugzilla: https://gitee.com/openeuler/kernel/issues/I5CHIG
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git/commit/?id=6f4f5cf9823387acc4f52e3d30f96b879acdff37

----------------------------------------------------------------------

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: Haoyue Xu <xuhaoyue1@hisilicon.com>
Signed-off-by: Wenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Signed-off-by: Zhengfeng Luo <luozhengfeng@h-partners.com>
Reviewed-by: Yangyang Li <liyangyang20@huawei.com>
Reviewed-by: Yue Haibing <yuehaibing@huawei.com>
Signed-off-by: Zheng Zengkai <zhengzengkai@huawei.com>

parent 8d3bc495

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)