diff options
author | Grygorii Strashko <grygorii.strashko@ti.com> | 2015-08-18 07:10:53 -0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2015-08-26 03:52:43 -0400 |
commit | 89d18e3af8b99481589e07f92fd966ceb86eff5d (patch) | |
tree | 333c93ecf7d8cc9077ae700388b88b3d2e87592e /drivers/gpio/gpio-omap.c | |
parent | bb8cdf9510d5d2b67126b36747c51d531026215d (diff) |
gpio: omap: switch to use platform_get_irq
Switch OMAP GPIO driver to use platform_get_irq(), because
it is not recommened to use platform_get_resource(pdev, IORESOURCE_IRQ, ..)
for requesting IRQ resources any more, as they can be not ready yet
in case of DT-boot.
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
Acked-by: Santosh Shilimkar <ssantosh@kernel.org>
Tested-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/gpio/gpio-omap.c')
-rw-r--r-- | drivers/gpio/gpio-omap.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c index f38b01b5869c..03fd111e2d17 100644 --- a/drivers/gpio/gpio-omap.c +++ b/drivers/gpio/gpio-omap.c | |||
@@ -1178,13 +1178,16 @@ static int omap_gpio_probe(struct platform_device *pdev) | |||
1178 | irqc->irq_set_wake = omap_gpio_wake_enable, | 1178 | irqc->irq_set_wake = omap_gpio_wake_enable, |
1179 | irqc->name = dev_name(&pdev->dev); | 1179 | irqc->name = dev_name(&pdev->dev); |
1180 | 1180 | ||
1181 | res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); | 1181 | bank->irq = platform_get_irq(pdev, 0); |
1182 | if (unlikely(!res)) { | 1182 | if (bank->irq <= 0) { |
1183 | dev_err(dev, "Invalid IRQ resource\n"); | 1183 | if (!bank->irq) |
1184 | return -ENODEV; | 1184 | bank->irq = -ENXIO; |
1185 | if (bank->irq != -EPROBE_DEFER) | ||
1186 | dev_err(dev, | ||
1187 | "can't get irq resource ret=%d\n", bank->irq); | ||
1188 | return bank->irq; | ||
1185 | } | 1189 | } |
1186 | 1190 | ||
1187 | bank->irq = res->start; | ||
1188 | bank->dev = dev; | 1191 | bank->dev = dev; |
1189 | bank->chip.dev = dev; | 1192 | bank->chip.dev = dev; |
1190 | bank->chip.owner = THIS_MODULE; | 1193 | bank->chip.owner = THIS_MODULE; |