At Thu, 22 Jan 2009 09:29:59 +0100, Krzysztof Helt wrote: > > On Mon, 19 Jan 2009 14:11:47 +0100 > Takashi Iwai <tiwai@xxxxxxx> wrote: > > > At Mon, 19 Jan 2009 14:07:48 +0100, > > Krzysztof Helt wrote: > > > > > > Hi Takashi, > > > > > > I have posted the second version of this driver on 6 December > > > but it has still been waiting for moderator approval as it was > > > to big (about 90KB). > > > > Oh thats' bad. Possible to split and repost? > > > > I can repost it but a split would be very hard. I prepared a patch to > just add a new driver inside alsa-kernel. The old one from the > alsa-driver could be removed then. The diff between the new and > the old driver is over 140KB - only headers are not changed much. OK. > > > Is there any chances to replace an old experimental driver > > > in the alsa-driver package to this new one? > > > A long term target is to remove old OSS driver for the card > > > and use the new one for ALSA. > > > > Sure, I didn't merge your patch just because it seems not much tested > > (as I remembered in your patch description). > > > > I have tested audio (PCM) playback and recording (hence > pcm format bugfixes). I have tested MIDI playback (through > built in wavetable). I have not fixed more exotic features like > digital output/input and external MIDI handling (no device > to test). I think that the basic support (pcm playback/recording > and midi playback) is enough to add the driver. > > > > The summary of major changes are: > > > 1. New driver uses PnP framework > > > 2. It uses MPU401 framework instead of raw midi (so less code) > > > 3. It has fixed handling of 8-bit audio formats (bug in the old driver) > > > 4. It has fixed audio capture formats (bug in the old driver) > > > 5. It has fixed requesting shared memory region (bug in the old driver) > > > 6. ioremap memory region once instead before each access (improvement) > > > 7. Removed common module as it is very small (improvement) > > > 8. Code conforms to the Linux kernel CS. > > > 9. It is shorter by at list 10KB of source and few KB of binary. > > > 10. Few smaller bugs fixed (e.g. digital output option in mixer). > > > > > > I tested this driver on a borrowed card. It has digital output add-on card. > > > If anybody can tell me how to test a digital input/output (e.g. by using > > > another sound card) I can do this as well. > > > > As long as you tested, certainly it's fine to replace. > > Or, we could disable only the old driver's Kconfig and put yours. > > > > Can I repost as a single patch? Yes, please post. thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
