On 03.10.2012 09:16, Hebbar, Gururaja wrote:
On Wed, Oct 03, 2012 at 03:58:45, Daniel Mack wrote:
On 31.08.2012 14:50, Hebbar, Gururaja wrote:
The OMAP2+ variant of McASP is different from Davinci variant w.r.to some register offset and missing generic SRAM APIs support
Changes
- Add new MCASP_VERSION_3 to identify new variant. New DT compatible "ti,omap2-mcasp-audio" to identify version 3 controller.
- The register offsets are handled depending on the version.
- Provide a config option to indicate missing SRAM API support.
Could you give some insight which hardware this was tested on? I'm trying to get this up and running on a AM33xx board, and the patches look all reasonable to me. My problem is that I can't make the DMA engine move forward, I fail to receive a single interrupt on this peripheral after the stream starts. I will continue searching for the reason of this tomorrow, but maybe you can give me some hint by explaining your setup?
Note that I'm using your patches together with Matt's from this series:
https://github.com/ohporter/linux/tree/edma-dmaengine-am33xx-rfc-v1
... but it doesn't work without those either.
When I started working on adding DT support to McASP driver, Matt Porter EDMA port was not yet ready.
So
I took existing edma driver from AM335x Arago release [1] (driver + edma device init).
Added this to Vaibhav's Local (linux-next + AM335x patches) tree [2]
Added DT support to McASP driver.
I tested this on AM335x EVM board.
Hmm, so I think there's a tiny bit missing for the EDMA conversion on DT kernels.
If you need the code, I can share it as a patch (I will send the patch as a private mail since the patch is huge).
Please do :) Or just push your entire tree somewhere.
Thanks, Daniel