dmaengine: at_xdmac: fix bug in prep_dma_cyclic
authorLudovic Desroches <ludovic.desroches@atmel.com>
Wed, 22 Jul 2015 14:12:29 +0000 (16:12 +0200)
committerVinod Koul <vinod.koul@intel.com>
Wed, 19 Aug 2015 15:51:28 +0000 (21:21 +0530)
In cyclic mode, the round chaining has been broken by the introduction
of at_xdmac_queue_desc(): AT_XDMAC_MBR_UBC_NDE is set for all descriptors
excepted for the last one. at_xdmac_queue_desc() has to be called one
more time to chain the last and the first descriptors.

Signed-off-by: Ludovic Desroches <ludovic.desroches@atmel.com>
Fixes: 0d0ee751f7f7 ("dmaengine: xdmac: Rework the chaining logic")
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/at_xdmac.c

index 1a2d9a39ff257c523561230d73506b3910b8e8d2..99243f6a9c87b4d2c466ef99f9abba28a538837a 100644 (file)
@@ -795,10 +795,7 @@ at_xdmac_prep_dma_cyclic(struct dma_chan *chan, dma_addr_t buf_addr,
                list_add_tail(&desc->desc_node, &first->descs_list);
        }
 
-       prev->lld.mbr_nda = first->tx_dma_desc.phys;
-       dev_dbg(chan2dev(chan),
-               "%s: chain lld: prev=0x%p, mbr_nda=%pad\n",
-               __func__, prev, &prev->lld.mbr_nda);
+       at_xdmac_queue_desc(chan, prev, first);
        first->tx_dma_desc.flags = flags;
        first->xfer_size = buf_len;
        first->direction = direction;