From: Wei Yongjun <weiyongj...@huawei.com>

[ Upstream commit d0b1e4a638d670a09f42017a3e567dc846931ba8 ]

Fix to return negative error code -ENOMEM from create_afu error handling
case instead of 0, as done elsewhere in this function.

Link: https://lore.kernel.org/r/20200428141855.88704-1-weiyongj...@huawei.com
Acked-by: Matthew R. Ochs <mro...@linux.ibm.com>
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
Signed-off-by: Martin K. Petersen <martin.peter...@oracle.com>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/scsi/cxlflash/main.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/scsi/cxlflash/main.c b/drivers/scsi/cxlflash/main.c
index f987c40c47a13..443813feaef47 100644
--- a/drivers/scsi/cxlflash/main.c
+++ b/drivers/scsi/cxlflash/main.c
@@ -3749,6 +3749,7 @@ static int cxlflash_probe(struct pci_dev *pdev,
        cfg->afu_cookie = cfg->ops->create_afu(pdev);
        if (unlikely(!cfg->afu_cookie)) {
                dev_err(dev, "%s: create_afu failed\n", __func__);
+               rc = -ENOMEM;
                goto out_remove;
        }
 
-- 
2.25.1



Reply via email to