rtc: omap: fix class-device registration
authorJohan Hovold <johan@kernel.org>
Wed, 10 Dec 2014 23:52:43 +0000 (15:52 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 11 Dec 2014 01:41:13 +0000 (17:41 -0800)
Make sure not to register the class device until after the device has
been configured.

Currently, the device is not fully configured (e.g. 24-hour mode) when
the class device is registered, something which involves driver
callbacks for example to read the current time.

Signed-off-by: Johan Hovold <johan@kernel.org>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Benot Cousson <bcousson@baylibre.com>
Cc: Lokesh Vutla <lokeshvutla@ti.com>
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Sekhar Nori <nsekhar@ti.com>
Cc: Tero Kristo <t-kristo@ti.com>
Cc: Keerthy J <j-keerthy@ti.com>
Tested-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/rtc/rtc-omap.c

index 6b10db5a5702eba62b49a893d2d58bc9dda23154..813bed27086788cabb1ba443b5fbbb5b8e9b707f 100644 (file)
@@ -413,16 +413,6 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
                rtc_writel(KICK1_VALUE, OMAP_RTC_KICK1_REG);
        }
 
-       device_init_wakeup(&pdev->dev, true);
-
-       rtc = devm_rtc_device_register(&pdev->dev, pdev->name,
-                       &omap_rtc_ops, THIS_MODULE);
-       if (IS_ERR(rtc)) {
-               ret = PTR_ERR(rtc);
-               goto err;
-       }
-       platform_set_drvdata(pdev, rtc);
-
        /*
         * disable interrupts
         *
@@ -446,19 +436,6 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
        if (reg & (u8) OMAP_RTC_STATUS_ALARM)
                rtc_write(OMAP_RTC_STATUS_ALARM, OMAP_RTC_STATUS_REG);
 
-       /* handle periodic and alarm irqs */
-       ret = devm_request_irq(&pdev->dev, omap_rtc_timer, rtc_irq, 0,
-                       dev_name(&rtc->dev), rtc);
-       if (ret)
-               goto err;
-
-       if (omap_rtc_timer != omap_rtc_alarm) {
-               ret = devm_request_irq(&pdev->dev, omap_rtc_alarm, rtc_irq, 0,
-                               dev_name(&rtc->dev), rtc);
-               if (ret)
-                       goto err;
-       }
-
        /* On boards with split power, RTC_ON_NOFF won't reset the RTC */
        reg = rtc_read(OMAP_RTC_CTRL_REG);
        if (reg & (u8) OMAP_RTC_CTRL_STOP)
@@ -488,6 +465,29 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
        if (reg != new_ctrl)
                rtc_write(new_ctrl, OMAP_RTC_CTRL_REG);
 
+       device_init_wakeup(&pdev->dev, true);
+
+       rtc = devm_rtc_device_register(&pdev->dev, pdev->name,
+                       &omap_rtc_ops, THIS_MODULE);
+       if (IS_ERR(rtc)) {
+               ret = PTR_ERR(rtc);
+               goto err;
+       }
+       platform_set_drvdata(pdev, rtc);
+
+       /* handle periodic and alarm irqs */
+       ret = devm_request_irq(&pdev->dev, omap_rtc_timer, rtc_irq, 0,
+                       dev_name(&rtc->dev), rtc);
+       if (ret)
+               goto err;
+
+       if (omap_rtc_timer != omap_rtc_alarm) {
+               ret = devm_request_irq(&pdev->dev, omap_rtc_alarm, rtc_irq, 0,
+                               dev_name(&rtc->dev), rtc);
+               if (ret)
+                       goto err;
+       }
+
        return 0;
 
 err: