From: Vinod Koul <vinod.koul@intel.com>
Date: Fri, 8 Jul 2016 05:00:57 +0000 (+0530)
Subject: dmaengine: cppi: remove unused and bogus check
X-Git-Tag: v4.8-rc1~117^2^2~4
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=a03811045ee3c2be1d333fb2136d06f244302d46;p=users%2Fjedix%2Flinux-maple.git

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>
---

diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c
index ceedafbd23e01..4b2317426c8e3 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);