clk: qcom: gdsc: Support 'synced_poweroff' genpd flag

Add support for the newly added 'synced_poweroff' genpd flag. This allows
some clients (like adreno gpu driver) to request gdsc driver to ensure
a votable gdsc (like gpucc cx gdsc) has collapsed at hardware.

Signed-off-by: Akhil P Oommen <quic_akhilpo@quicinc.com>
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230102161757.v5.2.Ic128c1df50b7fc9a6b919932a3b41a799b5ed5e8@changeid

diff --git a/drivers/clk/qcom/gdsc.c b/drivers/clk/qcom/gdsc.c
index 9e4d6ce891aad7894af19e63540ea485664bc65d..5358e28122abe48d2b07d05a3794c96069750766 100644 (file)
--- a/drivers/clk/qcom/gdsc.c
+++ b/drivers/clk/qcom/gdsc.c
@@ -136,7 +136,8 @@ static int gdsc_update_collapse_bit(struct gdsc *sc, bool val)
        return 0;
 }
 
-static int gdsc_toggle_logic(struct gdsc *sc, enum gdsc_status status)
+static int gdsc_toggle_logic(struct gdsc *sc, enum gdsc_status status,
+               bool wait)
 {
        int ret;
 
@@ -149,7 +150,7 @@ static int gdsc_toggle_logic(struct gdsc *sc, enum gdsc_status status)
        ret = gdsc_update_collapse_bit(sc, status == GDSC_OFF);
 
        /* If disabling votable gdscs, don't poll on status */
-       if ((sc->flags & VOTABLE) && status == GDSC_OFF) {
+       if ((sc->flags & VOTABLE) && status == GDSC_OFF && !wait) {
                /*
                 * Add a short delay here to ensure that an enable
                 * right after it was disabled does not put it in an
@@ -275,7 +276,7 @@ static int gdsc_enable(struct generic_pm_domain *domain)
                gdsc_deassert_clamp_io(sc);
        }
 
-       ret = gdsc_toggle_logic(sc, GDSC_ON);
+       ret = gdsc_toggle_logic(sc, GDSC_ON, false);
        if (ret)
                return ret;
 
@@ -352,7 +353,7 @@ static int gdsc_disable(struct generic_pm_domain *domain)
        if (sc->pwrsts == PWRSTS_RET_ON)
                return 0;
 
-       ret = gdsc_toggle_logic(sc, GDSC_OFF);
+       ret = gdsc_toggle_logic(sc, GDSC_OFF, domain->synced_poweroff);
        if (ret)
                return ret;
 
@@ -392,7 +393,7 @@ static int gdsc_init(struct gdsc *sc)
 
        /* Force gdsc ON if only ON state is supported */
        if (sc->pwrsts == PWRSTS_ON) {
-               ret = gdsc_toggle_logic(sc, GDSC_ON);
+               ret = gdsc_toggle_logic(sc, GDSC_ON, false);
                if (ret)
                        return ret;
        }