Re: mx31 snd and mc13783 codec status

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

On Fri, Apr 02, 2010 at 11:53:58AM +0200, Sascha Hauer wrote:
> On Thu, Apr 01, 2010 at 06:01:38PM +0200, Valentin Longchamp wrote:

> > The thing is that I get a less than a second sound loop (I use aplay to  
> > test, so userspace app should be ok), as if the buffer that the fiq asm  
> > interrupt (from ssi_fiq.S) copies to the SSI hardware never was updated.

> > If I have understood the fiq behaviour correctly, you have a asm fiq  
> > interrupt that does copy a larger tx buffer into the SSI hardware.  
> > Besides it, you have the imx_ssi_timer_callback that checks when the tx  
> > buffer was completely copied. If it is the case, then a new buffer tx  
> > buffer is "issued" with the snd_pcm_period_elapsed call (and then  
> > snd_pcm_update_hw_ptr0). Is this behaviour correct ?

> Yes.

What sample rate are you trying to play and what buffer size?  In my
testing the FIQ was really struggling with most applications at sample
rates over ~16kHz since you need each audio period to be long enough to
at least fill the interval between timer polls but applications wanted
to select buffer sizes that were consumed faster than the timer tick.

Using an application like speaker-test which has minimal overhead and no
I/O to worry about helps too.

> > If then it looks like on my system, I have a problem with the  
> > snd_pcm_update_hw_ptr0 call.

> You could try reverting b4e82b5b785670b68136765059d1afc65c0ae023. Though
> I have tested it on my boards it may have some implications.

I was seeing similar behaviour before and after that patch too, the main
effect there was to improve the error handling when the FIQ and the
timer get out of sync with each other.