mmc: davinci_mmc: Handle error for clk_enable
authorJiasheng Jiang <jiasheng@iscas.ac.cn>
Tue, 8 Mar 2022 07:14:15 +0000 (15:14 +0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Tue, 15 Mar 2022 09:29:53 +0000 (10:29 +0100)
As the potential failure of the clk_enable(),
it should be better to check it and return error
if fails.

Fixes: bbce5802afc5 ("davinci: mmc: updates to suspend/resume implementation")
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Link: https://lore.kernel.org/r/20220308071415.1093393-1-jiasheng@iscas.ac.cn
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/davinci_mmc.c

index 0cf646e..7138dfa 100644 (file)
@@ -1373,8 +1373,12 @@ static int davinci_mmcsd_suspend(struct device *dev)
 static int davinci_mmcsd_resume(struct device *dev)
 {
        struct mmc_davinci_host *host = dev_get_drvdata(dev);
+       int ret;
+
+       ret = clk_enable(host->clk);
+       if (ret)
+               return ret;
 
-       clk_enable(host->clk);
        mmc_davinci_reset_ctrl(host, 0);
 
        return 0;