Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1542632

Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches

Show all headers | View raw

2016-12-14 18:36 GMT+01:00 Jonathan Cameron <jic23@jic23.retrosnub.co.uk>:
>
>
> On 14 December 2016 16:58:21 GMT+00:00, Bartosz Golaszewski
> <bgolaszewski@baylibre.com> wrote:
>>2016-12-13 20:27 GMT+01:00 Rob Herring <robh@kernel.org>:
>>> On Sun, Dec 11, 2016 at 11:21:44PM +0100, Bartosz Golaszewski wrote:
>>>> Some boards are equipped with simple, GPIO-driven power load
>>switches.
>>>> An example of such ICs is the TI tps229* series.
>>>
>>> How is this different than a GPIO regulator? The input and output
>>> voltages just happen to be the same. I could be convinced this is
>>> different enough to have a different compatible, but it somewhat
>>seems
>>> you want to use this for IIO, so you are creating a different binding
>>> for that usecase.
>>>
>>
>>It's more of a fixed regulator I suppose. Do you mean adding a new
>>compatible to the fixed-regulator binding (e.g. "gpio-power-switch" or
>>"simple-power-switch") and then providing an iio driver for toggling
>>the switch?
>
> I am rather torn on whether the IIO interface makes any sense beyond that of
> convenience.
>

The question is: will the iio maintainers be ok with adding support
for interfaces different than iio (I guess so since Lars already
mentioned wanting to support the GPIO chardev)? If so, I'm ok with not
using the iio framework in the kernel.

> A bridge to a regulator in general might make sense to cover the case of a
> reg effectively
> acting as a DAC at the edge of the known hardware.
>
> Lars' point about perhaps adding support for the new gpio userspace stuff to
> libiio would in
> this case also be rather papering over the issue.
> There is known hardware there so we should describe it!
>
> I think we should be considering this as the general case of the Linux
> controlled power supply.
> How do we want to represent that?
> Ultimately does a general regulator userspace interface make sense?
>
> Classic case of people cutting our hardware in half and making boundaries
> beyond which lie dragons.
>
> I would love to see the general case covered.
>

It seems as if this is already covered by the userspace-consumer
regulator driver, but it doesn't speak device tree yet. I guess we
could reuse it by merging the proposed gpio-power-switch binding and
extending it to parse DT.

Best regards,
Bartosz Golaszewski

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] iio: GPIO power switch support Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-11 23:30 +0100
  [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-11 23:30 +0100
    Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven  power switches Rob Herring <robh@kernel.org> - 2016-12-13 20:30 +0100
      Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-14 18:00 +0100
        Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-15 12:00 +0100
        Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Rob Herring <robh@kernel.org> - 2016-12-15 16:10 +0100
          Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Geert Uytterhoeven <geert@linux-m68k.org> - 2016-12-23 10:10 +0100
            Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power  switches Lars-Peter Clausen <lars@metafoo.de> - 2016-12-23 12:50 +0100
              Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power  switches Jonathan Cameron <jic23@kernel.org> - 2016-12-30 16:10 +0100
    Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches Linus Walleij <linus.walleij@linaro.org> - 2016-12-28 14:00 +0100

csiph-web