[PATCH] dt-bindings: serial: snps-dw-apb-uart: correct number of DMAs

Krzysztof Kozlowski posted 1 patch 2 years, 10 months ago
Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] dt-bindings: serial: snps-dw-apb-uart: correct number of DMAs
Posted by Krzysztof Kozlowski 2 years, 10 months ago
The "minItems" alone does not impose any upper limit of DMAs, so switch
it to "maxItems" which also implies same value for minItems.

Fixes: 370f696e4474 ("dt-bindings: serial: snps-dw-apb-uart: add dma & dma-names properties")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
index 2becdfab4f15..8212a9f483b5 100644
--- a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
+++ b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
@@ -68,7 +68,7 @@ properties:
       - const: apb_pclk
 
   dmas:
-    minItems: 2
+    maxItems: 2
 
   dma-names:
     items:
-- 
2.34.1
Re: [PATCH] dt-bindings: serial: snps-dw-apb-uart: correct number of DMAs
Posted by Rob Herring 2 years, 10 months ago
On Fri, 17 Mar 2023 16:57:12 +0100, Krzysztof Kozlowski wrote:
> The "minItems" alone does not impose any upper limit of DMAs, so switch
> it to "maxItems" which also implies same value for minItems.
> 
> Fixes: 370f696e4474 ("dt-bindings: serial: snps-dw-apb-uart: add dma & dma-names properties")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Re: [PATCH] dt-bindings: serial: snps-dw-apb-uart: correct number of DMAs
Posted by Conor Dooley 2 years, 10 months ago
On Fri, Mar 17, 2023 at 04:57:12PM +0100, Krzysztof Kozlowski wrote:
> The "minItems" alone does not impose any upper limit of DMAs, so switch
> it to "maxItems" which also implies same value for minItems.
> 
> Fixes: 370f696e4474 ("dt-bindings: serial: snps-dw-apb-uart: add dma & dma-names properties")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

> ---
>  Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> index 2becdfab4f15..8212a9f483b5 100644
> --- a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> +++ b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> @@ -68,7 +68,7 @@ properties:
>        - const: apb_pclk
>  
>    dmas:
> -    minItems: 2
> +    maxItems: 2
>  
>    dma-names:
>      items:
> -- 
> 2.34.1
>