uint32-array with voltages should be within one bracket pair <>, not each number in its own <>. Also the number of elements in the array should be defined within "items:". This fixes DT schema warnings like: maxim,max8997.example.dtb: pmic@66: max8997,pmic-buck1-dvs-voltage: [[1350000, 1300000, 1250000, 1200000, 1150000, 1100000, 1000000, 950000]] is too short Fixes: 1d2104f21618 ("regulator: dt-bindings: maxim,max8997: convert to dtschema") Reported-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> --- Patch is independent of DTS. --- .../bindings/regulator/maxim,max8997.yaml | 55 +++++++++++-------- 1 file changed, 31 insertions(+), 24 deletions(-) diff --git a/Documentation/devicetree/bindings/regulator/maxim,max8997.yaml b/Documentation/devicetree/bindings/regulator/maxim,max8997.yaml index d5a44ca3df04..6305a9abaead 100644 --- a/Documentation/devicetree/bindings/regulator/maxim,max8997.yaml +++ b/Documentation/devicetree/bindings/regulator/maxim,max8997.yaml @@ -32,8 +32,9 @@ properties: max8997,pmic-buck1-dvs-voltage: $ref: /schemas/types.yaml#/definitions/uint32-array - minItems: 1 - maxItems: 8 + items: + minItems: 1 + maxItems: 8 description: | A set of 8 voltage values in micro-volt (uV) units for buck1 when changing voltage using GPIO DVS. @@ -44,8 +45,9 @@ properties: max8997,pmic-buck2-dvs-voltage: $ref: /schemas/types.yaml#/definitions/uint32-array - minItems: 1 - maxItems: 8 + items: + minItems: 1 + maxItems: 8 description: | A set of 8 voltage values in micro-volt (uV) units for buck2 when changing voltage using GPIO DVS. @@ -56,8 +58,9 @@ properties: max8997,pmic-buck5-dvs-voltage: $ref: /schemas/types.yaml#/definitions/uint32-array - minItems: 1 - maxItems: 8 + items: + minItems: 1 + maxItems: 8 description: | A set of 8 voltage values in micro-volt (uV) units for buck5 when changing voltage using GPIO DVS. @@ -267,14 +270,17 @@ if: then: properties: max8997,pmic-buck1-dvs-voltage: - minItems: 8 - maxItems: 8 + items: + minItems: 8 + maxItems: 8 max8997,pmic-buck2-dvs-voltage: - minItems: 8 - maxItems: 8 + items: + minItems: 8 + maxItems: 8 max8997,pmic-buck5-dvs-voltage: - minItems: 8 - maxItems: 8 + items: + minItems: 8 + maxItems: 8 examples: - | @@ -303,20 +309,21 @@ examples: <&gpx0 6 GPIO_ACTIVE_HIGH>, <&gpl0 0 GPIO_ACTIVE_HIGH>; - max8997,pmic-buck1-dvs-voltage = <1350000>, <1300000>, - <1250000>, <1200000>, - <1150000>, <1100000>, - <1000000>, <950000>; + max8997,pmic-buck1-dvs-voltage = <1350000 1300000 + 1250000 1200000 + 1150000 1100000 + 1000000 950000>; + + max8997,pmic-buck2-dvs-voltage = <1100000 1000000 + 950000 900000 + 1100000 1000000 + 950000 900000>; - max8997,pmic-buck2-dvs-voltage = <1100000>, <1000000>, - <950000>, <900000>, - <1100000>, <1000000>, - <950000>, <900000>; - max8997,pmic-buck5-dvs-voltage = <1200000>, <1200000>, - <1200000>, <1200000>, - <1200000>, <1200000>, - <1200000>, <1200000>; + max8997,pmic-buck5-dvs-voltage = <1200000 1200000 + 1200000 1200000 + 1200000 1200000 + 1200000 1200000>; pinctrl-0 = <&max8997_irq>, <&otg_gp>, <&usb_sel>; pinctrl-names = "default"; -- 2.32.0