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 <bengardi...@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. Thanks, Sekhar _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
