diff options
| author | Tony Lindgren <tony@atomide.com> | 2016-11-09 09:47:58 -0700 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2016-11-14 10:57:21 +0530 |
| commit | 12f5908080bdccca2cb2f7ad850cb360c92f481a (patch) | |
| tree | a9a9863c7e5bcb9879d90c009c70f1bcd536f608 /drivers/dma/cppi41.c | |
| parent | f732c5b7c734cfc2c563c918fe2842175c7eb073 (diff) | |
| download | linux-12f5908080bdccca2cb2f7ad850cb360c92f481a.tar.gz linux-12f5908080bdccca2cb2f7ad850cb360c92f481a.tar.xz | |
dmaengine: cppi41: Fix list not empty warning on module removal
If musb controller is configured with USB peripherals and we have
enumerated with a USB host, we can get warnings on removal of the
modules:
WARNING: CPU: 0 PID: 1269 at drivers/dma/cppi41.c:391
cppi41_dma_free_chan_resources
Fix the issue by adding the missing pm_runtime_get to
cppi41_dma_free_chan_resources to make sure the pending work
list is cleared on removal.
Fixes: fdea2d09b997 ("dmaengine: cppi41: Add basic PM runtime support")
Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/cppi41.c')
| -rw-r--r-- | drivers/dma/cppi41.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c index bac5f023013b..6ed99d926358 100644 --- a/drivers/dma/cppi41.c +++ b/drivers/dma/cppi41.c @@ -1072,7 +1072,12 @@ err_get_sync: static int cppi41_dma_remove(struct platform_device *pdev) { struct cppi41_dd *cdd = platform_get_drvdata(pdev); + int error; + error = pm_runtime_get_sync(&pdev->dev); + if (error < 0) + dev_err(&pdev->dev, "%s could not pm_runtime_get: %i\n", + __func__, error); of_dma_controller_free(pdev->dev.of_node); dma_async_device_unregister(&cdd->ddev); |