| Message ID | 20221130112852.2977816-4-abel.vesa@linaro.org |
|---|---|
| State | Accepted |
| Commit | 7364379d725fc8240a90190dc9da662ada43d9d1 |
| Headers | show |
| Series | clk: qcom: Add support for SM8550 | expand |
On 30.11.2022 12:28, Abel Vesa wrote: > The SM8550 GCC GDSCs need a higher timeout value when polling for status, > so increase it to 1500us, while leaving the delay between disable-enable > sequence for votable gdscs to stay the same. > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > drivers/clk/qcom/gdsc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/clk/qcom/gdsc.c b/drivers/clk/qcom/gdsc.c > index 0f21a8a767ac..9e4d6ce891aa 100644 > --- a/drivers/clk/qcom/gdsc.c > +++ b/drivers/clk/qcom/gdsc.c > @@ -46,6 +46,7 @@ > #define RETAIN_MEM BIT(14) > #define RETAIN_PERIPH BIT(13) > > +#define STATUS_POLL_TIMEOUT_US 1500 > #define TIMEOUT_US 500 > > #define domain_to_gdsc(domain) container_of(domain, struct gdsc, pd) > @@ -107,7 +108,7 @@ static int gdsc_poll_status(struct gdsc *sc, enum gdsc_status status) > do { > if (gdsc_check_status(sc, status)) > return 0; > - } while (ktime_us_delta(ktime_get(), start) < TIMEOUT_US); > + } while (ktime_us_delta(ktime_get(), start) < STATUS_POLL_TIMEOUT_US); > > if (gdsc_check_status(sc, status)) > return 0;
diff --git a/drivers/clk/qcom/gdsc.c b/drivers/clk/qcom/gdsc.c index 0f21a8a767ac..9e4d6ce891aa 100644 --- a/drivers/clk/qcom/gdsc.c +++ b/drivers/clk/qcom/gdsc.c @@ -46,6 +46,7 @@ #define RETAIN_MEM BIT(14) #define RETAIN_PERIPH BIT(13) +#define STATUS_POLL_TIMEOUT_US 1500 #define TIMEOUT_US 500 #define domain_to_gdsc(domain) container_of(domain, struct gdsc, pd) @@ -107,7 +108,7 @@ static int gdsc_poll_status(struct gdsc *sc, enum gdsc_status status) do { if (gdsc_check_status(sc, status)) return 0; - } while (ktime_us_delta(ktime_get(), start) < TIMEOUT_US); + } while (ktime_us_delta(ktime_get(), start) < STATUS_POLL_TIMEOUT_US); if (gdsc_check_status(sc, status)) return 0;
The SM8550 GCC GDSCs need a higher timeout value when polling for status, so increase it to 1500us, while leaving the delay between disable-enable sequence for votable gdscs to stay the same. Signed-off-by: Abel Vesa <abel.vesa@linaro.org> --- drivers/clk/qcom/gdsc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)