PATCH: Looking for nm256 audio driver users to test this...

Anyone have any sound hardware (usually found on laptops) which uses the nm256 audio driver? The attached patch would allow us to hook this nm256 driver into a more modern AC97 kernel interface, but unfortunately I have no hardware for testing this change.

Any testing and "it works!" or "it doesn't work!" feedback would be greatly appreciated.

Simple cookbook testing procedure:

1) download and unpack latest development kernel tarball from ftp://ftp.us.kernel.org/pub/linux/kernel/v2.4/

2) download latest development kernel Alan Cox "ac" patch kit from ftp://ftp.us.kernel.org/pub/linux/kernel/people/alan/2.4.0test/ and apply this patch to the kernel unpacked in step #1.

3) build and test a "plain" kernel. note how well your audio works. nothing fancy, just make sure MP3s play and stuff. :)

4) apply the patch attached to this e-mail.

5) build and test a modified kernel. note differences (hopefully none...) in functioning of audio. This patch changes the mixer code, so testing the mixer is a good idea.

For more info see the Linux Kernel HOWTO:
http://www.linuxdoc.org/HOWTO/Kernel-HOWTO.html

Thanks,

Jeff

Jeff Garzik              | Liberty is always dangerous, but
Building 1024            | it is the safest thing we have.
MandrakeSoft, Inc.       |      -- Harry Emerson Fosdick

diff -urN vanilla/linux-2.4.0-test1-ac7/drivers/sound/Makefile linux/drivers/sound/Makefile
@@ -71,7 +71,7 @@
 obj-$(CONFIG_SOUND_MSNDCLAS)	+= msnd.o msnd_classic.o
 obj-$(CONFIG_SOUND_MSNDPIN)	+= msnd.o msnd_pinnacle.o
 obj-$(CONFIG_SOUND_VWSND)	+= vwsnd.o
-obj-$(CONFIG_SOUND_NM256)	+= nm256_audio.o ac97.o
+obj-$(CONFIG_SOUND_NM256)	+= nm256_audio.o ac97_codec.o
 obj-$(CONFIG_SOUND_ICH)		+= i810_audio.o ac97_codec.o
 obj-$(CONFIG_SOUND_SONICVIBES)	+= sonicvibes.o
 obj-$(CONFIG_SOUND_CMPCI)	+= cmpci.o

diff -urN vanilla/linux-2.4.0-test1-ac7/drivers/sound/nm256.h linux/drivers/sound/nm256.h
@@ -1,7 +1,6 @@
 #ifndef _NM256_H_
 #define _NM256_H_
 
-#include "ac97.h"
 
 /* The revisions that we currently handle.  */
 enum nm256rev {

@@ -18,7 +17,7 @@
     /* Revision number */
     enum nm256rev rev;
 
-    struct ac97_hwint mdev;
+    struct ac97_codec ac97;
 
     /* Our audio device numbers. */
     int dev[2];

@@ -32,9 +31,6 @@
        these are the actual device numbers. */
     int dev_for_play;
     int dev_for_record;
-
-    /* The mixer device. */
-    int mixer_oss_dev;
 
     /* 
      * Can only be opened once for each operation.  These aren't set