diff options
| author | Wenpeng Liang <liangwenpeng@huawei.com> | 2022-03-02 14:48:28 +0800 |
|---|---|---|
| committer | Jason Gunthorpe <jgg@nvidia.com> | 2022-03-04 17:36:32 -0400 |
| commit | 904de76c42b7d758e75197f8d532a1ffafc3caad (patch) | |
| tree | e3ac110165bc1c5f46c706375c6eff47cb7783b4 /drivers/infiniband/hw/hns/hns_roce_srq.c | |
| parent | cf7f8f5c1c541ece6ec265432c8cf22a6edf067b (diff) | |
| download | linux-904de76c42b7d758e75197f8d532a1ffafc3caad.tar.gz linux-904de76c42b7d758e75197f8d532a1ffafc3caad.tar.bz2 linux-904de76c42b7d758e75197f8d532a1ffafc3caad.zip | |
RDMA/hns: Clean up the return value check of hns_roce_alloc_cmd_mailbox()
hns_roce_alloc_cmd_mailbox() never returns NULL, so the check should be
IS_ERR(). And the error code should be converted as the function's return
value.
Link: https://lore.kernel.org/r/20220302064830.61706-8-liangwenpeng@huawei.com
Signed-off-by: Wenpeng Liang <liangwenpeng@huawei.com>
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Diffstat (limited to 'drivers/infiniband/hw/hns/hns_roce_srq.c')
| -rw-r--r-- | drivers/infiniband/hw/hns/hns_roce_srq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/hns/hns_roce_srq.c b/drivers/infiniband/hw/hns/hns_roce_srq.c index e316276e18c2..97032a357b00 100644 --- a/drivers/infiniband/hw/hns/hns_roce_srq.c +++ b/drivers/infiniband/hw/hns/hns_roce_srq.c @@ -89,9 +89,9 @@ static int alloc_srqc(struct hns_roce_dev *hr_dev, struct hns_roce_srq *srq) } mailbox = hns_roce_alloc_cmd_mailbox(hr_dev); - if (IS_ERR_OR_NULL(mailbox)) { + if (IS_ERR(mailbox)) { ibdev_err(ibdev, "failed to alloc mailbox for SRQC.\n"); - ret = -ENOMEM; + ret = PTR_ERR(mailbox); goto err_xa; } |