From: Daniel Silsby <dansil...@gmail.com>

Simple cleanup, no changes to actual logic here.

Signed-off-by: Daniel Silsby <dansil...@gmail.com>
Tested-by: Mathieu Malaterre <ma...@debian.org>
---
 drivers/dma/dma-jz4780.c | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

 v2: No change

 v3: No change

diff --git a/drivers/dma/dma-jz4780.c b/drivers/dma/dma-jz4780.c
index fa926de082ba..cc2a86844db4 100644
--- a/drivers/dma/dma-jz4780.c
+++ b/drivers/dma/dma-jz4780.c
@@ -608,22 +608,17 @@ static size_t jz4780_dma_desc_residue(struct 
jz4780_dma_chan *jzchan,
        struct jz4780_dma_desc *desc, unsigned int next_sg)
 {
        struct jz4780_dma_dev *jzdma = jz4780_dma_chan_parent(jzchan);
-       unsigned int residue, count;
+       unsigned int count = 0;
        unsigned int i;
 
-       residue = 0;
-
        for (i = next_sg; i < desc->count; i++)
-               residue += (desc->desc[i].dtc & 0xffffff) <<
-                       jzchan->transfer_shift;
+               count += desc->desc[i].dtc & 0xffffff;
 
-       if (next_sg != 0) {
-               count = jz4780_dma_chn_readl(jzdma, jzchan->id,
+       if (next_sg != 0)
+               count += jz4780_dma_chn_readl(jzdma, jzchan->id,
                                         JZ_DMA_REG_DTC);
-               residue += count << jzchan->transfer_shift;
-       }
 
-       return residue;
+       return count << jzchan->transfer_shift;
 }
 
 static enum dma_status jz4780_dma_tx_status(struct dma_chan *chan,
-- 
2.11.0

Reply via email to