Re: [PATCH V2 RESEND 4/6] dt-bindings: clock: qcom: Add SM8650 camera clock controller

[Jagadeesh Kona <quic_jkona@xxxxxxxxxxx>]

On 3/21/2024 3:50 PM, Johan Hovold wrote:
> On Thu, Mar 21, 2024 at 02:55:27PM +0530, Jagadeesh Kona wrote:
> > Add device tree bindings for the camera clock controller on
> > Qualcomm SM8650 platform.
> >
> > Signed-off-by: Jagadeesh Kona <quic_jkona@xxxxxxxxxxx>
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> Did Krzysztof really review this version?

Thanks Johan for your review.

I carry forwarded the R-B tag from earlier version. Will fix the below 
comments and post the next series.

Thanks,
Jagadeesh

> > @@ -18,6 +19,7 @@ description: |
> >       include/dt-bindings/clock/qcom,sm8550-camcc.h
> >       include/dt-bindings/clock/qcom,sc8280xp-camcc.h
> >       include/dt-bindings/clock/qcom,x1e80100-camcc.h
> > +    include/dt-bindings/clock/qcom,sm8650-camcc.h
>
> This does not look like alphabetical order.
>
> >   allOf:
> >     - $ref: qcom,gcc.yaml#
> > @@ -29,6 +31,7 @@ properties:
> >         - qcom,sm8450-camcc
> >         - qcom,sm8550-camcc
> >         - qcom,x1e80100-camcc
> > +      - qcom,sm8650-camcc
>
> And neither does this.
>
> Johan