usb: musb: remove a few is_dma_capable() in init/exit code
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Wed, 19 Jun 2013 15:38:13 +0000 (17:38 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 29 Jul 2013 10:53:20 +0000 (13:53 +0300)
This patch removes is_dma_capable() and an ifdef in the init/exit path
around init/de-init of the dma_controller. Since we have the empty stubs
in the PIO code we can call it without gcc trouble. Earlier we had an
ifdef and the is_dma_capable() macro where gcc ignored the if (0) path
even that the function was not around :)

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/musb/musb_core.c

index a4434d2..b33bed5 100644 (file)
@@ -1764,7 +1764,7 @@ static void musb_free(struct musb *musb)
                        disable_irq_wake(musb->nIrq);
                free_irq(musb->nIrq, musb);
        }
-       if (is_dma_capable() && musb->dma_controller)
+       if (musb->dma_controller)
                dma_controller_destroy(musb->dma_controller);
 
        musb_host_free(musb);
@@ -1840,12 +1840,11 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
 
        pm_runtime_get_sync(musb->controller);
 
-#ifndef CONFIG_MUSB_PIO_ONLY
        if (use_dma && dev->dma_mask)
                musb->dma_controller = dma_controller_create(musb, musb->mregs);
-#endif
+
        /* ideally this would be abstracted in platform setup */
-       if (!is_dma_capable() || !musb->dma_controller)
+       if (!musb->dma_controller)
                dev->dma_mask = NULL;
 
        /* be sure interrupts are disabled before connecting ISR */