gpio: omap: use devm_irq_alloc_descs()
authorBartosz Golaszewski <bgolaszewski@baylibre.com>
Sat, 4 Mar 2017 16:23:31 +0000 (17:23 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 15 Mar 2017 10:16:33 +0000 (11:16 +0100)
This driver never frees the allocated interrupt descriptors. Fix it by
using a resource managed variant of irq_alloc_descs().

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-omap.c

index efc85a2..5d6a574 100644 (file)
@@ -1085,7 +1085,8 @@ static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc)
         * REVISIT: Once we have OMAP1 supporting SPARSE_IRQ, we can drop
         * irq_alloc_descs() since a base IRQ offset will no longer be needed.
         */
-       irq_base = irq_alloc_descs(-1, 0, bank->width, 0);
+       irq_base = devm_irq_alloc_descs(bank->chip.parent,
+                                       -1, 0, bank->width, 0);
        if (irq_base < 0) {
                dev_err(bank->chip.parent, "Couldn't allocate IRQ numbers\n");
                return -ENODEV;