diff options
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>2013-06-19 15:38:10 (GMT)
committerFelipe Balbi <balbi@ti.com>2013-07-29 10:53:17 (GMT)
commitff2283229da616d9a029eaa0d483fa8b0ad55e77 (patch)
parent260eba39bcfb5681115ec934cdfbd97e0b3a1775 (diff)
usb: musb: musbhsdma: drop the controller check in dma_controller_destroy()
This check is hardly required and alas is wrong. 'c' might be NULL but the chances are low that 'controller' after the container_of() becomes NULL. Since no other DMA implementation is doing that and musb-core does not call it with a NULL pointer it can dropped. Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Signed-off-by: Felipe Balbi <balbi@ti.com>
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/usb/musb/musbhsdma.c b/drivers/usb/musb/musbhsdma.c
index 3d1fd52..3f5e121 100644
--- a/drivers/usb/musb/musbhsdma.c
+++ b/drivers/usb/musb/musbhsdma.c
@@ -371,9 +371,6 @@ void dma_controller_destroy(struct dma_controller *c)
struct musb_dma_controller *controller = container_of(c,
struct musb_dma_controller, controller);
- if (!controller)
- return;
if (controller->irq)
free_irq(controller->irq, c);

Privacy Policy