diff options
author | Cong Ding <dinggnu@gmail.com> | 2013-01-14 19:23:48 -0500 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2013-01-20 08:49:40 -0500 |
commit | 855372c013bbad8369223f7c75242bd3c94f9345 (patch) | |
tree | 93ae6496fe982c599e0b67a1506fe71227c54626 /drivers/dma/sh | |
parent | ed30933e6f3dbeaaab1de91e1bec25f42d5d32df (diff) |
dma: sh/shdma-base.c: remove unnecessary null pointer check
the variable chan is dereferenced in line 635, so it is no reason to check
null again in line 641.
Signed-off-by: Cong Ding <dinggnu@gmail.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/sh')
-rw-r--r-- | drivers/dma/sh/shdma-base.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/dma/sh/shdma-base.c b/drivers/dma/sh/shdma-base.c index f4cd946d259d..4acb85a10250 100644 --- a/drivers/dma/sh/shdma-base.c +++ b/drivers/dma/sh/shdma-base.c | |||
@@ -638,9 +638,6 @@ static int shdma_control(struct dma_chan *chan, enum dma_ctrl_cmd cmd, | |||
638 | unsigned long flags; | 638 | unsigned long flags; |
639 | int ret; | 639 | int ret; |
640 | 640 | ||
641 | if (!chan) | ||
642 | return -EINVAL; | ||
643 | |||
644 | switch (cmd) { | 641 | switch (cmd) { |
645 | case DMA_TERMINATE_ALL: | 642 | case DMA_TERMINATE_ALL: |
646 | spin_lock_irqsave(&schan->chan_lock, flags); | 643 | spin_lock_irqsave(&schan->chan_lock, flags); |