Re: [PATCH 2/2] davinci: da850evm: enable mcasp ping-pong bufs and evq's

[Liam Girdwood <lrg@ti.com>]

On 20/05/11 08:08, Nori, Sekhar wrote:
> Hi Mark,
> 
> On Fri, May 20, 2011 at 02:44:06, Mark Brown wrote:
>> On Wed, May 18, 2011 at 09:27:46AM -0400, Ben Gardiner wrote:
>>> The davinci-mcasp driver will copy the platform data specified sram sizes and
>>> eventq's.
>>>
>>> Set the event queues and sram sizes for da850. 8K SRAM buffers are selected
>>> because it is the minimum that resulted in the same period size when testing
>>> 48KHz S16_LE stereo. Event queues 0 and 1 are assigned to match those 
>>> specified by Troy Kisky in his introduction of ping-pong buffers for dm644x.
>>>
>>> Signed-off-by: Ben Gardiner <bengardiner@nanometrics.ca>
>>
>> Davinci folks, are you OK with this?
> 
> No, please don't merge this just yet. As Ben mentioned in his cover
> letter, the default SRAM pool on DA850 is not suitable for Audio
> transfers. This patch depends on SRAM not being available for audio
> usage in spite of the platform asking for it. This is liable to break.
> 
> The SRAM allocation on DaVinci and OMAP is under a bit of flux and
> once that is sorted out, this patch should be safe to merge. So,
> this probably needs to wait for 2.6.41.
> 

Ok.

Sekhar, Ben, please resend with Acks when the DaVinci depedencies have been merged.

Thanks

Liam