summaryrefslogtreecommitdiff
path: root/drivers/gpio/gpiolib-of.h
diff options
context:
space:
mode:
authorStephen Boyd <swboyd@chromium.org>2019-08-01 01:28:26 +0300
committerLinus Walleij <linus.walleij@linaro.org>2019-08-01 11:04:16 +0300
commit49281a222ac42724b94f6c874f43a284ec58d37e (patch)
tree93bd75d6c9031876e8e67b63f8dcf2dcbd17f62f /drivers/gpio/gpiolib-of.h
parent2838bf941b120ec846a3903db13e319368d51b08 (diff)
downloadlinux-49281a222ac42724b94f6c874f43a284ec58d37e.tar.xz
gpio: of: Fix hard-assigned valid_mask for OF case
The recent refactoring to break out OF code to its own file contained a bug letting the need_valid_mask be overridden by the need of the device tree range check, and if there were no ranges, but device tree was active and the reserved GPIO used in another way, things likely crash. Cc: Mark Brown <broonie@kernel.org> Cc: Bjorn Andersson <bjorn.andersson@linaro.org> Cc: Stephen Boyd <swboyd@chromium.org> Fixes: f626d6dfb709 ("gpio: of: Break out OF-only code") Reported-by: Mark Brown <broonie@kernel.org> Suggested-by: Stephen Boyd <swboyd@chromium.org> Signed-off-by: Stephen Boyd <swboyd@chromium.org> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio/gpiolib-of.h')
-rw-r--r--drivers/gpio/gpiolib-of.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpio/gpiolib-of.h b/drivers/gpio/gpiolib-of.h
index 34954921d96e..454d1658ee2d 100644
--- a/drivers/gpio/gpiolib-of.h
+++ b/drivers/gpio/gpiolib-of.h
@@ -16,7 +16,7 @@ struct gpio_desc *of_get_named_gpiod_flags(struct device_node *np,
int of_gpiochip_add(struct gpio_chip *gc);
void of_gpiochip_remove(struct gpio_chip *gc);
int of_gpio_get_count(struct device *dev, const char *con_id);
-bool of_gpio_need_valid_mask(struct gpio_chip *gc);
+bool of_gpio_need_valid_mask(const struct gpio_chip *gc);
#else
static inline struct gpio_desc *of_find_gpio(struct device *dev,
const char *con_id,
@@ -36,7 +36,7 @@ static inline int of_gpio_get_count(struct device *dev, const char *con_id)
{
return 0;
}
-static inline bool of_gpio_need_valid_mask(struct gpio_chip *gc)
+static inline bool of_gpio_need_valid_mask(const struct gpio_chip *gc)
{
return false;
}