| Message ID | 20220825113334.196908-2-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
Hi Krzysztof, Thank you for the patch. On Thu, Aug 25, 2022 at 02:33:31PM +0300, 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> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.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:
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:
On Thu, 25 Aug 2022 14:33:31 +0300, 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> > --- > 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(-) > Applied, thanks!
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:
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> --- 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(-)