Hi, Thanks for the review... <Snip> > >On Thu, Dec 07, 2017 at 10:51:02AM +0530, Kedareswara rao Appana wrote: > >> @@ -2029,6 +2006,7 @@ static int xilinx_dma_terminate_all(struct >> dma_chan *dchan) >> >> /* Remove and free all of the descriptors in the lists */ >> xilinx_dma_free_descriptors(chan); >> + chan->idle = true; >> >> if (chan->cyclic) { >> reg = dma_ctrl_read(chan, XILINX_DMA_REG_DMACR); @@ - >2344,6 >> +2322,12 @@ static int xilinx_dma_chan_probe(struct xilinx_dma_device >*xdev, >> chan->has_sg = xdev->has_sg; >> chan->desc_pendingcount = 0x0; >> chan->ext_addr = xdev->ext_addr; >> + /* This variable enusres that descripotrs are not > ^^^^^^^^^^ > >typo Since this patch got applied Will take care of this next time on wards... Thanks for the review... Regards, Kedar. > >-- >~Vinod -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html