Re: [PATCH v2 1/4] clk: qcom: gcc-glymur: Move EVA clocks to critical clock list

[Konrad Dybcio]

On 6/17/26 1:07 PM, Taniya Das wrote:
> The gcc_eva_ahb_clk and gcc_eva_xo_clk branch clocks should not be
> registered as standalone GCC branch clocks. Drop these clocks from
> the GCC clock list and instead add their CBCR registers to the GCC
> critical clocks list to ensure they remain enabled during early boot.
> 
> If these clocks are registered as normal branch clocks, they may be
> gated, which breaks access to the EVA clock controller during clock
> controller probe, thus leave them as critical clocks similar to other
> subsystem AHB and XO clocks.
> 
> Fixes: efe504300a17 ("clk: qcom: gcc: Add support for Global Clock Controller")
> Signed-off-by: Taniya Das <taniya.das@xxxxxxxxxxxxxxxx>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>

Konrad