On Fri, 11 Oct 2002, Abramo Bagnara wrote:

> Jaroslav Kysela wrote:
> > 
> > Hi all,
> > 
> >         these enhancements are in CVS for PCM API:
> > 
> > - added snd_pcm_avail() function - this function returns count of
> >   available frames for write or read operations, the position
> >   is read directly from hardware; the snd_pcm_avail_update() call
> >   is still mandatory before any I/O!!!
> 
> I really don't understand why you've added another ioctl and another
> function to replicate in all PCM classes.
> snd_pcm_delay existence is enough to implement that.
> I'm missing something?

You're completely right, but in my eyes, the avail ioctl looks much better
for kernel space (it's a base value for computing delay, computing avail
from delay needs twice substractions which doesn't look very good).
Unfortunately, delay ioctl was first, and we should keep binary
compatibility for 2.5, so we should leave both ioctls there. In library,
we can remove the delay operation from chains and use avail to compute it 
as well to simplify code.

                                                Jaroslav

-----
Jaroslav Kysela <[EMAIL PROTECTED]>
Linux Kernel Sound Maintainer
ALSA Project  http://www.alsa-project.org
SuSE Linux    http://www.suse.com



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Alsa-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Reply via email to