clk: qcom: gdsc: Bump parent usage count when GDSC is found enabled
authorBjorn Andersson <bjorn.andersson@linaro.org>
Wed, 13 Jul 2022 21:28:18 +0000 (14:28 -0700)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Mon, 18 Jul 2022 21:35:48 +0000 (16:35 -0500)
When a GDSC is found to be enabled at boot the pm_runtime state will
be unbalanced as the GDSC is later turned off. Fix this by increasing
the usage counter on the power-domain, in line with how we handled the
regulator state.

Fixes: 1b771839de05 ("clk: qcom: gdsc: enable optional power domain support")
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Acked-by: Stephen Boyd <sboyd@kernel.org>
Link: https://lore.kernel.org/r/20220713212818.130277-1-bjorn.andersson@linaro.org
drivers/clk/qcom/gdsc.c

index 6f746158d28fc057fb544e6457e9fa5a9a1578de..d3244006c661a90eb4d00763fefc15cfa6f75142 100644 (file)
@@ -437,6 +437,14 @@ static int gdsc_init(struct gdsc *sc)
                                return ret;
                }
 
+               /* ...and the power-domain */
+               ret = gdsc_pm_runtime_get(sc);
+               if (ret) {
+                       if (sc->rsupply)
+                               regulator_disable(sc->rsupply);
+                       return ret;
+               }
+
                /*
                 * Votable GDSCs can be ON due to Vote from other masters.
                 * If a Votable GDSC is ON, make sure we have a Vote.