On Tue, 20 Aug 2002, Takashi Iwai wrote: > Hi, > > At Mon, 19 Aug 2002 22:29:12 +0200, > Thierry Vignaud wrote: > > > > snd-ens1370 and 1371're just the same driver which compilation is > > altered by preprocessors directives. > > > > snd-via8232 came from via686 and some common code can still be seen. > > > > it would be nice if the alsa team can make a unified snd-ens137[01] > > driver and a snd-via-common sub module used by snd-via{686,8232}. > > both jobs would be relatively easy. > in fact, integration of ens137x was already done experimentally on my > tree, and the change for via driver should be also small. > > but the decision is different. > > pro: > - merging via drivers will make the maintenance and bug-hunt easier. > merging ens137x drivers makes no difference, because it's already > in a source. > > con: > - merging two drivers to a unified driver will increase the size. > - we may put the common stuff into lowlevel module. but the > lowlevel module cannot be marked as __devinit, although this > doesn't matter for modules anyway. > > > basically i vote for this action, but i'd like to hear the opinion > from other people in this regard.
I don't like very much to merging of ENS1370 and ENS1371, because different codecs are used (snd-ak4531-codec and snd-ac97-codec). Sure, there is common code (MIDI UART, most of PCM routines), but I think that the final difference (separate module with common code) will be negligible, but we may do it. VIA code may be merged. 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: OSDN - Tired of that same old cell phone? Get a new here for FREE! https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390 _______________________________________________ Alsa-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/alsa-devel