[alsa-devel] another device for opl3sa2

[Takashi Iwai]

At Mon, 2 Jul 2007 17:27:39 -0700,
Paul Vojta wrote:
> 
> Folks:
> 
> Here's a patch to add support for an (admittedly old) device:
> 
> 
> --- linux-2.6.21.5/sound/isa/opl3sa2.c.orig	2007-06-11 11:37:06.000000000 -0700
> +++ linux-2.6.21.5/sound/isa/opl3sa2.c	2007-06-30 23:29:43.000000000 -0700
> @@ -165,6 +165,8 @@
>  	{ .id = "YMH0801", .devs = { { "YMH0021" } } },
>  	/* NeoMagic MagicWave 3DX */
>  	{ .id = "NMX2200", .devs = { { "YMH2210" } } },
> +	/* NeoMagic MagicWave 3D */
> +	{ .id = "NMX2200", .devs = { { "NMX2210" } } },
>  	/* --- */
>  	{ .id = "" }	/* end */
>  };
> 

Thanks, could you give your sign-off?


> (There may be a bug in the initialization code, though -- it doesn't work
> the first time after booting, just the second and subsequent times.
> For example, the first time I try running "aplay file.wav" it'll sit there
> silently instead of playing the file, but if I type "aplay file.wav" again
> then it'll play.)

What do you mean "doesn't work"?  The module wasn't loaded or
malfunctional?

> I don't know whether snd_opl3sa2_pnpbiosids[] should also be changed.

The table you changed is for ISA-PnP, while *_pnpbiosids[] is for PnP
BIOS.  There are slight differences.  And, "NMX2210" is already
there.

> Also, why does nm256.c direct users to the cs423x drivers and not opl3sa2?
> If it had mentioned opl3sa2 it would have saved me many hours.

Patch please?  Also, it'd be better to fix ALSA-Configuration.txt,
too...


Takashi