On Wed, 3 Feb 2021 at 14:53, Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> wrote: > > - Adds some missing indices to the videocc index > - Adds videocc clock tree hirearchy for msv0 > - Adds a regulator to power the videocc consistent with downstream > > Once done we can move on to enabling these clocks in the DTS and switch on > the venus. > > Bryan O'Donoghue (4): > dt-bindings: clock: Add missing SM8250 videoc clock indices > clk: qcom: videocc: Add sm8250 VIDEO_CC_MVS0_DIV_CLK_SRC > clk: qcom: videocc: Add sm8250 VIDEO_CC_MVS0_CLK > clk: qcom: videocc: Add gdsc mmcx-reg supply hook I remember now the old discussion about these two clocks (https://lore.kernel.org/linux-arm-msm/160092826778.310579.12225989905897101118@xxxxxxxxxxxxxxxxxxxxxxxxxx/). Is the message. reported by Jonathan, solved by the mmcx-reg supply? Also are these clocks necessary at all? -- With best wishes Dmitry
