gpiolib: separation of pin concerns
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 6 Nov 2012 15:03:35 +0000 (16:03 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Sun, 11 Nov 2012 18:06:07 +0000 (19:06 +0100)
commit1e63d7b9363f0c57d00991f9f2e0af374dfc591a
tree6bea7bfdd9dbfbe21433629b3f2a9758c92447be
parent9ef0d6f7628bdcb5cc3c11623930f2527a3881a0
gpiolib: separation of pin concerns

The fact that of_gpiochip_add_pin_range() and
gpiochip_add_pin_range() share too much code is fragile and
will invariably mean that bugs need to be fixed in two places
instead of one.

So separate the concerns of gpiolib.c and gpiolib-of.c and
have the latter call the former as back-end. This is necessary
also when going forward with other device descriptions such
as ACPI.

This is done by:

- Adding a return code to gpiochip_add_pin_range() so we can
  reliably check whether this succeeds.

- Get rid of the custom of_pinctrl_add_gpio_range() from
  pinctrl. Instead create of_pinctrl_get() to just retrive the
  pin controller per se from an OF node. This composite
  function was just begging to be deleted, it was way to
  purpose-specific.

- Use pinctrl_dev_get_name() to get the name of the retrieved
  pin controller and use that to call back into the generic
  gpiochip_add_pin_range().

Now the pin range is only allocated and tied to a pin
controller from the core implementation in gpiolib.c.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpiolib-of.c
drivers/gpio/gpiolib.c
drivers/pinctrl/devicetree.c
include/asm-generic/gpio.h
include/linux/gpio.h
include/linux/pinctrl/pinctrl.h