Skip to content

Commit

Permalink
watchdog: JZ4740: Use devm_* functions
Browse files Browse the repository at this point in the history
- Use devm_clk_get instead of clk_get
- Use devm_watchdog_register_device instead of watchdog_register_device

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
Cc: Mathieu Malaterre <malat@debian.org>
Cc: linux-watchdog@vger.kernel.org
Cc: linux-mips@linux-mips.org
Signed-off-by: James Hogan <jhogan@kernel.org>
  • Loading branch information
Paul Cercueil authored and James Hogan committed May 14, 2018
1 parent 212c105 commit 6bdbc1f
Showing 1 changed file with 8 additions and 19 deletions.
27 changes: 8 additions & 19 deletions drivers/watchdog/jz4740_wdt.c
Original file line number Diff line number Diff line change
Expand Up @@ -179,40 +179,29 @@ static int jz4740_wdt_probe(struct platform_device *pdev)

res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
drvdata->base = devm_ioremap_resource(&pdev->dev, res);
if (IS_ERR(drvdata->base)) {
ret = PTR_ERR(drvdata->base);
goto err_out;
}
if (IS_ERR(drvdata->base))
return PTR_ERR(drvdata->base);

drvdata->rtc_clk = clk_get(&pdev->dev, "rtc");
drvdata->rtc_clk = devm_clk_get(&pdev->dev, "rtc");
if (IS_ERR(drvdata->rtc_clk)) {
dev_err(&pdev->dev, "cannot find RTC clock\n");
ret = PTR_ERR(drvdata->rtc_clk);
goto err_out;
return PTR_ERR(drvdata->rtc_clk);
}

ret = watchdog_register_device(&drvdata->wdt);
ret = devm_watchdog_register_device(&pdev->dev, &drvdata->wdt);
if (ret < 0)
goto err_disable_clk;
return ret;

platform_set_drvdata(pdev, drvdata);
return 0;

err_disable_clk:
clk_put(drvdata->rtc_clk);
err_out:
return ret;
return 0;
}

static int jz4740_wdt_remove(struct platform_device *pdev)
{
struct jz4740_wdt_drvdata *drvdata = platform_get_drvdata(pdev);

jz4740_wdt_stop(&drvdata->wdt);
watchdog_unregister_device(&drvdata->wdt);
clk_put(drvdata->rtc_clk);

return 0;
return jz4740_wdt_stop(&drvdata->wdt);
}

static struct platform_driver jz4740_wdt_driver = {
Expand Down

0 comments on commit 6bdbc1f

Please sign in to comment.