| Message ID | 20180110015848.11480-2-sboyd@codeaurora.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue 09 Jan 17:58 PST 2018, Stephen Boyd wrote: > Some pinctrl drivers can use the gpiochip irq valid information > to figure out if certain gpios are exposed to the kernel for > usage or not. Expose this API so we can use it in the > pinmux_ops::request ops. > > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org> Regards, Bjorn > --- > drivers/gpio/gpiolib.c | 5 +++-- > include/linux/gpio/driver.h | 3 +++ > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c > index b80936a25caa..c18b7b60ea1d 100644 > --- a/drivers/gpio/gpiolib.c > +++ b/drivers/gpio/gpiolib.c > @@ -1503,14 +1503,15 @@ static void gpiochip_irqchip_free_valid_mask(struct gpio_chip *gpiochip) > gpiochip->irq.valid_mask = NULL; > } > > -static bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, > - unsigned int offset) > +bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, > + unsigned int offset) > { > /* No mask means all valid */ > if (likely(!gpiochip->irq.valid_mask)) > return true; > return test_bit(offset, gpiochip->irq.valid_mask); > } > +EXPORT_SYMBOL_GPL(gpiochip_irqchip_irq_valid); > > /** > * gpiochip_set_cascaded_irqchip() - connects a cascaded irqchip to a gpiochip > diff --git a/include/linux/gpio/driver.h b/include/linux/gpio/driver.h > index 7258cd676df4..1ba9a331ec51 100644 > --- a/include/linux/gpio/driver.h > +++ b/include/linux/gpio/driver.h > @@ -436,6 +436,9 @@ int gpiochip_irqchip_add_key(struct gpio_chip *gpiochip, > struct lock_class_key *lock_key, > struct lock_class_key *request_key); > > +bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, > + unsigned int offset); > + > #ifdef CONFIG_LOCKDEP > > /* > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project >
On Wed, Jan 10, 2018 at 2:58 AM, Stephen Boyd <sboyd@codeaurora.org> wrote: > Some pinctrl drivers can use the gpiochip irq valid information > to figure out if certain gpios are exposed to the kernel for > usage or not. Expose this API so we can use it in the > pinmux_ops::request ops. > > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> Makes a lot of sense. Patch applied with Björn's ACK. Yours, Linus Walleij
diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c index b80936a25caa..c18b7b60ea1d 100644 --- a/drivers/gpio/gpiolib.c +++ b/drivers/gpio/gpiolib.c @@ -1503,14 +1503,15 @@ static void gpiochip_irqchip_free_valid_mask(struct gpio_chip *gpiochip) gpiochip->irq.valid_mask = NULL; } -static bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, - unsigned int offset) +bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, + unsigned int offset) { /* No mask means all valid */ if (likely(!gpiochip->irq.valid_mask)) return true; return test_bit(offset, gpiochip->irq.valid_mask); } +EXPORT_SYMBOL_GPL(gpiochip_irqchip_irq_valid); /** * gpiochip_set_cascaded_irqchip() - connects a cascaded irqchip to a gpiochip diff --git a/include/linux/gpio/driver.h b/include/linux/gpio/driver.h index 7258cd676df4..1ba9a331ec51 100644 --- a/include/linux/gpio/driver.h +++ b/include/linux/gpio/driver.h @@ -436,6 +436,9 @@ int gpiochip_irqchip_add_key(struct gpio_chip *gpiochip, struct lock_class_key *lock_key, struct lock_class_key *request_key); +bool gpiochip_irqchip_irq_valid(const struct gpio_chip *gpiochip, + unsigned int offset); + #ifdef CONFIG_LOCKDEP /*
Some pinctrl drivers can use the gpiochip irq valid information to figure out if certain gpios are exposed to the kernel for usage or not. Expose this API so we can use it in the pinmux_ops::request ops. Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- drivers/gpio/gpiolib.c | 5 +++-- include/linux/gpio/driver.h | 3 +++ 2 files changed, 6 insertions(+), 2 deletions(-)