Commit 8b6af3b5 authored by Akhil P Oommen's avatar Akhil P Oommen Committed by Bjorn Andersson
Browse files

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: default avatarAkhil P Oommen <quic_akhilpo@quicinc.com>
Reviewed-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230102161757.v5.2.Ic128c1df50b7fc9a6b919932a3b41a799b5ed5e8@changeid
parent a9236a0a

drivers/clk/qcom/gdsc.c

+6 −5
Original line number Diff line number Diff line
@@ -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;

	}