Replace the hand-rolled da8xx_rproc_mem_release() devres action with devm_of_reserved_mem_device_init(), which handles the cleanup automatically. This removes both the wrapper function and the devm_add_action_or_reset() call.
Signed-off-by: Mukesh Ojha <[email protected]> --- drivers/remoteproc/da8xx_remoteproc.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/remoteproc/da8xx_remoteproc.c b/drivers/remoteproc/da8xx_remoteproc.c index 23fca7176539..cd9a8f2b3163 100644 --- a/drivers/remoteproc/da8xx_remoteproc.c +++ b/drivers/remoteproc/da8xx_remoteproc.c @@ -230,13 +230,6 @@ static int da8xx_rproc_get_internal_memories(struct platform_device *pdev, return 0; } -static void da8xx_rproc_mem_release(void *data) -{ - struct device *dev = data; - - of_reserved_mem_device_release(dev); -} - static int da8xx_rproc_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; @@ -267,10 +260,9 @@ static int da8xx_rproc_probe(struct platform_device *pdev) "unable to get reset control\n"); if (dev->of_node) { - ret = of_reserved_mem_device_init(dev); + ret = devm_of_reserved_mem_device_init(dev); if (ret) return dev_err_probe(dev, ret, "device does not have specific CMA pool\n"); - devm_add_action_or_reset(&pdev->dev, da8xx_rproc_mem_release, &pdev->dev); } ret = da8xx_rproc_get_internal_memories(pdev, drproc); -- 2.53.0
