RDMA/hns: Remove redundant variable "ret"
authorGuofeng Yue <yueguofeng@hisilicon.com>
Sat, 9 Apr 2022 08:32:52 +0000 (16:32 +0800)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 11 Apr 2022 14:00:34 +0000 (11:00 -0300)
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: Guofeng Yue <yueguofeng@hisilicon.com>
Signed-off-by: Wenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/hns/hns_roce_hw_v2.c

index cd87e6e..aa3eca1 100644 (file)
@@ -3257,7 +3257,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));
@@ -3296,9 +3295,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,