Re: [PATCH 0/4] Add camera clock controller support for SM8550

[Jagadeesh Kona]

Hi Dmitry,

Thanks for your review!

On 5/20/2023 12:59 AM, Dmitry Baryshkov wrote:
> On 19/05/2023 18:55, Jagadeesh Kona wrote:
> > Add bindings, driver and devicetree node for camera clock controller
> > on SM8550.
> >
> > Depends on [1] for lucid ole pll ops definition
> > [1] 
> > https://patchwork.kernel.org/project/linux-clk/list/?series=746186&state=%2A&archive=both
> >
> > Jagadeesh Kona (4):
> >    clk: qcom: clk-alpha-pll: Add support for rivian ole pll ops
> >    dt-bindings: clock: qcom: Add SM8550 camera clock controller
> >    clk: qcom: camcc-sm8550: Add camera clock controller driver for SM8550
>
> This patch didn't make it to the list. Please consider splitting it 
> somehow.
Yes, will split it in next series.

> >    arm64: dts: qcom: sm8550: Add camera clock controller
> >
> >   .../bindings/clock/qcom,sm8550-camcc.yaml     |   82 +
> >   arch/arm64/boot/dts/qcom/sm8550.dtsi          |   15 +
> >   drivers/clk/qcom/Kconfig                      |    7 +
> >   drivers/clk/qcom/Makefile                     |    1 +
> >   drivers/clk/qcom/camcc-sm8550.c               | 3572 +++++++++++++++++
> >   drivers/clk/qcom/clk-alpha-pll.h              |    4 +
> >   include/dt-bindings/clock/qcom,sm8550-camcc.h |  187 +
> >   7 files changed, 3868 insertions(+)
> >   create mode 100644 
> > Documentation/devicetree/bindings/clock/qcom,sm8550-camcc.yaml
> >   create mode 100644 drivers/clk/qcom/camcc-sm8550.c
> >   create mode 100644 include/dt-bindings/clock/qcom,sm8550-camcc.h

Thanks & Regards,
Jagadeesh