]> git.karo-electronics.de Git - karo-tx-linux.git/commit
pinctrl: add explicit gpio_disable_free pinmux_op
authorStephen Warren <swarren@nvidia.com>
Fri, 21 Oct 2011 18:25:53 +0000 (12:25 -0600)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 8 Dec 2011 15:18:59 +0000 (16:18 +0100)
commit92efa0e5238bd3178039b30d36b84f7ac20a840f
treeb8551be17366bbf415f0e414da8558ca67831b5e
parentd29dbc4641f8c011cd6e47f749fca555690b9835
pinctrl: add explicit gpio_disable_free pinmux_op

Some pinctrl drivers (Tegra at least) program a pin to be a GPIO in a
completely different manner than they select which function to mux out of
that pin. In order to support a single "free" pinmux_op, the driver would
need to maintain a per-pin state of requested-for-gpio vs. requested-for-
function. However, that's a lot of work when the core already has explicit
separate paths for gpio request/free and function request/free.

So, add a gpio_disable_free op to struct pinmux_ops, and make pin_free()
call it when appropriate.

When doing this, I noticed that when calling pin_request():

    !!gpio == (gpio_range != NULL)

... and so I collapsed those two parameters in both pin_request(), and
when adding writing the new code in pin_free().

Also, for pin_free():

    !!free_func == (gpio_range != NULL)

However, I didn't want pin_free() to know about the GPIO function naming
special case, so instead, I reworked pin_free() to always return the pin's
previously requested function, and now pinmux_free_gpio() calls
kfree(function). This is much more balanced with the allocation having
been performed in pinmux_request_gpio().

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinmux.c
include/linux/pinctrl/pinmux.h