[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