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