On Thu, 12 Aug 2010, Raymond Yau wrote:
It seem that there is a bug in the patch which remove "if ( err < 0 )"
if (err < 0) return err;
if (pcm->minperiodtime > 0) {
Thanks. It's usuall typo. Fixed now.
http://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=09879a4bb58199f64...
BTW , it seem this patch change the behaviour of alsa-lib too.
all sound cards will now have a minperiodtime 5000
Nope. If application settles the period time/size, the behaviour is unchanged (see the min/max condition). We just want to limit the period sizes for applications which do not know what they are doing and allow any period time/sizes. It's better to limit the minimal period size to avoid generation of many interrupts per second in this case.
Jaroslav
----- Jaroslav Kysela perex@perex.cz Linux Kernel Sound Maintainer ALSA Project, Red Hat, Inc.