Commit 8d431dbe authored by David Brownell's avatar David Brownell Committed by Linus Torvalds

rtc-at32ap700x: fix irq init oops

Reorder at32_rtc_probe() so that it's safe (no oopsing) to fire the
IRQ handler the instant that it's registered.  (Bug noted via "Debug
shared IRQ handlers" kernel debug option.)
Signed-off-by: default avatarDavid Brownell <dbrownell@users.sourceforge.net>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: <hcegtvedt@atmel.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent fe4304ba
...@@ -225,18 +225,12 @@ static int __init at32_rtc_probe(struct platform_device *pdev) ...@@ -225,18 +225,12 @@ static int __init at32_rtc_probe(struct platform_device *pdev)
goto out; goto out;
} }
ret = request_irq(irq, at32_rtc_interrupt, IRQF_SHARED, "rtc", rtc);
if (ret) {
dev_dbg(&pdev->dev, "could not request irq %d\n", irq);
goto out;
}
rtc->irq = irq; rtc->irq = irq;
rtc->regs = ioremap(regs->start, regs->end - regs->start + 1); rtc->regs = ioremap(regs->start, regs->end - regs->start + 1);
if (!rtc->regs) { if (!rtc->regs) {
ret = -ENOMEM; ret = -ENOMEM;
dev_dbg(&pdev->dev, "could not map I/O memory\n"); dev_dbg(&pdev->dev, "could not map I/O memory\n");
goto out_free_irq; goto out;
} }
spin_lock_init(&rtc->lock); spin_lock_init(&rtc->lock);
...@@ -253,12 +247,18 @@ static int __init at32_rtc_probe(struct platform_device *pdev) ...@@ -253,12 +247,18 @@ static int __init at32_rtc_probe(struct platform_device *pdev)
| RTC_BIT(CTRL_EN)); | RTC_BIT(CTRL_EN));
} }
ret = request_irq(irq, at32_rtc_interrupt, IRQF_SHARED, "rtc", rtc);
if (ret) {
dev_dbg(&pdev->dev, "could not request irq %d\n", irq);
goto out_iounmap;
}
rtc->rtc = rtc_device_register(pdev->name, &pdev->dev, rtc->rtc = rtc_device_register(pdev->name, &pdev->dev,
&at32_rtc_ops, THIS_MODULE); &at32_rtc_ops, THIS_MODULE);
if (IS_ERR(rtc->rtc)) { if (IS_ERR(rtc->rtc)) {
dev_dbg(&pdev->dev, "could not register rtc device\n"); dev_dbg(&pdev->dev, "could not register rtc device\n");
ret = PTR_ERR(rtc->rtc); ret = PTR_ERR(rtc->rtc);
goto out_iounmap; goto out_free_irq;
} }
platform_set_drvdata(pdev, rtc); platform_set_drvdata(pdev, rtc);
...@@ -268,10 +268,10 @@ static int __init at32_rtc_probe(struct platform_device *pdev) ...@@ -268,10 +268,10 @@ static int __init at32_rtc_probe(struct platform_device *pdev)
return 0; return 0;
out_iounmap:
iounmap(rtc->regs);
out_free_irq: out_free_irq:
free_irq(irq, rtc); free_irq(irq, rtc);
out_iounmap:
iounmap(rtc->regs);
out: out:
kfree(rtc); kfree(rtc);
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