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=09879a4bb58199f64abcb8df506f917c8efc2383;hp=e6d5dcf1f625984605d362338d71162de45a6c60
>
> 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@xxxxxxxx>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
