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 b6951acc764362416dc4441cf006096c4260b730..f368966cf83abe917aa8c542a5ebd09358871cbf 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 97d86a002a90e9d47e104b4cb35978fd7d855789..e08fc1e4115f153e938b74a75c0c59bb5822b8b6 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: {}