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

Reply via email to