scsi: ufs: qcom: Remove redundant error print for devm_kzalloc() failure
authorManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Fri, 8 Dec 2023 06:58:54 +0000 (12:28 +0530)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 14 Dec 2023 04:03:46 +0000 (23:03 -0500)
devm_kzalloc() will itself print the error message on failure. So let's get
rid of the redundant error message in ufs_qcom_init().

Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20231208065902.11006-10-manivannan.sadhasivam@linaro.org
Reviewed-by: Nitin Rawat <quic_nitirawa@quicinc.com>
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 b141dd2a9346a0343e3616c2c3f52027dc462254..05a9a25bc34c61955ae68a11968e9a7dde76b40d 100644 (file)
@@ -1109,10 +1109,8 @@ static int ufs_qcom_init(struct ufs_hba *hba)
        struct ufs_clk_info *clki;
 
        host = devm_kzalloc(dev, sizeof(*host), GFP_KERNEL);
-       if (!host) {
-               dev_err(dev, "%s: no memory for qcom ufs host\n", __func__);
+       if (!host)
                return -ENOMEM;
-       }
 
        /* Make a two way bind between the qcom host and the hba */
        host->hba = hba;