[PATCH v4 2/2] dt-bindings: serial: samsung: fix maxItems for gs101

André Draszik posted 2 patches 1 year, 5 months ago
There is a newer version of this series
[PATCH v4 2/2] dt-bindings: serial: samsung: fix maxItems for gs101
Posted by André Draszik 1 year, 5 months ago
While gs101 needs exactly two clocks for the UART, the schema doesn't
currently limit the maximum number to this and instead the default of
five from this schema is applied.

Update the schema accordingly.

Signed-off-by: André Draszik <andre.draszik@linaro.org>

---
v4:
* drop description from clocks:, it was Linux-specific and a we'll
  implement something that makes it obsolete anyway
* no need to duplicate clock-names anymore
---
 Documentation/devicetree/bindings/serial/samsung_uart.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/serial/samsung_uart.yaml b/Documentation/devicetree/bindings/serial/samsung_uart.yaml
index cfa1c0de946f..37ffa953b064 100644
--- a/Documentation/devicetree/bindings/serial/samsung_uart.yaml
+++ b/Documentation/devicetree/bindings/serial/samsung_uart.yaml
@@ -167,6 +167,12 @@ allOf:
       properties:
         reg-io-width: false
 
+        clocks:
+          maxItems: 2
+
+        clock-names:
+          maxItems: 2
+
 unevaluatedProperties: false
 
 examples:

-- 
2.45.2.993.g49e7a77208-goog

Re: [PATCH v4 2/2] dt-bindings: serial: samsung: fix maxItems for gs101
Posted by Rob Herring (Arm) 1 year, 4 months ago
On Fri, 12 Jul 2024 15:51:18 +0100, André Draszik wrote:
> While gs101 needs exactly two clocks for the UART, the schema doesn't
> currently limit the maximum number to this and instead the default of
> five from this schema is applied.
> 
> Update the schema accordingly.
> 
> Signed-off-by: André Draszik <andre.draszik@linaro.org>
> 
> ---
> v4:
> * drop description from clocks:, it was Linux-specific and a we'll
>   implement something that makes it obsolete anyway
> * no need to duplicate clock-names anymore
> ---
>  Documentation/devicetree/bindings/serial/samsung_uart.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>