diff options
| author | Vinod Koul <vinod.koul@intel.com> | 2016-07-01 05:24:56 (GMT) |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2016-07-16 14:48:57 (GMT) |
| commit | 7bb45f669610e0fae4b3dfe66056cf85a57014c6 (patch) | |
| tree | 98f77b0e8880cb818595ca3428761b7c846f6b09 /drivers/dma/coh901318.c | |
| parent | a0d4cb44da6ca0e87f068dd6644370e6eaca422c (diff) | |
| download | linux-7bb45f669610e0fae4b3dfe66056cf85a57014c6.tar.xz | |
dmaengine: coh901318: explicitly freeup irq
dmaengine device should explicitly call devm_free_irq() when using
devm_request_irq().
The irq is still ON when devices remove is executed and irq should be
quiesced before remove is completed.
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/dma/coh901318.c')
| -rw-r--r-- | drivers/dma/coh901318.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c index c100616..ba044d4 100644 --- a/drivers/dma/coh901318.c +++ b/drivers/dma/coh901318.c @@ -1280,6 +1280,7 @@ struct coh901318_desc { struct coh901318_base { struct device *dev; void __iomem *virtbase; + unsigned int irq; struct coh901318_pool pool; struct powersave pm; struct dma_device dma_slave; @@ -2680,6 +2681,8 @@ static int __init coh901318_probe(struct platform_device *pdev) if (err) return err; + base->irq = irq; + err = coh901318_pool_create(&base->pool, &pdev->dev, sizeof(struct coh901318_lli), 32); @@ -2760,6 +2763,8 @@ static int coh901318_remove(struct platform_device *pdev) { struct coh901318_base *base = platform_get_drvdata(pdev); + devm_free_irq(&pdev->dev, base->irq, base); + of_dma_controller_free(pdev->dev.of_node); dma_async_device_unregister(&base->dma_memcpy); dma_async_device_unregister(&base->dma_slave); |