Messages in this thread | | | Date | Fri, 26 May 2023 21:37:38 +0100 | Subject | Re: [PATCH V2] clk: qcom: camcc-sc7180: Add parent dependency to all camera GDSCs | From | Bryan O'Donoghue <> |
| |
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@quicinc.com> >> 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.
--- bod
| |