From: Jing Xiangfeng <jingxiangf...@huawei.com>

[ Upstream commit 055f15ab2cb4a5cbc4c0a775ef3d0066e0fa9b34 ]

Return PTR_ERR() from the error handling case instead of 0.

Link: https://lore.kernel.org/r/20200910123848.93649-1-jingxiangf...@huawei.com
Signed-off-by: Jing Xiangfeng <jingxiangf...@huawei.com>
Signed-off-by: Martin K. Petersen <martin.peter...@oracle.com>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/scsi/mvumi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/scsi/mvumi.c b/drivers/scsi/mvumi.c
index 8906aceda4c43..0354898d7cac1 100644
--- a/drivers/scsi/mvumi.c
+++ b/drivers/scsi/mvumi.c
@@ -2425,6 +2425,7 @@ static int mvumi_io_attach(struct mvumi_hba *mhba)
        if (IS_ERR(mhba->dm_thread)) {
                dev_err(&mhba->pdev->dev,
                        "failed to create device scan thread\n");
+               ret = PTR_ERR(mhba->dm_thread);
                mutex_unlock(&mhba->sas_discovery_mutex);
                goto fail_create_thread;
        }
-- 
2.25.1

Reply via email to