Commit 11cf3f0b authored by Guofeng Yue's avatar Guofeng Yue Committed by Zheng Zengkai
Browse files

RDMA/hns: Remove redundant variable "ret" 

mainline inclusion
from mainline-for-linus
commit 601cdd86
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=601cdd861cf551e330c85c4dfa6d25bef3b8d554

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

It is completely redundant for this function to use "ret" to store the
return value of the subfunction.

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


Signed-off-by: default avatarGuofeng Yue <yueguofeng@hisilicon.com>
Signed-off-by: default avatarWenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
Signed-off-by: default avatarZhengfeng Luo <luozhengfeng@h-partners.com>
Reviewed-by: default avatarYangyang Li <liyangyang20@huawei.com>
Reviewed-by: default avatarYue Haibing <yuehaibing@huawei.com>
Signed-off-by: default avatarZheng Zengkai <zhengzengkai@huawei.com>
parent 25d11a4d

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+1 −4
Original line number Diff line number Diff line
@@ -3045,7 +3045,6 @@ static int hns_roce_v2_write_mtpt(struct hns_roce_dev *hr_dev, 
				  void *mb_buf, struct hns_roce_mr *mr)

{

	struct hns_roce_v2_mpt_entry *mpt_entry;

	int ret;



	mpt_entry = mb_buf;

	memset(mpt_entry, 0, sizeof(*mpt_entry));
@@ -3084,9 +3083,7 @@ static int hns_roce_v2_write_mtpt(struct hns_roce_dev *hr_dev, 
		     to_hr_hw_page_shift(mr->pbl_mtr.hem_cfg.ba_pg_shift));

	hr_reg_enable(mpt_entry, MPT_INNER_PA_VLD);



	ret = set_mtpt_pbl(hr_dev, mpt_entry, mr);



	return ret;

	return set_mtpt_pbl(hr_dev, mpt_entry, mr);

}



static int hns_roce_v2_rereg_write_mtpt(struct hns_roce_dev *hr_dev,