mirror of
https://github.com/torvalds/linux.git
synced 2024-11-11 22:51:42 +00:00
watchdog: omap_wdt: convert to devm_ functions
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>
This commit is contained in:
parent
67c0f55468
commit
4f4753d96d
@ -208,28 +208,23 @@ 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;
|
||||
@ -237,11 +232,9 @@ static int omap_wdt_probe(struct platform_device *pdev)
|
||||
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;
|
||||
@ -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;
|
||||
}
|
||||
|
||||
@ -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;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user