[patch 5/6] edma: Make reading the position of active channels work

Thomas Gleixner tglx at linutronix.de
Fri Apr 18 02:24:10 PDT 2014


On Fri, 18 Apr 2014, Thomas Gleixner wrote:
> On Thu, 17 Apr 2014, Joel Fernandes wrote:
> > Here's a thread that discusses it and the end conclusion is that it
> > should be fine (Kyle Kastile is one of the EDMA hardware designers).
> > http://e2e.ti.com/support/dsp/davinci_digital_media_processors/f/99/t/66011.aspx
> 
> Gah, right I saw the issues with the original code, which does a
> memcpy_fromio which copies byte by byte. Did not think about retrying
> with the edma_read() based one.

Actually I did expect memcpy_fromio to be more clever. But you are
right, using the 32bit wide read works like a charm.

Thanks,

	tglx




More information about the linux-arm-kernel mailing list