Unverified Commit a498db69 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Mark Brown
Browse files

regulator: dt-bindings: richtek,rt4801: use existing ena_gpiod feature 



The binding and driver duplicated regulator core feature of controlling
regulators with GPIOs (of_parse_cb + ena_gpiod) and created its own
enable-gpios property with multiple GPIOs.

This is a less preferred way, because enable-gpios should enable only one
element, not multiple. It also duplicates existing solution.

Deprecate the original 'enable-gpios' and add per-regulator property.

Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20220425072455.27356-2-krzysztof.kozlowski@linaro.org


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

Documentation/devicetree/bindings/regulator/richtek,rt4801-regulator.yaml

+13 −1
Original line number Diff line number Diff line
@@ -32,8 +32,11 @@ properties: 
      If only one gpio is specified, only one gpio used to control ENP/ENM.

      Else if both are specified, DSVP/DSVN could be controlled individually.

      If this property not specified, treat both as always-on regulators.



      Property is deprecated. Use enable-gpios in each regulator.

    minItems: 1

    maxItems: 2

    deprecated: true



patternProperties:

  "^DSV(P|N)$":
@@ -42,6 +45,14 @@ patternProperties: 
    description:

      Properties for single display bias regulator.



    properties:

      enable-gpios:

        description:

          GPIO to use to enable DSVP/DSVN regulator. One GPIO can be configured

          for controlling both regulators.  If this property not specified for

          any regulator, treat both as always-on regulators.

        maxItems: 1



required:

  - compatible

  - reg
@@ -57,19 +68,20 @@ examples: 
        rt4801@73 {

            compatible = "richtek,rt4801";

            reg = <0x73>;

            enable-gpios = <&gpio26 2 0>, <&gpio26 3 0>;



            dsvp: DSVP {

                regulator-name = "rt4801,dsvp";

                regulator-min-microvolt = <4000000>;

                regulator-max-microvolt = <6000000>;

                regulator-boot-on;

                enable-gpios = <&gpio26 2 0>;

            };

            dsvn: DSVN {

                regulator-name = "rt4801,dsvn";

                regulator-min-microvolt = <4000000>;

                regulator-max-microvolt = <6000000>;

                regulator-boot-on;

                enable-gpios = <&gpio26 3 0>;

            };



        };