diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2013-06-19 17:38:10 +0200 |
|---|---|---|
| committer | Felipe Balbi <balbi@ti.com> | 2013-07-29 13:53:17 +0300 |
| commit | ff2283229da616d9a029eaa0d483fa8b0ad55e77 (patch) | |
| tree | 8f9351c4f7dae38ddf1298dd8d67f00cc36cbd34 | |
| parent | 260eba39bcfb5681115ec934cdfbd97e0b3a1775 (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>
| -rw-r--r-- | drivers/usb/musb/musbhsdma.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/usb/musb/musbhsdma.c b/drivers/usb/musb/musbhsdma.c index 3d1fd52a15a9..3f5e121dc154 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); |