watchdog: imx2_wdt: use devm_clk_get()
authorJingoo Han <jg1.han@samsung.com>
Mon, 29 Apr 2013 09:15:53 +0000 (18:15 +0900)
committerWim Van Sebroeck <wim@iguana.be>
Thu, 11 Jul 2013 19:01:06 +0000 (21:01 +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/imx2_wdt.c

index 62946c2cb4f8b1d579f42cd18c5e72a6e4531434..693ac3f4de5aaba92d2a2a7f7004be468b1b0b77 100644 (file)
@@ -261,7 +261,7 @@ static int __init imx2_wdt_probe(struct platform_device *pdev)
        if (IS_ERR(imx2_wdt.base))
                return PTR_ERR(imx2_wdt.base);
 
-       imx2_wdt.clk = clk_get(&pdev->dev, NULL);
+       imx2_wdt.clk = devm_clk_get(&pdev->dev, NULL);
        if (IS_ERR(imx2_wdt.clk)) {
                dev_err(&pdev->dev, "can't get Watchdog clock\n");
                return PTR_ERR(imx2_wdt.clk);
@@ -286,7 +286,6 @@ static int __init imx2_wdt_probe(struct platform_device *pdev)
 
 fail:
        imx2_wdt_miscdev.parent = NULL;
-       clk_put(imx2_wdt.clk);
        return ret;
 }
 
@@ -299,8 +298,7 @@ static int __exit imx2_wdt_remove(struct platform_device *pdev)
 
                dev_crit(imx2_wdt_miscdev.parent,
                        "Device removed: Expect reboot!\n");
-       } else
-               clk_put(imx2_wdt.clk);
+       }
 
        imx2_wdt_miscdev.parent = NULL;
        return 0;