Re: [PATCH V2] clk: qcom: camcc-sc7180: Add parent dependency to all camera GDSCs

From: Bryan O'Donoghue
Date: Fri May 26 2023 - 16:37:47 EST

On 26/05/2023 12:10, Satya Priya Kakitapalli (Temp) wrote:
Hi Stephen,

On 5/11/2023 1:02 AM, Stephen Boyd wrote:
Quoting Taniya Das (2023-05-01 07:29:32)
Camera titan top GDSC is a parent supply to all other camera GDSCs. Titan
top GDSC is required to be enabled before enabling any other camera GDSCs
and it should be disabled only after all other camera GDSCs are disabled.
Ensure this behavior by marking titan top GDSC as parent of all other
camera GDSCs.

Fixes: 15d09e830bbc ("clk: qcom: camcc: Add camera clock controller driver for SC7180")
Signed-off-by: Taniya Das <quic_tdas@xxxxxxxxxxx>
Is something broken right now? The commit text doesn't tell me if we
need to backport this to stable kernels or merge it as soon as possible.
What's the priority of this fix?

Modularization of camx driver triggered this issue now, but there could be some other scenarios which could trigger the same issue on stable kernels.  Hence it needs to be back ported to stable kernels.

So ... I'm 99% sure we need to do this for all titan IP blocks, not just for IFE but BPS, IPE...

I guess I'll spin a series for this.