diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-09-03 01:01:06 +0200 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2016-09-05 13:10:52 +0200 |
| commit | 028e84a1de7cba6a3e4cf1a22094b76a4b92cefb (patch) | |
| tree | 20098fdc562cec68d1af118e84e76ff7d0837c6d /drivers/dma | |
| parent | dmaengine: moxart: remove NO_IRQ (diff) | |
| download | linux-028e84a1de7cba6a3e4cf1a22094b76a4b92cefb.tar.xz linux-028e84a1de7cba6a3e4cf1a22094b76a4b92cefb.zip | |
dmaengine: mxs: remove NO_IRQ check
The mxs_chan->chan_irq variable is guaranteed to never be NO_IRQ,
as it gets assigned the result of platform_get_irq() that returns
either a valid positive interrupt number, or a negative failure
code that leads to the channel not being used.
This removes the redundant check, eliminating one more instance
of NO_IRQ.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma')
| -rw-r--r-- | drivers/dma/mxs-dma.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/dma/mxs-dma.c b/drivers/dma/mxs-dma.c index 60de35251da5..1e3572bb7012 100644 --- a/drivers/dma/mxs-dma.c +++ b/drivers/dma/mxs-dma.c @@ -429,12 +429,10 @@ static int mxs_dma_alloc_chan_resources(struct dma_chan *chan) goto err_alloc; } - if (mxs_chan->chan_irq != NO_IRQ) { - ret = request_irq(mxs_chan->chan_irq, mxs_dma_int_handler, - 0, "mxs-dma", mxs_dma); - if (ret) - goto err_irq; - } + ret = request_irq(mxs_chan->chan_irq, mxs_dma_int_handler, + 0, "mxs-dma", mxs_dma); + if (ret) + goto err_irq; ret = clk_prepare_enable(mxs_dma->clk); if (ret) |