gpiolib: safer implementation of desc_to_gpio()
authorAlexandre Courbot <acourbot@nvidia.com>
Fri, 4 Oct 2013 17:59:57 +0000 (10:59 -0700)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 11 Oct 2013 14:31:53 +0000 (16:31 +0200)
The current implementation of desc_to_gpio() relies on the chip pointer
to be set to a valid value in order to compute the GPIO number. This
was done in the hope that we can get rid of the gpio_desc global array,
but this is not happening anytime soon.

This patch reimplements desc_to_gpio() in a fashion similar to that of
gpio_to_desc(). As a result, desc_to_gpio(gpio_to_desc(gpio)) == gpio is
now always true. This allows to call desc_to_gpio() on non-initialized
descriptors as some error-handling code currently does.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
Reported-by: Dr. H. Nikolaus Schaller <hns@goldelico.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpiolib.c

index 86ef3461ec0647b42f1e1ff94a9f47225151f7f6..f33063114d568e54d2fb17793959602a520b699c 100644 (file)
@@ -136,7 +136,7 @@ static struct gpio_desc *gpio_to_desc(unsigned gpio)
  */
 static int desc_to_gpio(const struct gpio_desc *desc)
 {
-       return desc->chip->base + gpio_chip_hwgpio(desc);
+       return desc - &gpio_desc[0];
 }