Hi Konrad,

On 29-10-2025 10:36, Konrad Dybcio wrote:
On 10/28/25 5:40 PM, Luca Weiss wrote:
Add the msm8953 CCI device string compatible.

Signed-off-by: Luca Weiss <[email protected]>
---
  Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml 
b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
index 9bc99d736343..ef8f5fe3a8e1 100644
--- a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
+++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
@@ -15,6 +15,7 @@ properties:
      oneOf:
        - enum:
            - qcom,msm8226-cci
+          - qcom,msm8953-cci
            - qcom,msm8974-cci
            - qcom,msm8996-cci
@@ -146,6 +147,7 @@ allOf:
              - contains:
                  enum:
                    - qcom,msm8916-cci
+                  - qcom,msm8953-cci
- const: qcom,msm8996-cci

Sorry for only bringing it up now - could you check whether
all the clocks that you defined are actually necessary?

It may be that you can do without CAMSS_AHB_CLK and/or
TOP_AHB_CLK

Removing either GCC_CAMSS_AHB_CLK or GCC_CAMSS_TOP_AHB_CLK from dts leads to an instant reboot during bootup - presumably during cci (or eeprom) probe.

Regards
Luca


Konrad


Reply via email to