Commit f8ad8aa5 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Linus Walleij

gpio: vr41xx: Bail out on gpiochip_lock_as_irq() error

gpiochip_lock_as_irq() may return a few error codes,
bail out if it fails with corresponding returned code.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent d124339d
...@@ -138,10 +138,16 @@ static void unmask_giuint_low(struct irq_data *d) ...@@ -138,10 +138,16 @@ static void unmask_giuint_low(struct irq_data *d)
static unsigned int startup_giuint(struct irq_data *data) static unsigned int startup_giuint(struct irq_data *data)
{ {
if (gpiochip_lock_as_irq(&vr41xx_gpio_chip, data->hwirq)) int ret;
ret = gpiochip_lock_as_irq(&vr41xx_gpio_chip, irqd_to_hwirq(data));
if (ret) {
dev_err(vr41xx_gpio_chip.parent, dev_err(vr41xx_gpio_chip.parent,
"unable to lock HW IRQ %lu for IRQ\n", "unable to lock HW IRQ %lu for IRQ\n",
data->hwirq); data->hwirq);
return ret;
}
/* Satisfy the .enable semantics by unmasking the line */ /* Satisfy the .enable semantics by unmasking the line */
unmask_giuint_low(data); unmask_giuint_low(data);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment