Patch 3 should reference 'video_cc_mvs0_clk' rather than 'video_cc_mvs0_div_clk' in the commit message. With that fixed all patches are: Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> Tested-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> 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 > > drivers/clk/qcom/videocc-sm8250.c | 39 +++++++++++++++++++ > .../dt-bindings/clock/qcom,videocc-sm8250.h | 2 + > 2 files changed, 41 insertions(+) > > -- > 2.29.2 > -- With best wishes Dmitry