Re: [PATCH 01/16] clk: qcom: gcc-sm8250: Tie the CX power domain to controller

On 26-08-19 17:25:57, Konrad Dybcio wrote:
> On 8/19/26 2:51 PM, Abel Vesa wrote:
> > Enable runtime PM for the controller so the common GCC probe path resumes
> > the attached domain while registering clocks, resets and GDSCs.
> > 
> > This lets GDSC consumers propagate their votes through the GCC provider to
> > the CX parent domain.
> 
> Not really, this is rather taken care of by the special-cased
> single-power-domain automatic attachment somewhere in the pm core

Right, my wording was poor. The PM core's automatic attachment creates
the CX-to-GDSC parent-child relationship. Without runtime PM, however,
the GCC provider remains active and prevents CX from powering down.

I'll respin with:

Enable runtime PM for the controller, allowing CX to be powered down
once all GCC consumers have released their clocks and GDSCs.

Thanks for reviewing.

Abel