Browse Source

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>
Sebastian Andrzej Siewior 12 years ago
parent
commit
ff2283229d
1 changed files with 0 additions and 3 deletions
  1. 0 3
      drivers/usb/musb/musbhsdma.c

+ 0 - 3
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);