dma/intel_mid_dma: remove unneeded null check
authorDan Carpenter <error27@gmail.com>
Thu, 28 Oct 2010 13:41:56 +0000 (15:41 +0200)
committerDan Williams <dan.j.williams@intel.com>
Fri, 3 Dec 2010 09:05:24 +0000 (01:05 -0800)
Smatch complains because we dereference "mid" before checking it.  It
turns out that "mid" is always a valid pointer here so we can just
remove the check.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/dma/intel_mid_dma.c

index 338bc4eed1f3f55b4de66c623a135289862c783f..41941d05bfb9fc129070ddbcbab8e345e3fe16ae 100644 (file)
@@ -1021,11 +1021,6 @@ static irqreturn_t intel_mid_dma_interrupt(int irq, void *data)
 
        /*DMA Interrupt*/
        pr_debug("MDMA:Got an interrupt on irq %d\n", irq);
-       if (!mid) {
-               pr_err("ERR_MDMA:null pointer mid\n");
-               return -EINVAL;
-       }
-
        pr_debug("MDMA: Status %x, Mask %x\n", tfr_status, mid->intr_mask);
        tfr_status &= mid->intr_mask;
        if (tfr_status) {