summaryrefslogtreecommitdiffstats
path: root/drivers/dma
diff options
context:
space:
mode:
authorVinod Koul <vinod.koul@intel.com>2016-07-08 10:30:57 +0530
committerVinod Koul <vinod.koul@intel.com>2016-07-23 16:07:27 +0530
commita03811045ee3c2be1d333fb2136d06f244302d46 (patch)
tree1cd876b69ac290bb17fb1ac378fe0ff6af7f709a /drivers/dma
parent4cad91b2a9977c8d92f342c4a9b73cd9dbce8f21 (diff)
downloadlinux-a03811045ee3c2be1d333fb2136d06f244302d46.tar.gz
linux-a03811045ee3c2be1d333fb2136d06f244302d46.tar.bz2
linux-a03811045ee3c2be1d333fb2136d06f244302d46.zip
dmaengine: cppi: remove unused and bogus check
In cppi41_dma_prep_slave_sg() variable num is initialized to zero, but never updated and a BUG_ON is checked for it being greater than zero which will be always false. Remove the bogus check and this variable Reported-by: David Binderman <linuxdev.baldrick@gmail.com> Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma')
-rw-r--r--drivers/dma/cppi41.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c
index ceedafbd23e0..4b2317426c8e 100644
--- a/drivers/dma/cppi41.c
+++ b/drivers/dma/cppi41.c
@@ -497,16 +497,13 @@ static struct dma_async_tx_descriptor *cppi41_dma_prep_slave_sg(
struct cppi41_desc *d;
struct scatterlist *sg;
unsigned int i;
- unsigned int num;
- num = 0;
d = c->desc;
for_each_sg(sgl, sg, sg_len, i) {
u32 addr;
u32 len;
/* We need to use more than one desc once musb supports sg */
- BUG_ON(num > 0);
addr = lower_32_bits(sg_dma_address(sg));
len = sg_dma_len(sg);