Skip to content

Commit

Permalink
ASoC: Tegra DAS: Use devm_ APIs and module_platform_driver
Browse files Browse the repository at this point in the history
module_platform_drive saves some boiler-plate code.

The devm_ APIs remove the need to manually clean up allocations,
thus removing some code.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
  • Loading branch information
Stephen Warren authored and Mark Brown committed Nov 23, 2011
1 parent 99c92ae commit f2296d7
Showing 1 changed file with 10 additions and 35 deletions.
45 changes: 10 additions & 35 deletions sound/soc/tegra/tegra_das.c
Original file line number Diff line number Diff line change
Expand Up @@ -172,34 +172,34 @@ static int __devinit tegra_das_probe(struct platform_device *pdev)
if (das)
return -ENODEV;

das = kzalloc(sizeof(struct tegra_das), GFP_KERNEL);
das = devm_kzalloc(&pdev->dev, sizeof(struct tegra_das), GFP_KERNEL);
if (!das) {
dev_err(&pdev->dev, "Can't allocate tegra_das\n");
ret = -ENOMEM;
goto exit;
goto err;
}
das->dev = &pdev->dev;

res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!res) {
dev_err(&pdev->dev, "No memory resource\n");
ret = -ENODEV;
goto err_free;
goto err;
}

region = request_mem_region(res->start, resource_size(res),
pdev->name);
region = devm_request_mem_region(&pdev->dev, res->start,
resource_size(res), pdev->name);
if (!region) {
dev_err(&pdev->dev, "Memory region already claimed\n");
ret = -EBUSY;
goto err_free;
goto err;
}

das->regs = ioremap(res->start, resource_size(res));
das->regs = devm_ioremap(&pdev->dev, res->start, resource_size(res));
if (!das->regs) {
dev_err(&pdev->dev, "ioremap failed\n");
ret = -ENOMEM;
goto err_release;
goto err;
}

tegra_das_debug_add(das);
Expand All @@ -208,32 +208,18 @@ static int __devinit tegra_das_probe(struct platform_device *pdev)

return 0;

err_release:
release_mem_region(res->start, resource_size(res));
err_free:
kfree(das);
err:
das = NULL;
exit:
return ret;
}

static int __devexit tegra_das_remove(struct platform_device *pdev)
{
struct resource *res;

if (!das)
return -ENODEV;

platform_set_drvdata(pdev, NULL);

tegra_das_debug_remove(das);

iounmap(das->regs);

res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
release_mem_region(res->start, resource_size(res));

kfree(das);
das = NULL;

return 0;
Expand All @@ -246,18 +232,7 @@ static struct platform_driver tegra_das_driver = {
.name = DRV_NAME,
},
};

static int __init tegra_das_modinit(void)
{
return platform_driver_register(&tegra_das_driver);
}
module_init(tegra_das_modinit);

static void __exit tegra_das_modexit(void)
{
platform_driver_unregister(&tegra_das_driver);
}
module_exit(tegra_das_modexit);
module_platform_driver(tegra_das_driver);

MODULE_AUTHOR("Stephen Warren <swarren@nvidia.com>");
MODULE_DESCRIPTION("Tegra DAS driver");
Expand Down

0 comments on commit f2296d7

Please sign in to comment.