Commit 81930328 authored by Grygorii Strashko's avatar Grygorii Strashko Committed by Linus Walleij

gpio: omap: initialize gpioirq chip as part of gpiochip_add_data

Hence, series from Thierry Reding [1] merged - the OMAP GPIO driver can be
switched to reuse new feature and just fill new struct gpio_irq_chip before
calling gpiochip_add_data() instead of using few separate gpioirq chip
APIs. gpiochip_add_data() will do the job and create and initialize gpioirq
chip

[1] https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1531592.htmlSigned-off-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 588fc3bc
...@@ -1058,6 +1058,7 @@ static void omap_gpio_mod_init(struct gpio_bank *bank) ...@@ -1058,6 +1058,7 @@ static void omap_gpio_mod_init(struct gpio_bank *bank)
static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc) static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc)
{ {
struct gpio_irq_chip *irq;
static int gpio; static int gpio;
int irq_base = 0; int irq_base = 0;
int ret; int ret;
...@@ -1085,16 +1086,6 @@ static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc) ...@@ -1085,16 +1086,6 @@ static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc)
} }
bank->chip.ngpio = bank->width; bank->chip.ngpio = bank->width;
ret = gpiochip_add_data(&bank->chip, bank);
if (ret) {
dev_err(bank->chip.parent,
"Could not register gpio chip %d\n", ret);
return ret;
}
if (!bank->is_mpuio)
gpio += bank->width;
#ifdef CONFIG_ARCH_OMAP1 #ifdef CONFIG_ARCH_OMAP1
/* /*
* REVISIT: Once we have OMAP1 supporting SPARSE_IRQ, we can drop * REVISIT: Once we have OMAP1 supporting SPARSE_IRQ, we can drop
...@@ -1115,25 +1106,30 @@ static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc) ...@@ -1115,25 +1106,30 @@ static int omap_gpio_chip_init(struct gpio_bank *bank, struct irq_chip *irqc)
irqc->irq_set_wake = NULL; irqc->irq_set_wake = NULL;
} }
ret = gpiochip_irqchip_add(&bank->chip, irqc, irq = &bank->chip.irq;
irq_base, handle_bad_irq, irq->chip = irqc;
IRQ_TYPE_NONE); irq->handler = handle_bad_irq;
irq->default_type = IRQ_TYPE_NONE;
irq->num_parents = 1;
irq->parents = &bank->irq;
irq->first = irq_base;
ret = gpiochip_add_data(&bank->chip, bank);
if (ret) { if (ret) {
dev_err(bank->chip.parent, dev_err(bank->chip.parent,
"Couldn't add irqchip to gpiochip %d\n", ret); "Could not register gpio chip %d\n", ret);
gpiochip_remove(&bank->chip); return ret;
return -ENODEV;
} }
gpiochip_set_chained_irqchip(&bank->chip, irqc, bank->irq, NULL);
ret = devm_request_irq(bank->chip.parent, bank->irq, ret = devm_request_irq(bank->chip.parent, bank->irq,
omap_gpio_irq_handler, omap_gpio_irq_handler,
0, dev_name(bank->chip.parent), bank); 0, dev_name(bank->chip.parent), bank);
if (ret) if (ret)
gpiochip_remove(&bank->chip); gpiochip_remove(&bank->chip);
if (!bank->is_mpuio)
gpio += bank->width;
return ret; return ret;
} }
......
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