Re: [RFT PATCH 1/2] regulator: dt-bindings: richtek,rt4801: use existing ena_gpiod feature

From: Rob Herring
Date: Mon Apr 04 2022 - 18:12:28 EST


On Fri, 01 Apr 2022 17:42:36 +0200, Krzysztof Kozlowski wrote:
> 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: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> ---
> .../regulator/richtek,rt4801-regulator.yaml | 14 +++++++++++++-
> 1 file changed, 13 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>