Skip to content

Commit 8d3739d

Browse files
glneomathieupoirier
authored andcommitted
remoteproc: omap: Use devm_rproc_add() helper
Use the device lifecycle managed add function. This helps prevent mistakes like deleting out of order in cleanup functions and forgetting to delete on error paths. Signed-off-by: Andrew Davis <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Mathieu Poirier <[email protected]>
1 parent 5efcc36 commit 8d3739d

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

drivers/remoteproc/omap_remoteproc.c

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1359,20 +1359,13 @@ static int omap_rproc_probe(struct platform_device *pdev)
13591359

13601360
platform_set_drvdata(pdev, rproc);
13611361

1362-
ret = rproc_add(rproc);
1362+
ret = devm_rproc_add(&pdev->dev, rproc);
13631363
if (ret)
13641364
return ret;
13651365

13661366
return 0;
13671367
}
13681368

1369-
static void omap_rproc_remove(struct platform_device *pdev)
1370-
{
1371-
struct rproc *rproc = platform_get_drvdata(pdev);
1372-
1373-
rproc_del(rproc);
1374-
}
1375-
13761369
static const struct dev_pm_ops omap_rproc_pm_ops = {
13771370
SET_SYSTEM_SLEEP_PM_OPS(omap_rproc_suspend, omap_rproc_resume)
13781371
SET_RUNTIME_PM_OPS(omap_rproc_runtime_suspend,
@@ -1381,7 +1374,6 @@ static const struct dev_pm_ops omap_rproc_pm_ops = {
13811374

13821375
static struct platform_driver omap_rproc_driver = {
13831376
.probe = omap_rproc_probe,
1384-
.remove_new = omap_rproc_remove,
13851377
.driver = {
13861378
.name = "omap-rproc",
13871379
.pm = &omap_rproc_pm_ops,

0 commit comments

Comments
 (0)