scsi: ufs: qcom: Fix the return value when platform_get_resource_byname() fails
authorManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Fri, 8 Dec 2023 06:58:48 +0000 (12:28 +0530)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 14 Dec 2023 04:03:45 +0000 (23:03 -0500)
The return value should be -ENODEV indicating that the resource is not
provided in DT, not -ENOMEM. Fix it!

Fixes: c263b4ef737e ("scsi: ufs: core: mcq: Configure resource regions")
Reviewed-by: Andrew Halaney <ahalaney@redhat.com>
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20231208065902.11006-4-manivannan.sadhasivam@linaro.org
Tested-by: Andrew Halaney <ahalaney@redhat.com> # sa8775p-ride
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/ufs/host/ufs-qcom.c

index 852179e456f22bbb2f1cad2967a9ef3edbc40314..778df0a9c65e09b1661a91146db12f3705c73a09 100644 (file)
@@ -1701,7 +1701,7 @@ static int ufs_qcom_mcq_config_resource(struct ufs_hba *hba)
                if (!res->resource) {
                        dev_info(hba->dev, "Resource %s not provided\n", res->name);
                        if (i == RES_UFS)
-                               return -ENOMEM;
+                               return -ENODEV;
                        continue;
                } else if (i == RES_UFS) {
                        res_mem = res->resource;