clk: qcom: gdsc: Update retain_ff sequence and timeout for GDSC

[PATCH 0/2] clk: qcom: gdsc: Update retain_ff sequence and timeout for GDSC

Posted by Taniya Das 10 months, 1 week ago

The retain_ff bit should be updated for a GDSC when it is under SW
control and ON. The current sequence needs to be fixed as the GDSC
needs to update retention and is moved to HW control which does not
guarantee the GDSC to be in enabled state.

During the GDSC FSM state, the GDSC hardware waits for an ACK and the
timeout for the ACK is 2000us as per design requirements.

Signed-off-by: Taniya Das <quic_tdas@quicinc.com>
---
Taniya Das (2):
      clk: qcom: gdsc: Set retain_ff before moving to HW CTRL
      clk: qcom: gdsc: Update the status poll timeout for GDSC

 drivers/clk/qcom/gdsc.c | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)
---
base-commit: c674aa7c289e51659e40dda0f954886ef7f80042
change-id: 20250212-gdsc_fixes-77e8b8e27e2f

Best regards,
-- 
Taniya Das <quic_tdas@quicinc.com>

Re: [PATCH 0/2] clk: qcom: gdsc: Update retain_ff sequence and timeout for GDSC

Posted by Bjorn Andersson 9 months, 1 week ago

On Fri, 14 Feb 2025 09:56:58 +0530, Taniya Das wrote:
> The retain_ff bit should be updated for a GDSC when it is under SW
> control and ON. The current sequence needs to be fixed as the GDSC
> needs to update retention and is moved to HW control which does not
> guarantee the GDSC to be in enabled state.
> 
> During the GDSC FSM state, the GDSC hardware waits for an ACK and the
> timeout for the ACK is 2000us as per design requirements.
> 
> [...]

Applied, thanks!

[1/2] clk: qcom: gdsc: Set retain_ff before moving to HW CTRL
      commit: 25708f73ff171bb4171950c9f4be5aa8504b8459
[2/2] clk: qcom: gdsc: Update the status poll timeout for GDSC
      commit: 172320f5ead5d1a0eed14472ce84146221c75675

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>