On 17/03/2025 17:44, Dmitry Baryshkov wrote:

nit: Subject:

s/qcom,coresight-static-replicator/arm,coresight-static-replicator

As most other CoreSight devices the replicator can use either of the
optional clocks (or both). Document those optional clocks in the schema.

Fixes: 3c15fddf3121 ("dt-bindings: arm: Convert CoreSight bindings to DT 
schema")
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@oss.qualcomm.com>
---
  .../bindings/arm/arm,coresight-static-replicator.yaml          | 10 ++++++++++
  1 file changed, 10 insertions(+)

diff --git 
a/Documentation/devicetree/bindings/arm/arm,coresight-static-replicator.yaml 
b/Documentation/devicetree/bindings/arm/arm,coresight-static-replicator.yaml
index 
a6f793ea03b6c193fc0ff72a45e0249a63a2ba3c..56e64067ed3d63c5e293a0840858f13428bacb45
 100644
--- a/Documentation/devicetree/bindings/arm/arm,coresight-static-replicator.yaml
+++ b/Documentation/devicetree/bindings/arm/arm,coresight-static-replicator.yaml
@@ -30,6 +30,16 @@ properties:
    power-domains:
      maxItems: 1
+ clocks:
+    minItems: 1
+    maxItems: 2
+

For the static replicator, you don't have an APB clock, as they can't be programmed. It may have an ATB clock. So minItems 0, maxItems: 1

Suzuki



+  clock-names:
+    minItems: 1
+    enum:
+      - apb_pclk
+      - atclk
+
    in-ports:
      $ref: /schemas/graph.yaml#/properties/ports
      additionalProperties: false


Reply via email to