On Tue, Mar 18, 2014 at 04:28:55PM +0200, Peter Ujfalusi wrote:
On 03/18/2014 03:23 PM, Peter Ujfalusi wrote:
Oh, I edited the commit message and removed the text related to buffer size but I failed to do that for the commit title :(
Actually I still have issue with the constraint.
OK, so is the patch an improvement or not? If it fixes some cases it's probably worth applying even if further fixes are still needed. No need to resend for the subject, I can fix that up.
It seams that I need to place the same constraint step to both period size and buffer size. For some reason snd_pcm_hw_params_set_buffer_time_near() fails in case of 44.1KHz when asking for 0.5s buffer when I placed step 32 constraint to period size. Placing the same 32 steps constraint to the buffer size as well will make things working (for mplayer at least)
That seems like the constraint code is failing, at a guess probably looking for too near a value of near if you see what I mean. I don't really have the time/enthusiasm to investigate this right now, sorry.