Skip to content

Commit

Permalink
ASoC: omap-dmic: Use devm_clk_get
Browse files Browse the repository at this point in the history
This patch introduces the use of managed interfaces like devm_clk_get
and does away with the clk_puts in the probe and remove functions. A
label is also done away with.

Signed-off-by: Himangi Saraogi <himangi774@gmail.com>
Acked-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
  • Loading branch information
Himangi Saraogi authored and Mark Brown committed Jul 14, 2014
1 parent 159baad commit 9cb0fe9
Showing 1 changed file with 6 additions and 23 deletions.
29 changes: 6 additions & 23 deletions sound/soc/omap/omap-dmic.c
Original file line number Diff line number Diff line change
Expand Up @@ -466,7 +466,7 @@ static int asoc_dmic_probe(struct platform_device *pdev)

mutex_init(&dmic->mutex);

dmic->fclk = clk_get(dmic->dev, "fck");
dmic->fclk = devm_clk_get(dmic->dev, "fck");
if (IS_ERR(dmic->fclk)) {
dev_err(dmic->dev, "cant get fck\n");
return -ENODEV;
Expand All @@ -475,43 +475,27 @@ static int asoc_dmic_probe(struct platform_device *pdev)
res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "dma");
if (!res) {
dev_err(dmic->dev, "invalid dma memory resource\n");
ret = -ENODEV;
goto err_put_clk;
return -ENODEV;
}
dmic->dma_data.addr = res->start + OMAP_DMIC_DATA_REG;

dmic->dma_data.filter_data = "up_link";

res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mpu");
dmic->io_base = devm_ioremap_resource(&pdev->dev, res);
if (IS_ERR(dmic->io_base)) {
ret = PTR_ERR(dmic->io_base);
goto err_put_clk;
}
if (IS_ERR(dmic->io_base))
return PTR_ERR(dmic->io_base);


ret = devm_snd_soc_register_component(&pdev->dev,
&omap_dmic_component,
&omap_dmic_dai, 1);
if (ret)
goto err_put_clk;
return ret;

ret = omap_pcm_platform_register(&pdev->dev);
if (ret)
goto err_put_clk;

return 0;

err_put_clk:
clk_put(dmic->fclk);
return ret;
}

static int asoc_dmic_remove(struct platform_device *pdev)
{
struct omap_dmic *dmic = platform_get_drvdata(pdev);

clk_put(dmic->fclk);
return ret;

return 0;
}
Expand All @@ -529,7 +513,6 @@ static struct platform_driver asoc_dmic_driver = {
.of_match_table = omap_dmic_of_match,
},
.probe = asoc_dmic_probe,
.remove = asoc_dmic_remove,
};

module_platform_driver(asoc_dmic_driver);
Expand Down

0 comments on commit 9cb0fe9

Please sign in to comment.