Commit e8917266 authored by Rob Herring's avatar Rob Herring
Browse files

dt-bindings: More dropping redundant minItems/maxItems



Another round of removing redundant minItems/maxItems from new schema in
the recent merge window.

If a property has an 'items' list, then a 'minItems' or 'maxItems' with the
same size as the list is redundant and can be dropped. Note that is DT
schema specific behavior and not standard json-schema behavior. The tooling
will fixup the final schema adding any unspecified minItems/maxItems.

This condition is partially checked with the meta-schema already, but
only if both 'minItems' and 'maxItems' are equal to the 'items' length.
An improved meta-schema is pending.

Cc: Stephen Boyd <sboyd@kernel.org>
Cc: Joerg Roedel <joro@8bytes.org>
Cc: Will Deacon <will@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>
Cc: Richard Weinberger <richard@nod.at>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Sureshkumar Relli <naga.sureshkumar.relli@xilinx.com>
Cc: Brian Norris <computersforpeace@gmail.com>
Cc: Kamal Dasu <kdasu.kdev@gmail.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Sebastian Siewior <bigeasy@linutronix.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: linux-clk@vger.kernel.org
Cc: iommu@lists.linux-foundation.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rtc@vger.kernel.org
Cc: linux-usb@vger.kernel.org
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Reviewed-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: default avatarAlexandre Belloni <alexandre.belloni@bootlin.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://lore.kernel.org/r/20210713193453.690290-1-robh@kernel.org
parent d549c667
Loading
Loading
Loading
Loading
+0 −1
Original line number Diff line number Diff line
@@ -50,7 +50,6 @@ properties:

  reg:
    minItems: 1
    maxItems: 3
    items:
      - description: base register
      - description: power register
+0 −2
Original line number Diff line number Diff line
@@ -28,14 +28,12 @@ properties:
      - description: configuration registers for MMU instance 0
      - description: configuration registers for MMU instance 1
    minItems: 1
    maxItems: 2

  interrupts:
    items:
      - description: interruption for MMU instance 0
      - description: interruption for MMU instance 1
    minItems: 1
    maxItems: 2

  clocks:
    items:
+0 −1
Original line number Diff line number Diff line
@@ -57,7 +57,6 @@ properties:

  ranges:
    minItems: 1
    maxItems: 3
    description: |
      Memory bus areas for interacting with the devices. Reflects
      the memory layout with four integer values following:
+0 −8
Original line number Diff line number Diff line
@@ -84,7 +84,6 @@ properties:

  interrupts:
    minItems: 1
    maxItems: 3
    items:
      - description: NAND CTLRDY interrupt
      - description: FLASH_DMA_DONE if flash DMA is available
@@ -92,7 +91,6 @@ properties:

  interrupt-names:
    minItems: 1
    maxItems: 3
    items:
      - const: nand_ctlrdy
      - const: flash_dma_done
@@ -148,8 +146,6 @@ allOf:
    then:
      properties:
        reg-names:
          minItems: 2
          maxItems: 2
          items:
            - const: nand
            - const: nand-int-base
@@ -161,8 +157,6 @@ allOf:
    then:
      properties:
        reg-names:
          minItems: 3
          maxItems: 3
          items:
            - const: nand
            - const: nand-int-base
@@ -175,8 +169,6 @@ allOf:
    then:
      properties:
        reg-names:
          minItems: 3
          maxItems: 3
          items:
            - const: nand
            - const: iproc-idm
+0 −1
Original line number Diff line number Diff line
@@ -30,7 +30,6 @@ properties:
    maxItems: 1

  clocks:
    minItems: 2
    items:
      - description: PCLK clocks
      - description: EXTCLK clocks. Faraday calls it CLK1HZ and says the clock
Loading