clk: qcom: turingcc-qcs404: fix missing resume during probe
authorJohan Hovold <johan+linaro@kernel.org>
Tue, 18 Jul 2023 13:29:02 +0000 (15:29 +0200)
committerBjorn Andersson <andersson@kernel.org>
Tue, 18 Jul 2023 14:58:50 +0000 (07:58 -0700)
Drivers that enable runtime PM must make sure that the controller is
runtime resumed before accessing its registers to prevent the power
domain from being disabled.

Fixes: 892df0191b29 ("clk: qcom: Add QCS404 TuringCC")
Cc: stable@vger.kernel.org # 5.2
Cc: Bjorn Andersson <andersson@kernel.org>
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
Link: https://lore.kernel.org/r/20230718132902.21430-9-johan+linaro@kernel.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/clk/qcom/turingcc-qcs404.c

index 4318445..2cd288d 100644 (file)
@@ -125,11 +125,22 @@ static int turingcc_probe(struct platform_device *pdev)
                return ret;
        }
 
+       ret = pm_runtime_resume_and_get(&pdev->dev);
+       if (ret)
+               return ret;
+
        ret = qcom_cc_probe(pdev, &turingcc_desc);
        if (ret < 0)
-               return ret;
+               goto err_put_rpm;
+
+       pm_runtime_put(&pdev->dev);
 
        return 0;
+
+err_put_rpm:
+       pm_runtime_put_sync(&pdev->dev);
+
+       return ret;
 }
 
 static const struct dev_pm_ops turingcc_pm_ops = {