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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux