Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Thomas Sailer
Alan Cox wrote: > What format is it that causes the problems, the only badly supported key format > right know I know of is 16bit bigendian. That needs some small esd patches. S8 is a not very well supported format. And btw there are many applications that cannot live with esd for latency

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Alan Cox
> Anything in between is IMO silly. Killing the format > conversion drops the advantage of running many existing > applications but don't bring you much closer to the goal > of simplicity. Those applications already have to deal with the fact some devices only support 48KHz 16bit stereo audio. I

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Jeff Garzik
On Mon, 4 Dec 2000, Thomas Sailer wrote: > And before killing format conversion you should kill > the mmap stunt, because the format conversion complexity > (~25 LOC) is by far dwarfed by the mmap emulation stuff. mmap -emulation- ?? Ug. Is Quake really worth that much? :) > The underlying

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Thomas Sailer
Alan Cox wrote: > Definitely we should If you start killing format conversion then >99% of the existing applications won't work anymore with usbaudio. At that point you can dump the OSS interface just as well. And before killing format conversion you should kill the mmap stunt, because the

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Alan Cox
> > Tjeerd. I deliberately applied only small bits of your patch before because > > the mono mode stuff clutters the driver horribly and is not in the right place. > > It belongs in the application/libraries > > Then you should kill parts of drivers/usb/audio - it contains format conversions.

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Pavel Machek
Hi! > > It implements mono output and fixes a bug in the dma logic (reset necessary > > because some descriptors are already prefetched and are not updated > > This is wrong. Linus please do not apply this patch, or if you have done back > it out. Not only does it do format conversions in

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Pavel Machek
Hi! It implements mono output and fixes a bug in the dma logic (reset necessary because some descriptors are already prefetched and are not updated This is wrong. Linus please do not apply this patch, or if you have done back it out. Not only does it do format conversions in kernel

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Alan Cox
Tjeerd. I deliberately applied only small bits of your patch before because the mono mode stuff clutters the driver horribly and is not in the right place. It belongs in the application/libraries Then you should kill parts of drivers/usb/audio - it contains format conversions.

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Thomas Sailer
Alan Cox wrote: Definitely we should If you start killing format conversion then 99% of the existing applications won't work anymore with usbaudio. At that point you can dump the OSS interface just as well. And before killing format conversion you should kill the mmap stunt, because the

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Jeff Garzik
On Mon, 4 Dec 2000, Thomas Sailer wrote: And before killing format conversion you should kill the mmap stunt, because the format conversion complexity (~25 LOC) is by far dwarfed by the mmap emulation stuff. mmap -emulation- ?? Ug. Is Quake really worth that much? :) The underlying

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Alan Cox
Anything in between is IMO silly. Killing the format conversion drops the advantage of running many existing applications but don't bring you much closer to the goal of simplicity. Those applications already have to deal with the fact some devices only support 48KHz 16bit stereo audio. I run

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-04 Thread Thomas Sailer
Alan Cox wrote: What format is it that causes the problems, the only badly supported key format right know I know of is 16bit bigendian. That needs some small esd patches. S8 is a not very well supported format. And btw there are many applications that cannot live with esd for latency

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-01 Thread Alan Cox
> It implements mono output and fixes a bug in the dma logic (reset necessary > because some descriptors are already prefetched and are not updated This is wrong. Linus please do not apply this patch, or if you have done back it out. Not only does it do format conversions in kernel (which is a

Re: [PATCH] i810_audio 2.4.0-test11

2000-12-01 Thread Alan Cox
It implements mono output and fixes a bug in the dma logic (reset necessary because some descriptors are already prefetched and are not updated This is wrong. Linus please do not apply this patch, or if you have done back it out. Not only does it do format conversions in kernel (which is a