[ARM] MX1/MX2 DMA: add missing local_irq_restore()
This patch adds a missing call to local_irq_restore() and fixes some compiler warnings about unused variables for MX1. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
This commit is contained in:
parent
b71edbe9a0
commit
7c99502383
1 changed files with 5 additions and 2 deletions
|
@ -511,6 +511,7 @@ void imx_dma_disable(int channel)
|
|||
}
|
||||
EXPORT_SYMBOL(imx_dma_disable);
|
||||
|
||||
#ifdef CONFIG_ARCH_MX2
|
||||
static void imx_dma_watchdog(unsigned long chno)
|
||||
{
|
||||
struct imx_dma_channel *imxdma = &imx_dma_channels[chno];
|
||||
|
@ -522,6 +523,7 @@ static void imx_dma_watchdog(unsigned long chno)
|
|||
if (imxdma->err_handler)
|
||||
imxdma->err_handler(chno, imxdma->data, IMX_DMA_ERR_TIMEOUT);
|
||||
}
|
||||
#endif
|
||||
|
||||
static irqreturn_t dma_err_handler(int irq, void *dev_id)
|
||||
{
|
||||
|
@ -674,7 +676,7 @@ int imx_dma_request(int channel, const char *name)
|
|||
{
|
||||
struct imx_dma_channel *imxdma = &imx_dma_channels[channel];
|
||||
unsigned long flags;
|
||||
int ret;
|
||||
int ret = 0;
|
||||
|
||||
/* basic sanity checks */
|
||||
if (!name)
|
||||
|
@ -696,6 +698,7 @@ int imx_dma_request(int channel, const char *name)
|
|||
ret = request_irq(MXC_INT_DMACH0 + channel, dma_irq_handler, 0, "DMA",
|
||||
NULL);
|
||||
if (ret) {
|
||||
local_irq_restore(flags);
|
||||
printk(KERN_CRIT "Can't register IRQ %d for DMA channel %d\n",
|
||||
MXC_INT_DMACH0 + channel, channel);
|
||||
return ret;
|
||||
|
@ -712,7 +715,7 @@ int imx_dma_request(int channel, const char *name)
|
|||
imxdma->sg = NULL;
|
||||
|
||||
local_irq_restore(flags);
|
||||
return 0;
|
||||
return ret;
|
||||
}
|
||||
EXPORT_SYMBOL(imx_dma_request);
|
||||
|
||||
|
|
Loading…
Reference in a new issue