Blackfin arch: set_dma_callback: do not store .irq
authorMike Frysinger <vapier.adi@gmail.com>
Wed, 7 Jan 2009 15:14:39 +0000 (23:14 +0800)
committerBryan Wu <cooloney@kernel.org>
Wed, 7 Jan 2009 15:14:39 +0000 (23:14 +0800)
set_dma_callback: do not store .irq if request_irq() failed so we dont
turn around and attempt to free_irq() it later on in free_dma()

Signed-off-by: Mike Frysinger <vapier.adi@gmail.com>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
arch/blackfin/kernel/bfin_dma_5xx.c

index bdebab41419fa2c3789b3cb77a60c600b6fb9d9d..ed7d2859a626d07ae47c7008e717eb1857f6a671 100644 (file)
@@ -157,18 +157,16 @@ int set_dma_callback(unsigned int channel, irq_handler_t callback, void *data)
               && channel < MAX_DMA_CHANNELS));
 
        if (callback != NULL) {
-               int ret_val;
-               dma_ch[channel].irq = channel2irq(channel);
-               dma_ch[channel].data = data;
+               int ret;
+               unsigned int irq = channel2irq(channel);
 
-               ret_val =
-                   request_irq(dma_ch[channel].irq, callback, IRQF_DISABLED,
-                               dma_ch[channel].device_id, data);
-               if (ret_val) {
-                       printk(KERN_NOTICE
-                              "Request irq in DMA engine failed.\n");
-                       return -EPERM;
-               }
+               ret = request_irq(irq, callback, IRQF_DISABLED,
+                       dma_ch[channel].device_id, data);
+               if (ret)
+                       return ret;
+
+               dma_ch[channel].irq = irq;
+               dma_ch[channel].data = data;
        }
        return 0;
 }