Unverified Commit efe9a6da authored by Rob Herring's avatar Rob Herring Committed by Mark Brown
Browse files

regulator: dt-bindings: gpio-regulator: Fix "gpios-states" and "states" array bounds 



The minimum number of array entries for "gpios-states" was not not
specified, so the the default is the same as the max (8).

The minimum is also missing from "states", and the maximum is also wrong
as it should be 2^(# of GPIO lines). Since there can be 1 to 8 GPIOs,
the "states" range should be 2 to 256.

Signed-off-by: default avatarRob Herring <robh@kernel.org>
Link: https://msgid.link/r/20240224000752.3830665-1-robh@kernel.org


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent b9262cc1

Documentation/devicetree/bindings/regulator/gpio-regulator.yaml

+3 −1
Original line number Diff line number Diff line
@@ -47,6 +47,7 @@ properties: 
        1: HIGH

      Default is LOW if nothing else is specified.

    $ref: /schemas/types.yaml#/definitions/uint32-array

    minItems: 1

    maxItems: 8

    items:

      enum: [0, 1]
@@ -57,7 +58,8 @@ properties: 
      regulator and matching GPIO configurations to achieve them. If there are

      no states in the "states" array, use a fixed regulator instead.

    $ref: /schemas/types.yaml#/definitions/uint32-matrix

    maxItems: 8

    minItems: 2

    maxItems: 256

    items:

      items:

        - description: Voltage in microvolts