Hi,

Your patch to clean up the OSS stuff has broken the Ensoniq SoundScape.
Reverting one small portion fixes it.
Can you decide whether you want to wrap this in some magical #ifdef's
and send it on to Linus?


BTW, the documentation in Documentation/README.OSS suggests that this
card cannot be used with isapnp?
This is not my experience.
See http://www.huggie.dial.pipex.com/linux/sound.shtml for how it Works
For Me(TM).


Also since I applied my patch, I get this when I modprobe -r:

[root@byers /usr/src/linux/drivers/sound]# modprobe -r sscape
Trying to free nonexistent resource <0000-0001>
Sound device 2 went missing!
kmem_free: Either bad obj addr or double free (objp=c61bd000,
name=size-4096)

I imagine this is because of the changes too?


Simon.

-- 
"Now where did I put that magazine?" Tom asked periodically.                
--- mpu401.c.old        Sun Mar 12 14:17:11 2000
+++ mpu401.c    Sun Mar 12 14:17:51 2000
@@ -1726,18 +1726,14 @@
 {
        /* Can be loaded either for module use or to provide functions
           to others */
-       cfg.irq = irq;
-       cfg.io_base = io;
-       
-       if (cfg.io_base != -1 && cfg.irq != -1) {
-               printk(KERN_WARNING "mpu401: need io and irq !");
-               return -ENODEV;
+       if (io != -1 && irq != -1) {
+               cfg.irq = irq;
+               cfg.io_base = io;
+               if (probe_mpu401(&cfg) == 0)
+                       return -ENODEV;
+               attach_mpu401(&cfg);
        }
        
-       if (probe_mpu401(&cfg) == 0)
-               return -ENODEV;
-       attach_mpu401(&cfg);
-
        SOUND_LOCK;
        return 0;
 }

Reply via email to