devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML
@ 2024-10-15 22:42 Marek Vasut
  2024-10-16  7:24 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2024-10-15 22:42 UTC (permalink / raw)
  To: devicetree
  Cc: Marek Vasut, Conor Dooley, Krzysztof Kozlowski, Liam Girdwood,
	Mark Brown, Rob Herring, Tim Harvey, kernel

Convert Linear Technology LTC3676 8-output I2C voltage regulator IC
DT bindings to YAML DT. Add missing interrupts: property as this IC
does have interrupt line and it is used in existing DTs.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Conor Dooley <conor+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Cc: Rob Herring <robh@kernel.org>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: devicetree@vger.kernel.org
Cc: kernel@dh-electronics.com
---
 .../bindings/regulator/lltc,ltc3676.yaml      | 171 ++++++++++++++++++
 .../devicetree/bindings/regulator/ltc3676.txt |  94 ----------
 2 files changed, 171 insertions(+), 94 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
 delete mode 100644 Documentation/devicetree/bindings/regulator/ltc3676.txt

diff --git a/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
new file mode 100644
index 0000000000000..ec9ce856a944d
--- /dev/null
+++ b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
@@ -0,0 +1,171 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Linear Technology LTC3676 8-output regulators
+
+maintainers:
+  - Tim Harvey <tharvey@gateworks.com>
+
+description: |
+  LTC3676 contains eight regulators, 4 switching SW1..SW4 and four LDO1..4 .
+
+properties:
+  compatible:
+    const: lltc,ltc3676
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  regulators:
+    type: object
+    description: |
+      List of regulators provided by this controller, must be named
+      after their hardware counterparts (SW|LDO)[1-4].
+
+    patternProperties:
+      "^(sw[1-4]|ldo[24])$":
+        type: object
+        $ref: regulator.yaml#
+        description:
+          Properties for single SW or LDO regulator. Regulators SW1..SW4 can
+          regulate the feedback reference from 412.5mV to 800mV in 12.5 mV
+          steps. The output voltage thus ranges between 0.4125 * (1 + R1/R2) V
+          and 0.8 * (1 + R1/R2) V.
+          Regulators LDO1, LDO2, LDO4 have a fixed 0.725 V reference and thus
+          output 0.725 * (1 + R1/R2) V.
+          The LDO1 standby regulator can not be disabled and thus should have
+          the regulator-always-on property set.
+
+        properties:
+          lltc,fb-voltage-divider:
+            description:
+              An array of two integers containing the resistor values
+              R1 and R2 of the feedback voltage divider in ohms.
+            $ref: /schemas/types.yaml#/definitions/uint32-array
+            minItems: 2
+            maxItems: 2
+
+        required:
+          - lltc,fb-voltage-divider
+
+        unevaluatedProperties: false
+
+    properties:
+      ldo1:
+        type: object
+        $ref: regulator.yaml#
+        description:
+          The LDO1 standby regulator can not be disabled and thus should
+          have the regulator-always-on property set. See patternProperties
+          description above for the rest of the details.
+
+        properties:
+          lltc,fb-voltage-divider:
+            description:
+              An array of two integers containing the resistor values
+              R1 and R2 of the feedback voltage divider in ohms.
+            $ref: /schemas/types.yaml#/definitions/uint32-array
+            minItems: 2
+            maxItems: 2
+
+        required:
+          - lltc,fb-voltage-divider
+          - regulator-always-on
+
+        unevaluatedProperties: false
+
+      ldo3:
+        type: object
+        $ref: regulator.yaml#
+        description:
+          The LDO3 regulator is fixed to 1.8 V. See patternProperties
+          description above for the rest of the details.
+
+        unevaluatedProperties: false
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - regulators
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pmic@3c {
+            compatible = "lltc,ltc3676";
+            reg = <0x3c>;
+
+            regulators {
+                sw1_reg: sw1 {
+                    regulator-min-microvolt = <674400>;
+                    regulator-max-microvolt = <1308000>;
+                    lltc,fb-voltage-divider = <127000 200000>;
+                    regulator-ramp-delay = <7000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                sw2_reg: sw2 {
+                    regulator-min-microvolt = <1033310>;
+                    regulator-max-microvolt = <200400>;
+                    lltc,fb-voltage-divider = <301000 200000>;
+                    regulator-ramp-delay = <7000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                sw3_reg: sw3 {
+                    regulator-min-microvolt = <674400>;
+                    regulator-max-microvolt = <130800>;
+                    lltc,fb-voltage-divider = <127000 200000>;
+                    regulator-ramp-delay = <7000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                sw4_reg: sw4 {
+                    regulator-min-microvolt = <868310>;
+                    regulator-max-microvolt = <168400>;
+                    lltc,fb-voltage-divider = <221000 200000>;
+                    regulator-ramp-delay = <7000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                ldo2_reg: ldo2 {
+                    regulator-min-microvolt = <2490375>;
+                    regulator-max-microvolt = <2490375>;
+                    lltc,fb-voltage-divider = <487000 200000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                ldo3_reg: ldo3 {
+                    regulator-min-microvolt = <1800000>;
+                    regulator-max-microvolt = <1800000>;
+                    regulator-boot-on;
+                };
+
+                ldo4_reg: ldo4 {
+                    regulator-min-microvolt = <3023250>;
+                    regulator-max-microvolt = <3023250>;
+                    lltc,fb-voltage-divider = <634000 200000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+            };
+        };
+    };
diff --git a/Documentation/devicetree/bindings/regulator/ltc3676.txt b/Documentation/devicetree/bindings/regulator/ltc3676.txt
deleted file mode 100644
index d4eb366ce18ce..0000000000000
--- a/Documentation/devicetree/bindings/regulator/ltc3676.txt
+++ /dev/null
@@ -1,94 +0,0 @@
-Linear Technology LTC3676 8-output regulators
-
-Required properties:
-- compatible: "lltc,ltc3676"
-- reg: I2C slave address
-
-Required child node:
-- regulators: Contains eight regulator child nodes sw1, sw2, sw3, sw4,
-  ldo1, ldo2, ldo3, and ldo4, specifying the initialization data as
-  documented in Documentation/devicetree/bindings/regulator/regulator.txt.
-
-Each regulator is defined using the standard binding for regulators. The
-nodes for sw1, sw2, sw3, sw4, ldo1, ldo2 and ldo4 additionally need to specify
-the resistor values of their external feedback voltage dividers:
-
-Required properties (not on ldo3):
-- lltc,fb-voltage-divider: An array of two integers containing the resistor
-  values R1 and R2 of the feedback voltage divider in ohms.
-
-Regulators sw1, sw2, sw3, sw4 can regulate the feedback reference from:
-412.5mV to 800mV in 12.5 mV steps. The output voltage thus ranges between
-0.4125 * (1 + R1/R2) V and 0.8 * (1 + R1/R2) V.
-
-Regulators ldo1, ldo2, and ldo4 have a fixed 0.725 V reference and thus output
-0.725 * (1 + R1/R2) V. The ldo3 regulator is fixed to 1.8 V.  The ldo1 standby
-regulator can not be disabled and thus should have the regulator-always-on
-property set.
-
-Example:
-
-	ltc3676: pmic@3c {
-		compatible = "lltc,ltc3676";
-		reg = <0x3c>;
-
-		regulators {
-			sw1_reg: sw1 {
-				regulator-min-microvolt = <674400>;
-				regulator-max-microvolt = <1308000>;
-				lltc,fb-voltage-divider = <127000 200000>;
-				regulator-ramp-delay = <7000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw2_reg: sw2 {
-				regulator-min-microvolt = <1033310>;
-				regulator-max-microvolt = <200400>;
-				lltc,fb-voltage-divider = <301000 200000>;
-				regulator-ramp-delay = <7000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw3_reg: sw3 {
-				regulator-min-microvolt = <674400>;
-				regulator-max-microvolt = <130800>;
-				lltc,fb-voltage-divider = <127000 200000>;
-				regulator-ramp-delay = <7000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw4_reg: sw4 {
-				regulator-min-microvolt = <868310>;
-				regulator-max-microvolt = <168400>;
-				lltc,fb-voltage-divider = <221000 200000>;
-				regulator-ramp-delay = <7000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			ldo2_reg: ldo2 {
-				regulator-min-microvolt = <2490375>;
-				regulator-max-microvolt = <2490375>;
-				lltc,fb-voltage-divider = <487000 200000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			ldo3_reg: ldo3 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <1800000>;
-				regulator-boot-on;
-			};
-
-			ldo4_reg: ldo4 {
-				regulator-min-microvolt = <3023250>;
-				regulator-max-microvolt = <3023250>;
-				lltc,fb-voltage-divider = <634000 200000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-		};
-	};
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML
  2024-10-15 22:42 [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML Marek Vasut
@ 2024-10-16  7:24 ` Krzysztof Kozlowski
  2024-10-16 22:48   ` Marek Vasut
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-16  7:24 UTC (permalink / raw)
  To: Marek Vasut
  Cc: devicetree, Conor Dooley, Krzysztof Kozlowski, Liam Girdwood,
	Mark Brown, Rob Herring, Tim Harvey, kernel

On Wed, Oct 16, 2024 at 12:42:51AM +0200, Marek Vasut wrote:
> Convert Linear Technology LTC3676 8-output I2C voltage regulator IC
> DT bindings to YAML DT. Add missing interrupts: property as this IC
> does have interrupt line and it is used in existing DTs.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Rob Herring <robh@kernel.org>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: devicetree@vger.kernel.org
> Cc: kernel@dh-electronics.com
> ---
>  .../bindings/regulator/lltc,ltc3676.yaml      | 171 ++++++++++++++++++
>  .../devicetree/bindings/regulator/ltc3676.txt |  94 ----------
>  2 files changed, 171 insertions(+), 94 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
>  delete mode 100644 Documentation/devicetree/bindings/regulator/ltc3676.txt
> 
> diff --git a/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
> new file mode 100644
> index 0000000000000..ec9ce856a944d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
> @@ -0,0 +1,171 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Linear Technology LTC3676 8-output regulators
> +
> +maintainers:
> +  - Tim Harvey <tharvey@gateworks.com>
> +
> +description: |
> +  LTC3676 contains eight regulators, 4 switching SW1..SW4 and four LDO1..4 .
> +
> +properties:
> +  compatible:
> +    const: lltc,ltc3676
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  regulators:
> +    type: object

Move the additionalProperties:false here.

> +    description: |
> +      List of regulators provided by this controller, must be named
> +      after their hardware counterparts (SW|LDO)[1-4].
> +
> +    patternProperties:
> +      "^(sw[1-4]|ldo[24])$":
> +        type: object
> +        $ref: regulator.yaml#

Move the unevaluatedProperties here, ditto in other places.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML
  2024-10-16  7:24 ` Krzysztof Kozlowski
@ 2024-10-16 22:48   ` Marek Vasut
  2024-10-17  7:01     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2024-10-16 22:48 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: devicetree, Conor Dooley, Krzysztof Kozlowski, Liam Girdwood,
	Mark Brown, Rob Herring, Tim Harvey, kernel

On 10/16/24 9:24 AM, Krzysztof Kozlowski wrote:
> On Wed, Oct 16, 2024 at 12:42:51AM +0200, Marek Vasut wrote:
>> Convert Linear Technology LTC3676 8-output I2C voltage regulator IC
>> DT bindings to YAML DT. Add missing interrupts: property as this IC
>> does have interrupt line and it is used in existing DTs.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> ---
>> Cc: Conor Dooley <conor+dt@kernel.org>
>> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
>> Cc: Liam Girdwood <lgirdwood@gmail.com>
>> Cc: Mark Brown <broonie@kernel.org>
>> Cc: Rob Herring <robh@kernel.org>
>> Cc: Tim Harvey <tharvey@gateworks.com>
>> Cc: devicetree@vger.kernel.org
>> Cc: kernel@dh-electronics.com
>> ---
>>   .../bindings/regulator/lltc,ltc3676.yaml      | 171 ++++++++++++++++++
>>   .../devicetree/bindings/regulator/ltc3676.txt |  94 ----------
>>   2 files changed, 171 insertions(+), 94 deletions(-)
>>   create mode 100644 Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
>>   delete mode 100644 Documentation/devicetree/bindings/regulator/ltc3676.txt
>>
>> diff --git a/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
>> new file mode 100644
>> index 0000000000000..ec9ce856a944d
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
>> @@ -0,0 +1,171 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Linear Technology LTC3676 8-output regulators
>> +
>> +maintainers:
>> +  - Tim Harvey <tharvey@gateworks.com>
>> +
>> +description: |
>> +  LTC3676 contains eight regulators, 4 switching SW1..SW4 and four LDO1..4 .
>> +
>> +properties:
>> +  compatible:
>> +    const: lltc,ltc3676
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  interrupts:
>> +    maxItems: 1
>> +
>> +  regulators:
>> +    type: object
> 
> Move the additionalProperties:false here.
> 
>> +    description: |
>> +      List of regulators provided by this controller, must be named
>> +      after their hardware counterparts (SW|LDO)[1-4].
>> +
>> +    patternProperties:
>> +      "^(sw[1-4]|ldo[24])$":
>> +        type: object
>> +        $ref: regulator.yaml#
> 
> Move the unevaluatedProperties here, ditto in other places.
Fixed in V2.

I wanted to ask though, there are these patternProperties here which 
cover sw1..4 and ldo2 and ldo4, and then there is dedicated special case 
for ldo1 (because this LDO node must always contain regulator-always-on 
property) and ldo3 (because this LDO node must NOT contain 
lltc,fb-voltage-divider property). Is there any way to place all the 
sw1..4 and ldo1..4 into patternProperties, and then selectively DISABLE 
lltc,fb-voltage-divider requirement for LDO3 and ENABLE 
regulator-always-on for LDO1 ? I was looking at allOf:if: , but couldn't 
find anything.

Thanks !

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML
  2024-10-16 22:48   ` Marek Vasut
@ 2024-10-17  7:01     ` Krzysztof Kozlowski
  2024-10-17 20:50       ` Marek Vasut
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2024-10-17  7:01 UTC (permalink / raw)
  To: Marek Vasut
  Cc: devicetree, Conor Dooley, Krzysztof Kozlowski, Liam Girdwood,
	Mark Brown, Rob Herring, Tim Harvey, kernel

On 17/10/2024 00:48, Marek Vasut wrote:
> 
> I wanted to ask though, there are these patternProperties here which 
> cover sw1..4 and ldo2 and ldo4, and then there is dedicated special case 
> for ldo1 (because this LDO node must always contain regulator-always-on 
> property) and ldo3 (because this LDO node must NOT contain 
> lltc,fb-voltage-divider property). Is there any way to place all the 
> sw1..4 and ldo1..4 into patternProperties, and then selectively DISABLE 
> lltc,fb-voltage-divider requirement for LDO3 and ENABLE 
> regulator-always-on for LDO1 ? I was looking at allOf:if: , but couldn't 
> find anything.

Maybe this would work:
    patternPropeties:
      "^(sw[1-4]|ldo[1-4])$":
         ......
         ......
         allOf:
           - if:
             $nodename:
               const: ldo3
             then:
               ...

but I am not sure if this would be readable and beneficial. Current code
is OK.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML
  2024-10-17  7:01     ` Krzysztof Kozlowski
@ 2024-10-17 20:50       ` Marek Vasut
  0 siblings, 0 replies; 5+ messages in thread
From: Marek Vasut @ 2024-10-17 20:50 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: devicetree, Conor Dooley, Krzysztof Kozlowski, Liam Girdwood,
	Mark Brown, Rob Herring, Tim Harvey, kernel

On 10/17/24 9:01 AM, Krzysztof Kozlowski wrote:
> On 17/10/2024 00:48, Marek Vasut wrote:
>>
>> I wanted to ask though, there are these patternProperties here which
>> cover sw1..4 and ldo2 and ldo4, and then there is dedicated special case
>> for ldo1 (because this LDO node must always contain regulator-always-on
>> property) and ldo3 (because this LDO node must NOT contain
>> lltc,fb-voltage-divider property). Is there any way to place all the
>> sw1..4 and ldo1..4 into patternProperties, and then selectively DISABLE
>> lltc,fb-voltage-divider requirement for LDO3 and ENABLE
>> regulator-always-on for LDO1 ? I was looking at allOf:if: , but couldn't
>> find anything.
> 
> Maybe this would work:
>      patternPropeties:
>        "^(sw[1-4]|ldo[1-4])$":
>           ......
>           ......
>           allOf:
>             - if:
>               $nodename:
>                 const: ldo3
>               then:
>                 ...
> 
> but I am not sure if this would be readable and beneficial. Current code
> is OK.

I was hoping for something like this:
"
diff --git 
a/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml 
b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
index f47eacf96cd67..4fd5c7dd509dd 100644
--- a/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
+++ b/Documentation/devicetree/bindings/regulator/lltc,ltc3676.yaml
@@ -30,7 +30,7 @@ properties:
        after their hardware counterparts (SW|LDO)[1-4].

      patternProperties:
-      "^(sw[1-4]|ldo[24])$":
+      "^(sw[1-4]|ldo[1-4])$":
          type: object
          unevaluatedProperties: false
          $ref: regulator.yaml#
@@ -56,36 +56,22 @@ properties:
          required:
            - lltc,fb-voltage-divider

-    properties:
-      ldo1:
-        type: object
-        unevaluatedProperties: false
-        $ref: regulator.yaml#
-        description:
-          The LDO1 standby regulator can not be disabled and thus should
-          have the regulator-always-on property set. See patternProperties
-          description above for the rest of the details.
-
-        properties:
-          lltc,fb-voltage-divider:
-            description:
-              An array of two integers containing the resistor values
-              R1 and R2 of the feedback voltage divider in ohms.
-            $ref: /schemas/types.yaml#/definitions/uint32-array
-            minItems: 2
-            maxItems: 2
-
-        required:
-          - lltc,fb-voltage-divider
-          - regulator-always-on
-
-      ldo3:
-        type: object
-        unevaluatedProperties: false
-        $ref: regulator.yaml#
-        description:
-          The LDO3 regulator is fixed to 1.8 V. See patternProperties
-          description above for the rest of the details.
+        allOf:
+          - if:
+              properties:
+                $nodename:
+                  const: ldo3
+            then:
+              properties:
+                lltc,fb-voltage-divider: false
+          - if:
+              properties:
+                $nodename:
+                  const: ldo1
+            then:
+              required:
+                - lltc,fb-voltage-divider
+                - regulator-always-on

  required:
    - compatible
"

But I keep getting this:
"
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:sw1:lltc,fb-voltage-divider: False schema does not 
allow [127000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:sw2:lltc,fb-voltage-divider: False schema does not 
allow [301000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:sw3:lltc,fb-voltage-divider: False schema does not 
allow [127000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:sw4:lltc,fb-voltage-divider: False schema does not 
allow [221000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:ldo2:lltc,fb-voltage-divider: False schema does not 
allow [487000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:ldo3: 'lltc,fb-voltage-divider' is a required property
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:ldo3: 'lltc,fb-voltage-divider' is a required property
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:ldo3: 'regulator-always-on' is a required property
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
/Documentation/devicetree/bindings/regulator/lltc,ltc3676.example.dtb: 
pmic@3c: regulators:ldo4:lltc,fb-voltage-divider: False schema does not 
allow [634000, 200000]
         from schema $id: 
http://devicetree.org/schemas/regulator/lltc,ltc3676.yaml#
"

I am at a loss here, do you have any hint you could offer ?

Thank you!

^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-10-17 21:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-15 22:42 [PATCH] dt-bindings: regulator: lltc,ltc3676: convert to YAML Marek Vasut
2024-10-16  7:24 ` Krzysztof Kozlowski
2024-10-16 22:48   ` Marek Vasut
2024-10-17  7:01     ` Krzysztof Kozlowski
2024-10-17 20:50       ` Marek Vasut

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).