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

Joel Fernandes joelf at ti.com
Fri Apr 18 09:40:12 PDT 2014


On 04/18/2014 04:24 AM, Thomas Gleixner wrote:
> 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.

Cool, glad it does :)

regards,
  -Joel




More information about the linux-arm-kernel mailing list