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

Groups > linux.kernel > #1297398

Re: [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS GPIO

From Mark Brown <broonie@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS GPIO
Date 2015-12-23 12:50 +0100
Message-ID <qIQ5B-Cl-23@gated-at.bofh.it> (permalink)
References <qIPsR-oU-3@gated-at.bofh.it> <qIPsS-oU-13@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Wed, Dec 23, 2015 at 11:58:36AM +0100, Paul Kocialkowski wrote:
> Some devices don't hook the DVS pin to a GPIO but to ground or VCC.
> In those cases, it is not a problem to have no DVS GPIO.

I would expect the driver at least needs to know how the pins or
strapped, or otherwise have configuration for ignoring the input on the
pins.  This just deletes the warnings, it doesn't have any handling for
this case that I can see.

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

Thread

[PATCH 0/6] LG Optimus Black (P970) codename sniper support and lp872x improvements Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 12:10 +0100
  [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS GPIO Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 12:10 +0100
    Re: [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS  GPIO Mark Brown <broonie@kernel.org> - 2015-12-23 12:50 +0100
      Re: [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS  GPIO Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 13:00 +0100
        Re: [PATCH 3/6] regulator: lp872x: Remove warning about invalid DVS  GPIO Mark Brown <broonie@kernel.org> - 2015-12-23 13:00 +0100
  [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 12:10 +0100
    Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Mark Brown <broonie@kernel.org> - 2015-12-23 13:00 +0100
      Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 14:00 +0100
      Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-24 19:20 +0100
        Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Mark Brown <broonie@kernel.org> - 2015-12-24 20:40 +0100
          Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-24 21:10 +0100
      Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Milo Kim <milo.kim@ti.com> - 2015-12-28 03:50 +0100
        Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-29 00:00 +0100
          Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Milo Kim <milo.kim@ti.com> - 2015-12-29 01:50 +0100
            Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-29 12:20 +0100
              Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Milo Kim <milo.kim@ti.com> - 2015-12-30 01:30 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-30 09:40 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Mark Brown <broonie@kernel.org> - 2015-12-30 17:40 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-30 19:40 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-31 23:00 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Mark Brown <broonie@kernel.org> - 2015-12-31 23:20 +0100
                Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Mark Brown <broonie@kernel.org> - 2015-12-31 23:00 +0100
    Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Milo Kim <milo.kim@ti.com> - 2015-12-28 03:50 +0100
    Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Rob Herring <robh@kernel.org> - 2015-12-29 21:10 +0100
      Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Paul Kocialkowski <contact@paulk.fr> - 2015-12-29 22:30 +0100
        Re: [PATCH 4/6] regulator: lp872x: Add enable GPIO pin support Rob Herring <robh@kernel.org> - 2015-12-29 23:00 +0100
  [PATCH 6/6] ARM: multi_v7_defconfig: Enable LP872x regulator support Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 12:10 +0100
  [PATCH 2/6] regulator: lp872x: Get rid of duplicate reference to DVS GPIO Paul Kocialkowski <contact@paulk.fr> - 2015-12-23 12:10 +0100
    Applied "regulator: lp872x: Get rid of duplicate reference to DVS GPIO" to the regulator tree Mark Brown <broonie@kernel.org> - 2015-12-23 13:20 +0100
    Re: [PATCH 2/6] regulator: lp872x: Get rid of duplicate reference to  DVS GPIO Milo Kim <milo.kim@ti.com> - 2015-12-28 03:50 +0100

csiph-web