Skip to content

Commit

Permalink
watchdog: omap_wdt: convert to devm_ functions
Browse files Browse the repository at this point in the history
Use devm_kzalloc(), devm_request_mem_region() ande devm_ioremap()
to simplify the code.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
  • Loading branch information
Aaro Koskinen authored and Wim Van Sebroeck committed Dec 19, 2012
1 parent 67c0f55 commit 4f4753d
Showing 1 changed file with 13 additions and 37 deletions.
50 changes: 13 additions & 37 deletions drivers/watchdog/omap_wdt.c
Original file line number Diff line number Diff line change
Expand Up @@ -208,40 +208,33 @@ static int omap_wdt_probe(struct platform_device *pdev)
u32 rs;
int ret;

omap_wdt = kzalloc(sizeof(*omap_wdt), GFP_KERNEL);
omap_wdt = devm_kzalloc(&pdev->dev, sizeof(*omap_wdt), GFP_KERNEL);
if (!omap_wdt)
return -ENOMEM;

/* reserve static register mappings */
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!res) {
ret = -ENOENT;
goto err_get_resource;
}
if (!res)
return -ENOENT;

mem = request_mem_region(res->start, resource_size(res), pdev->name);
if (!mem) {
ret = -EBUSY;
goto err_busy;
}
mem = devm_request_mem_region(&pdev->dev, res->start,
resource_size(res), pdev->name);
if (!mem)
return -EBUSY;

wdev = kzalloc(sizeof(struct omap_wdt_dev), GFP_KERNEL);
if (!wdev) {
ret = -ENOMEM;
goto err_kzalloc;
}
wdev = devm_kzalloc(&pdev->dev, sizeof(*wdev), GFP_KERNEL);
if (!wdev)
return -ENOMEM;

wdev->omap_wdt_users = false;
wdev->mem = mem;
wdev->dev = &pdev->dev;
wdev->wdt_trgr_pattern = 0x1234;
mutex_init(&wdev->lock);

wdev->base = ioremap(res->start, resource_size(res));
if (!wdev->base) {
ret = -ENOMEM;
goto err_ioremap;
}
wdev->base = devm_ioremap(&pdev->dev, res->start, resource_size(res));
if (!wdev->base)
return -ENOMEM;

omap_wdt->info = &omap_wdt_info;
omap_wdt->ops = &omap_wdt_ops;
Expand Down Expand Up @@ -286,18 +279,7 @@ static int omap_wdt_probe(struct platform_device *pdev)
err_register:
pm_runtime_disable(wdev->dev);
platform_set_drvdata(pdev, NULL);
iounmap(wdev->base);

err_ioremap:
wdev->base = NULL;
kfree(wdev);

err_kzalloc:
release_mem_region(res->start, resource_size(res));

err_busy:
err_get_resource:
kfree(omap_wdt);
return ret;
}

Expand All @@ -322,14 +304,8 @@ static int omap_wdt_remove(struct platform_device *pdev)

pm_runtime_disable(wdev->dev);
watchdog_unregister_device(wdog);
release_mem_region(res->start, resource_size(res));
platform_set_drvdata(pdev, NULL);

iounmap(wdev->base);

kfree(wdev);
kfree(wdog);

return 0;
}

Expand Down

0 comments on commit 4f4753d

Please sign in to comment.