Commit 2911ee9e authored by Alexandre Belloni's avatar Alexandre Belloni

rtc: sirfsoc: convert to devm_rtc_allocate_device

This allows further improvement of the driver. Also remove the unnecessary
error string as the core will already display error messages.

Link: https://lore.kernel.org/r/20200305160452.27808-1-alexandre.belloni@bootlin.comSigned-off-by: default avatarAlexandre Belloni <alexandre.belloni@bootlin.com>
parent b427ca8c
...@@ -341,28 +341,21 @@ static int sirfsoc_rtc_probe(struct platform_device *pdev) ...@@ -341,28 +341,21 @@ static int sirfsoc_rtc_probe(struct platform_device *pdev)
rtcdrv->overflow_rtc = rtcdrv->overflow_rtc =
sirfsoc_rtc_readl(rtcdrv, RTC_SW_VALUE); sirfsoc_rtc_readl(rtcdrv, RTC_SW_VALUE);
rtcdrv->rtc = devm_rtc_device_register(&pdev->dev, pdev->name, rtcdrv->rtc = devm_rtc_allocate_device(&pdev->dev);
&sirfsoc_rtc_ops, THIS_MODULE); if (IS_ERR(rtcdrv->rtc))
if (IS_ERR(rtcdrv->rtc)) { return PTR_ERR(rtcdrv->rtc);
err = PTR_ERR(rtcdrv->rtc);
dev_err(&pdev->dev, "can't register RTC device\n"); rtcdrv->rtc->ops = &sirfsoc_rtc_ops;
return err;
}
rtcdrv->irq = platform_get_irq(pdev, 0); rtcdrv->irq = platform_get_irq(pdev, 0);
err = devm_request_irq( err = devm_request_irq(&pdev->dev, rtcdrv->irq, sirfsoc_rtc_irq_handler,
&pdev->dev, IRQF_SHARED, pdev->name, rtcdrv);
rtcdrv->irq,
sirfsoc_rtc_irq_handler,
IRQF_SHARED,
pdev->name,
rtcdrv);
if (err) { if (err) {
dev_err(&pdev->dev, "Unable to register for the SiRF SOC RTC IRQ\n"); dev_err(&pdev->dev, "Unable to register for the SiRF SOC RTC IRQ\n");
return err; return err;
} }
return 0; return rtc_register_device(rtcdrv->rtc);
} }
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
......
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