Re: [PATCH 1/4] dt-bindings: display: sun6i-dsi: Fix clock conditional

2022-08-12 Thread Krzysztof Kozlowski 
On 12/08/2022 10:42, Samuel Holland wrote:
> The A64 case should have limited maxItems, instead of duplicating the
> minItems value from the main binding. While here, simplify the binding
> by making this an "else" case of the two-clock conditional block.
> 
> Fixes: fe5040f2843a ("dt-bindings: sun6i-dsi: Document A64 MIPI-DSI 
> controller")
> Signed-off-by: Samuel Holland 


Acked-by: Krzysztof Kozlowski 


Best regards,
Krzysztof

[PATCH 1/4] dt-bindings: display: sun6i-dsi: Fix clock conditional

2022-08-12 Thread Samuel Holland 
The A64 case should have limited maxItems, instead of duplicating the
minItems value from the main binding. While here, simplify the binding
by making this an "else" case of the two-clock conditional block.

Fixes: fe5040f2843a ("dt-bindings: sun6i-dsi: Document A64 MIPI-DSI controller")
Signed-off-by: Samuel Holland 
---

 .../bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml | 10 ++
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git 
a/Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml 
b/Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml
index bf0bdf54e5f9..ae55ef3fb1fe 100644
--- 
a/Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml
+++ 
b/Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml
@@ -78,16 +78,10 @@ allOf:
   required:
 - clock-names
 
-  - if:
-  properties:
-compatible:
-  contains:
-const: allwinner,sun50i-a64-mipi-dsi
-
-then:
+else:
   properties:
 clocks:
-  minItems: 1
+  maxItems: 1
 
 unevaluatedProperties: false
 
-- 
2.35.1