[PATCH V1 0/3] Support for open drain gpios in gpilib/fixed regulators.

February 13th, 2012 - 01:40 am ET by Laxman Dewangan | Report spam
This patch is as per discussion on the patch
[PATCH V1] regulator: fixed: Support for open-drain gpio
where it was suggested to push the handling of open drain gpios
into the gpio libray.
Also gpio documentation says the possibility of open-drain gpio support.

This is series of patches to add support of the open drain gpios
in standard gpio library and using this in fixed regulators.
- Added GPIOF_OD flag so that client can request the gpio with this flag.
- Added support for the sysfs interface for open-drain.
- Handle open drain gpio configuration properly.
- Use this flag in fixed regualtor to tell that gpio is open drain.
- Gpio documentation changes to have support for open-drain.

Laxman Dewangan (3):
gpio: gpiolib: Support for open drain gpios
Documentation: gpio: Add details of open-drain configuration
regulator: fixed: Support for open-drain gpio

Documentation/gpio.txt | 21 +++++-
drivers/gpio/gpiolib.c | 144 ++++++++++++++++++++++++++++++++++++++-
drivers/regulator/fixed.c | 29 ++++-
include/linux/gpio.h | 3 +
include/linux/regulator/fixed.h | 7 ++
5 files changed, 184 insertions(+), 20 deletions(-)

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
email Follow the discussionReplies 10 repliesReplies Make a reply

Replies

#1 Linus Walleij
February 13th, 2012 - 03:10 pm ET | Report spam
On Mon, Feb 13, 2012 at 7:29 AM, Laxman Dewangan wrote:

This is series of patches  to add support of the open drain gpios
in standard gpio library and using this in fixed regulators.
- Added GPIOF_OD flag so that client can request the gpio with this flag.
- Added support for the sysfs interface for open-drain.
- Handle open drain gpio configuration properly.
- Use this flag in fixed regualtor to tell that gpio is open drain.
- Gpio documentation changes to have support for open-drain.



My gut feeling is that this stuff needs to go into pinctrl.

The reason, which has been discussed when similar patches (from me,
for example) were NACK:ed in the past, is that if we put in OD pin
properties, we put in all pin properties (open source, drive strength,
biasing pull-up/pull-down etc etc) and that is the reason to why the
orthogonal pinctrl subsystem was created in the first place.

So I'd opt for moving the driver you need to drivers/pinctrl and
expose the gpiolib interface from there, while adding pinctrl
on the side.

Also, we had a flamefest a while back as to whether we should
enumerate all config options (like OD, OS...) or have it all-custom,
and I was sort of shot down on that. But I have a generic control patch
set boiling so if you like that kind of stuff I'll respin it.

Yours,
Linus Walleij
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Similar topics