Messages in this thread | | | From | Appana Durga Kedareswara Rao <> | Subject | RE: [PATCH v5 1/3] dmaengine: xilinx_dma: Check for channel idle state before submitting dma descriptor | Date | Fri, 13 Jan 2017 05:32:39 +0000 |
| |
Hi Vinod,
Thanks for the review... > > On Fri, Jan 13, 2017 at 04:28:11AM +0000, Appana Durga Kedareswara Rao > wrote: > > Hi Vinod, > > > > Thanks for the review... > > > > > > On Sat, Jan 07, 2017 at 12:15:28PM +0530, Kedareswara rao Appana wrote: > > > > Add channel idle state to ensure that dma descriptor is not > > > > submitted when VDMA engine is in progress. > > > > > > any reason why you want to make your own varible and not use the HW > > > to query as done earlier. It is not clear to me why that is removed > > > from description > > > > We need to poll for a bit in the status register to know the dma state. > > We are currently doing that in the driver hot path To avoid this using > > own variables. > > It would be worthwhile to document these, down the line people may not > remeber the motivation
Sure will add comments during the variable initialization In the driver...
Regards, Kedar.
> > > -- > ~Vinod
| |