Thread (25 messages) 25 messages, 8 authors, 2026-01-07

Re: [PATCH v2 1/4] dt-bindings: i2c: qcom-cci: Document sm6150 compatible

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-01-07 08:34:45
Also in: linux-arm-msm, linux-i2c, linux-media, lkml

On 07/01/2026 09:17, Wenmeng Liu wrote:

On 1/7/2026 4:00 PM, Krzysztof Kozlowski wrote:
quoted
On Tue, Jan 06, 2026 at 05:39:53PM +0800, Wenmeng Liu wrote:
quoted
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - qcom,sm6150-cci
+    then:
+      properties:
+        clocks:
+          minItems: 3
+          maxItems: 3
+        clock-names:
+          items:
+            - const: soc_ahb
Same question as before. I did not see any resolution of this in
changelog or commit msg.
Will update commit msg in next version:

For this platform, it uses soc_ahb instead of camnoc_axi.
That was not my question from v1. AHB and AXI are both bus clocks,
meaning for this device the same. You again name the clocks how you
called them in clock controller which is wrong. You name here CLOCK
INPUTS. What is the role of this clock FOR THIS DEVICE?

I already asked to pay attention to this difference.

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help