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 6f17c935a6cf86b1fcb47856532a8d6374bd0d82..39e93f631c373600cc9753e9397bf2f48b64c1d7 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;
 }
 
This page took 0.037182 seconds and 5 git commands to generate.