Commit 05007c10 authored by Joel Fernandes's avatar Joel Fernandes Committed by Herbert Xu

crypto: omap-aes - Convert kzalloc to devm_kzalloc

Use devm_kzalloc instead of kzalloc. With this change, there is no need to
call kfree in error/exit paths.
Signed-off-by: default avatarJoel Fernandes <joelf@ti.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 6242332f
...@@ -1158,7 +1158,7 @@ static int omap_aes_probe(struct platform_device *pdev) ...@@ -1158,7 +1158,7 @@ static int omap_aes_probe(struct platform_device *pdev)
int err = -ENOMEM, i, j, irq = -1; int err = -ENOMEM, i, j, irq = -1;
u32 reg; u32 reg;
dd = kzalloc(sizeof(struct omap_aes_dev), GFP_KERNEL); dd = devm_kzalloc(dev, sizeof(struct omap_aes_dev), GFP_KERNEL);
if (dd == NULL) { if (dd == NULL) {
dev_err(dev, "unable to alloc data struct.\n"); dev_err(dev, "unable to alloc data struct.\n");
goto err_data; goto err_data;
...@@ -1251,7 +1251,6 @@ static int omap_aes_probe(struct platform_device *pdev) ...@@ -1251,7 +1251,6 @@ static int omap_aes_probe(struct platform_device *pdev)
tasklet_kill(&dd->queue_task); tasklet_kill(&dd->queue_task);
pm_runtime_disable(dev); pm_runtime_disable(dev);
err_res: err_res:
kfree(dd);
dd = NULL; dd = NULL;
err_data: err_data:
dev_err(dev, "initialization failed.\n"); dev_err(dev, "initialization failed.\n");
...@@ -1279,7 +1278,6 @@ static int omap_aes_remove(struct platform_device *pdev) ...@@ -1279,7 +1278,6 @@ static int omap_aes_remove(struct platform_device *pdev)
tasklet_kill(&dd->queue_task); tasklet_kill(&dd->queue_task);
omap_aes_dma_cleanup(dd); omap_aes_dma_cleanup(dd);
pm_runtime_disable(dd->dev); pm_runtime_disable(dd->dev);
kfree(dd);
dd = NULL; dd = NULL;
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment