From mboxrd@z Thu Jan  1 00:00:00 1970
From: Takashi Iwai <tiwai@suse.de>
Subject: Re: another device for opl3sa2
Date: Tue, 03 Jul 2007 12:05:50 +0200
Message-ID: <s5hfy45ajk1.wl%tiwai@suse.de>
References: <20070703002739.GA27389@math.berkeley.edu>
Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <alsa-devel-bounces@alsa-project.org>
Received: from mx1.suse.de (ns.suse.de [195.135.220.2])
	by alsa0.perex.cz (Postfix) with ESMTP id E895610388D
	for <alsa-devel@alsa-project.org>;
	Tue,  3 Jul 2007 12:05:52 +0200 (CEST)
In-Reply-To: <20070703002739.GA27389@math.berkeley.edu>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Sender: alsa-devel-bounces@alsa-project.org
Errors-To: alsa-devel-bounces@alsa-project.org
To: Paul Vojta <vojta@Math.Berkeley.EDU>
Cc: alsa-devel@alsa-project.org
List-Id: alsa-devel@alsa-project.org

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