watchdog: shwdt: use devm_clk_get()
authorJingoo Han <jg1.han@samsung.com>
Mon, 29 Apr 2013 09:16:33 +0000 (18:16 +0900)
committerWim Van Sebroeck <wim@iguana.be>
Thu, 11 Jul 2013 19:02:32 +0000 (21:02 +0200)
Use devm_clk_get() to make cleanup paths more simple.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/shwdt.c

index 6185af2b33109c5ce8c1ef71a93b35eb2542bfdd..ea2154bfe5abdc4974005cfe597a1c2581295f87 100644 (file)
@@ -241,7 +241,7 @@ static int sh_wdt_probe(struct platform_device *pdev)
 
        wdt->dev = &pdev->dev;
 
-       wdt->clk = clk_get(&pdev->dev, NULL);
+       wdt->clk = devm_clk_get(&pdev->dev, NULL);
        if (IS_ERR(wdt->clk)) {
                /*
                 * Clock framework support is optional, continue on
@@ -251,10 +251,8 @@ static int sh_wdt_probe(struct platform_device *pdev)
        }
 
        wdt->base = devm_ioremap_resource(wdt->dev, res);
-       if (IS_ERR(wdt->base)) {
-               rc = PTR_ERR(wdt->base);
-               goto err;
-       }
+       if (IS_ERR(wdt->base))
+               return PTR_ERR(wdt->base);
 
        watchdog_set_nowayout(&sh_wdt_dev, nowayout);
        watchdog_set_drvdata(&sh_wdt_dev, wdt);
@@ -277,7 +275,7 @@ static int sh_wdt_probe(struct platform_device *pdev)
        rc = watchdog_register_device(&sh_wdt_dev);
        if (unlikely(rc)) {
                dev_err(&pdev->dev, "Can't register watchdog (err=%d)\n", rc);
-               goto err;
+               return rc;
        }
 
        init_timer(&wdt->timer);
@@ -292,11 +290,6 @@ static int sh_wdt_probe(struct platform_device *pdev)
        pm_runtime_enable(&pdev->dev);
 
        return 0;
-
-err:
-       clk_put(wdt->clk);
-
-       return rc;
 }
 
 static int sh_wdt_remove(struct platform_device *pdev)
@@ -308,7 +301,6 @@ static int sh_wdt_remove(struct platform_device *pdev)
        watchdog_unregister_device(&sh_wdt_dev);
 
        pm_runtime_disable(&pdev->dev);
-       clk_put(wdt->clk);
 
        return 0;
 }