dt-bindings: gpu: samsung: constrain clocks in top-level properties
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Sun, 12 Nov 2023 18:44:01 +0000 (19:44 +0100)
committerRob Herring <robh@kernel.org>
Fri, 8 Dec 2023 13:24:19 +0000 (07:24 -0600)
When number of clock varies between variants, the Devicetree bindings
coding convention expects to have widest constraints in top-level
definition of the properties and narrow them in allOf:if:then block.

This is more readable and sometimes allows to spot some errors in the
bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Link: https://lore.kernel.org/r/20231112184403.3449-3-krzysztof.kozlowski@linaro.org
Signed-off-by: Rob Herring <robh@kernel.org>
Documentation/devicetree/bindings/gpu/samsung-g2d.yaml
Documentation/devicetree/bindings/gpu/samsung-scaler.yaml

index b6951ac..f368966 100644 (file)
@@ -22,8 +22,14 @@ properties:
   interrupts:
     maxItems: 1
 
-  clocks: {}
-  clock-names: {}
+  clocks:
+    minItems: 1
+    maxItems: 2
+
+  clock-names:
+    minItems: 1
+    maxItems: 2
+
   iommus: {}
   power-domains: {}
 
index 97d86a0..e08fc1e 100644 (file)
@@ -21,8 +21,14 @@ properties:
   interrupts:
     maxItems: 1
 
-  clocks: {}
-  clock-names: {}
+  clocks:
+    minItems: 1
+    maxItems: 3
+
+  clock-names:
+    minItems: 1
+    maxItems: 3
+
   iommus: {}
   power-domains: {}