Re: [PATCH 2/5] dt-bindings: ata: drop minItems equal to maxItems

Damien Le Moal
2022-08-25T22:31:26Z

On 8/25/22 20:33, Krzysztof Kozlowski wrote:
> minItems, if missing, are implicitly equal to maxItems, so drop
> redundant piece to reduce size of code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>

> ---
> Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml | 1 -
> .../devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml | 2 --
> Documentation/devicetree/bindings/ata/sata_highbank.yaml | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> index 235a93ac86b0..3766cc80cb17 100644
> --- a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> +++ b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> @@ -30,7 +30,6 @@ properties:
> - const: brcm,bcm-nsp-ahci
>
> reg:
> - minItems: 2
> maxItems: 2
>
> reg-names:
> diff --git a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> index 21a90975593b..529093666508 100644
> --- a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> +++ b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> @@ -22,7 +22,6 @@ properties:
> maxItems: 1
>
> resets:
> - minItems: 2
> maxItems: 2
> description: phandles to the reset lines for both SATA bridges
>
> @@ -32,7 +31,6 @@ properties:
> - const: sata1
>
> clocks:
> - minItems: 2
> maxItems: 2
> description: phandles to the compulsory peripheral clocks
>
> diff --git a/Documentation/devicetree/bindings/ata/sata_highbank.yaml b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> index 49679b58041c..f23f26a8f21c 100644
> --- a/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> +++ b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> @@ -52,7 +52,6 @@ properties:
> minItems: 1
> maxItems: 8
> items:
> - minItems: 2
> maxItems: 2
>
> calxeda,tx-atten:

--
Damien Le Moal
Western Digital Research