diff options
| author | Cong Ding <dinggnu@gmail.com> | 2013-01-15 04:19:48 +0400 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2013-01-20 17:49:40 +0400 |
| commit | ed30933e6f3dbeaaab1de91e1bec25f42d5d32df (patch) | |
| tree | f4953c62c1a7a5dfa0cf21ecce89d35d0a55906c | |
| parent | 661f7cb55c61fa7491e0caf21e55f59e5bc49abe (diff) | |
| download | linux-ed30933e6f3dbeaaab1de91e1bec25f42d5d32df.tar.xz | |
dma: remove unnecessary null pointer check in mmp_pdma.c
the pointer cfg is dereferenced in line 594, so it's no reason to check null
again in line 620.
Signed-off-by: Cong Ding <dinggnu@gmail.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
| -rw-r--r-- | drivers/dma/mmp_pdma.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma/mmp_pdma.c index 13bdf4a7e1ec..41ad6a62f838 100644 --- a/drivers/dma/mmp_pdma.c +++ b/drivers/dma/mmp_pdma.c @@ -617,10 +617,8 @@ static int mmp_pdma_control(struct dma_chan *dchan, enum dma_ctrl_cmd cmd, else if (maxburst == 32) chan->dcmd |= DCMD_BURST32; - if (cfg) { - chan->dir = cfg->direction; - chan->drcmr = cfg->slave_id; - } + chan->dir = cfg->direction; + chan->drcmr = cfg->slave_id; chan->dev_addr = addr; break; default: |