watchdog: omap: don't disable runtime pm before starting device
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 15 Dec 2015 10:37:40 +0000 (11:37 +0100)
committerWim Van Sebroeck <wim@iguana.be>
Sun, 27 Dec 2015 20:09:36 +0000 (21:09 +0100)
omap_wdt_start calls pm_runtime_get_sync so dropping a reference just
before calling omap_wdt_start doesn't make much sense. Moreover there is
no point to use the synchronous variant of pm_runtime_put because the
driver doesn't care if the clock is disabled before or after
omap_wdt_probe returns.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/omap_wdt.c

index 6f17c93..39e93f6 100644 (file)
@@ -283,11 +283,11 @@ static int omap_wdt_probe(struct platform_device *pdev)
                readl_relaxed(wdev->base + OMAP_WATCHDOG_REV) & 0xFF,
                wdev->wdog.timeout);
 
-       pm_runtime_put_sync(wdev->dev);
-
        if (early_enable)
                omap_wdt_start(&wdev->wdog);
 
+       pm_runtime_put(wdev->dev);
+
        return 0;
 }