include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit...
[safe/jmp/linux-2.6] / sound / isa / opti9xx / miro.c
index 1776103..8c24102 100644 (file)
@@ -25,8 +25,8 @@
 #include <linux/init.h>
 #include <linux/err.h>
 #include <linux/isa.h>
+#include <linux/pnp.h>
 #include <linux/delay.h>
-#include <linux/slab.h>
 #include <linux/ioport.h>
 #include <linux/moduleparam.h>
 #include <asm/io.h>
@@ -40,7 +40,7 @@
 #define SNDRV_LEGACY_FIND_FREE_IRQ
 #define SNDRV_LEGACY_FIND_FREE_DMA
 #include <sound/initval.h>
-#include "miro.h"
+#include <sound/aci.h>
 
 MODULE_AUTHOR("Martin Langer <martin-langer@gmx.de>");
 MODULE_LICENSE("GPL");
@@ -60,6 +60,9 @@ static int dma1 = SNDRV_DEFAULT_DMA1;         /* 0,1,3 */
 static int dma2 = SNDRV_DEFAULT_DMA1;          /* 0,1,3 */
 static int wss;
 static int ide;
+#ifdef CONFIG_PNP
+static int isapnp = 1;                         /* Enable ISA PnP detection */
+#endif
 
 module_param(index, int, 0444);
 MODULE_PARM_DESC(index, "Index value for miro soundcard.");
@@ -83,6 +86,10 @@ module_param(wss, int, 0444);
 MODULE_PARM_DESC(wss, "wss mode");
 module_param(ide, int, 0444);
 MODULE_PARM_DESC(ide, "enable ide port");
+#ifdef CONFIG_PNP
+module_param(isapnp, bool, 0444);
+MODULE_PARM_DESC(isapnp, "Enable ISA PnP detection for specified soundcard.");
+#endif
 
 #define OPTi9XX_HW_DETECT      0
 #define OPTi9XX_HW_82C928      1
@@ -96,7 +103,6 @@ MODULE_PARM_DESC(ide, "enable ide port");
 
 #define OPTi9XX_MC_REG(n)      n
 
-
 struct snd_miro {
        unsigned short hardware;
        unsigned char password;
@@ -110,7 +116,6 @@ struct snd_miro {
        unsigned long pwd_reg;
 
        spinlock_t lock;
-       struct snd_card *card;
        struct snd_pcm *pcm;
 
        long wss_base;
@@ -121,37 +126,45 @@ struct snd_miro {
        long mpu_port;
        int mpu_irq;
 
-       unsigned long aci_port;
-       int aci_vendor;
-       int aci_product;
-       int aci_version;
-       int aci_amp;
-       int aci_preamp;
-       int aci_solomode;
-
-       struct mutex aci_mutex;
+       struct snd_miro_aci *aci;
 };
 
-static void snd_miro_proc_init(struct snd_miro * miro);
+static struct snd_miro_aci aci_device;
 
 static char * snd_opti9xx_names[] = {
-       "unkown",
+       "unknown",
        "82C928", "82C929",
        "82C924", "82C925",
        "82C930", "82C931", "82C933"
 };
 
+static int snd_miro_pnp_is_probed;
+
+#ifdef CONFIG_PNP
+
+static struct pnp_card_device_id snd_miro_pnpids[] = {
+       /* PCM20 and PCM12 in PnP mode */
+       { .id = "MIR0924",
+         .devs = { { "MIR0000" }, { "MIR0002" }, { "MIR0005" } }, },
+       { .id = "" }
+};
+
+MODULE_DEVICE_TABLE(pnp_card, snd_miro_pnpids);
+
+#endif /* CONFIG_PNP */
+
 /* 
  *  ACI control
  */
 
-static int aci_busy_wait(struct snd_miro * miro)
+static int aci_busy_wait(struct snd_miro_aci *aci)
 {
        long timeout;
        unsigned char byte;
 
-       for (timeout = 1; timeout <= ACI_MINTIME+30; timeout++) {
-               if (((byte=inb(miro->aci_port + ACI_REG_BUSY)) & 1) == 0) {
+       for (timeout = 1; timeout <= ACI_MINTIME + 30; timeout++) {
+               byte = inb(aci->aci_port + ACI_REG_BUSY);
+               if ((byte & 1) == 0) {
                        if (timeout >= ACI_MINTIME)
                                snd_printd("aci ready in round %ld.\n",
                                           timeout-ACI_MINTIME);
@@ -177,10 +190,10 @@ static int aci_busy_wait(struct snd_miro * miro)
        return -EBUSY;
 }
 
-static inline int aci_write(struct snd_miro * miro, unsigned char byte)
+static inline int aci_write(struct snd_miro_aci *aci, unsigned char byte)
 {
-       if (aci_busy_wait(miro) >= 0) {
-               outb(byte, miro->aci_port + ACI_REG_COMMAND);
+       if (aci_busy_wait(aci) >= 0) {
+               outb(byte, aci->aci_port + ACI_REG_COMMAND);
                return 0;
        } else {
                snd_printk(KERN_ERR "aci busy, aci_write(0x%x) stopped.\n", byte);
@@ -188,12 +201,12 @@ static inline int aci_write(struct snd_miro * miro, unsigned char byte)
        }
 }
 
-static inline int aci_read(struct snd_miro * miro)
+static inline int aci_read(struct snd_miro_aci *aci)
 {
        unsigned char byte;
 
-       if (aci_busy_wait(miro) >= 0) {
-               byte=inb(miro->aci_port + ACI_REG_STATUS);
+       if (aci_busy_wait(aci) >= 0) {
+               byte = inb(aci->aci_port + ACI_REG_STATUS);
                return byte;
        } else {
                snd_printk(KERN_ERR "aci busy, aci_read() stopped.\n");
@@ -201,40 +214,50 @@ static inline int aci_read(struct snd_miro * miro)
        }
 }
 
-static int aci_cmd(struct snd_miro * miro, int write1, int write2, int write3)
+int snd_aci_cmd(struct snd_miro_aci *aci, int write1, int write2, int write3)
 {
        int write[] = {write1, write2, write3};
        int value, i;
 
-       if (mutex_lock_interruptible(&miro->aci_mutex))
+       if (mutex_lock_interruptible(&aci->aci_mutex))
                return -EINTR;
 
        for (i=0; i<3; i++) {
                if (write[i]< 0 || write[i] > 255)
                        break;
                else {
-                       value = aci_write(miro, write[i]);
+                       value = aci_write(aci, write[i]);
                        if (value < 0)
                                goto out;
                }
        }
 
-       value = aci_read(miro);
+       value = aci_read(aci);
 
-out:   mutex_unlock(&miro->aci_mutex);
+out:   mutex_unlock(&aci->aci_mutex);
        return value;
 }
+EXPORT_SYMBOL(snd_aci_cmd);
 
-static int aci_getvalue(struct snd_miro * miro, unsigned char index)
+static int aci_getvalue(struct snd_miro_aci *aci, unsigned char index)
 {
-       return aci_cmd(miro, ACI_STATUS, index, -1);
+       return snd_aci_cmd(aci, ACI_STATUS, index, -1);
 }
 
-static int aci_setvalue(struct snd_miro * miro, unsigned char index, int value)
+static int aci_setvalue(struct snd_miro_aci *aci, unsigned char index,
+                       int value)
 {
-       return aci_cmd(miro, index, value, -1);
+       return snd_aci_cmd(aci, index, value, -1);
 }
 
+struct snd_miro_aci *snd_aci_get_aci(void)
+{
+       if (aci_device.aci_port == 0)
+               return NULL;
+       return &aci_device;
+}
+EXPORT_SYMBOL(snd_aci_get_aci);
+
 /*
  *  MIXER part
  */
@@ -247,8 +270,10 @@ static int snd_miro_get_capture(struct snd_kcontrol *kcontrol,
        struct snd_miro *miro = snd_kcontrol_chip(kcontrol);
        int value;
 
-       if ((value = aci_getvalue(miro, ACI_S_GENERAL)) < 0) {
-               snd_printk(KERN_ERR "snd_miro_get_capture() failed: %d\n", value);
+       value = aci_getvalue(miro->aci, ACI_S_GENERAL);
+       if (value < 0) {
+               snd_printk(KERN_ERR "snd_miro_get_capture() failed: %d\n",
+                          value);
                return value;
        }
 
@@ -265,13 +290,15 @@ static int snd_miro_put_capture(struct snd_kcontrol *kcontrol,
 
        value = !(ucontrol->value.integer.value[0]);
 
-       if ((error = aci_setvalue(miro, ACI_SET_SOLOMODE, value)) < 0) {
-               snd_printk(KERN_ERR "snd_miro_put_capture() failed: %d\n", error);
+       error = aci_setvalue(miro->aci, ACI_SET_SOLOMODE, value);
+       if (error < 0) {
+               snd_printk(KERN_ERR "snd_miro_put_capture() failed: %d\n",
+                          error);
                return error;
        }
 
-       change = (value != miro->aci_solomode);
-       miro->aci_solomode = value;
+       change = (value != miro->aci->aci_solomode);
+       miro->aci->aci_solomode = value;
        
        return change;
 }
@@ -293,7 +320,7 @@ static int snd_miro_get_preamp(struct snd_kcontrol *kcontrol,
        struct snd_miro *miro = snd_kcontrol_chip(kcontrol);
        int value;
 
-       if (miro->aci_version <= 176) {
+       if (miro->aci->aci_version <= 176) {
 
                /* 
                   OSS says it's not readable with versions < 176.
@@ -301,12 +328,14 @@ static int snd_miro_get_preamp(struct snd_kcontrol *kcontrol,
                   which is a PCM12 with aci_version = 176.
                */
 
-               ucontrol->value.integer.value[0] = miro->aci_preamp;
+               ucontrol->value.integer.value[0] = miro->aci->aci_preamp;
                return 0;
        }
 
-       if ((value = aci_getvalue(miro, ACI_GET_PREAMP)) < 0) {
-               snd_printk(KERN_ERR "snd_miro_get_preamp() failed: %d\n", value);
+       value = aci_getvalue(miro->aci, ACI_GET_PREAMP);
+       if (value < 0) {
+               snd_printk(KERN_ERR "snd_miro_get_preamp() failed: %d\n",
+                          value);
                return value;
        }
        
@@ -323,13 +352,15 @@ static int snd_miro_put_preamp(struct snd_kcontrol *kcontrol,
 
        value = ucontrol->value.integer.value[0];
 
-       if ((error = aci_setvalue(miro, ACI_SET_PREAMP, value)) < 0) {
-               snd_printk(KERN_ERR "snd_miro_put_preamp() failed: %d\n", error);
+       error = aci_setvalue(miro->aci, ACI_SET_PREAMP, value);
+       if (error < 0) {
+               snd_printk(KERN_ERR "snd_miro_put_preamp() failed: %d\n",
+                          error);
                return error;
        }
 
-       change = (value != miro->aci_preamp);
-       miro->aci_preamp = value;
+       change = (value != miro->aci->aci_preamp);
+       miro->aci->aci_preamp = value;
 
        return change;
 }
@@ -340,7 +371,7 @@ static int snd_miro_get_amp(struct snd_kcontrol *kcontrol,
                            struct snd_ctl_elem_value *ucontrol)
 {
        struct snd_miro *miro = snd_kcontrol_chip(kcontrol);
-       ucontrol->value.integer.value[0] = miro->aci_amp;
+       ucontrol->value.integer.value[0] = miro->aci->aci_amp;
 
        return 0;
 }
@@ -353,13 +384,14 @@ static int snd_miro_put_amp(struct snd_kcontrol *kcontrol,
 
        value = ucontrol->value.integer.value[0];
 
-       if ((error = aci_setvalue(miro, ACI_SET_POWERAMP, value)) < 0) {
+       error = aci_setvalue(miro->aci, ACI_SET_POWERAMP, value);
+       if (error < 0) {
                snd_printk(KERN_ERR "snd_miro_put_amp() to %d failed: %d\n", value, error);
                return error;
        }
 
-       change = (value != miro->aci_amp);
-       miro->aci_amp = value;
+       change = (value != miro->aci->aci_amp);
+       miro->aci->aci_amp = value;
 
        return change;
 }
@@ -408,12 +440,14 @@ static int snd_miro_get_double(struct snd_kcontrol *kcontrol,
        int right_reg = kcontrol->private_value & 0xff;
        int left_reg = right_reg + 1;
 
-       if ((right_val = aci_getvalue(miro, right_reg)) < 0) {
+       right_val = aci_getvalue(miro->aci, right_reg);
+       if (right_val < 0) {
                snd_printk(KERN_ERR "aci_getvalue(%d) failed: %d\n", right_reg, right_val);
                return right_val;
        }
 
-       if ((left_val = aci_getvalue(miro, left_reg)) < 0) {
+       left_val = aci_getvalue(miro->aci, left_reg);
+       if (left_val < 0) {
                snd_printk(KERN_ERR "aci_getvalue(%d) failed: %d\n", left_reg, left_val);
                return left_val;
        }
@@ -449,6 +483,7 @@ static int snd_miro_put_double(struct snd_kcontrol *kcontrol,
                               struct snd_ctl_elem_value *ucontrol)
 {
        struct snd_miro *miro = snd_kcontrol_chip(kcontrol);
+       struct snd_miro_aci *aci = miro->aci;
        int left, right, left_old, right_old;
        int setreg_left, setreg_right, getreg_left, getreg_right;
        int change, error;
@@ -457,21 +492,21 @@ static int snd_miro_put_double(struct snd_kcontrol *kcontrol,
        right = ucontrol->value.integer.value[1];
 
        setreg_right = (kcontrol->private_value >> 8) & 0xff;
-       if (setreg_right == ACI_SET_MASTER) {
-               setreg_left = setreg_right + 1;
-       } else {
-               setreg_left = setreg_right + 8;
-       }
+       setreg_left = setreg_right + 8;
+       if (setreg_right == ACI_SET_MASTER)
+               setreg_left -= 7;
 
        getreg_right = kcontrol->private_value & 0xff;
        getreg_left = getreg_right + 1;
 
-       if ((left_old = aci_getvalue(miro, getreg_left)) < 0) {
+       left_old = aci_getvalue(aci, getreg_left);
+       if (left_old < 0) {
                snd_printk(KERN_ERR "aci_getvalue(%d) failed: %d\n", getreg_left, left_old);
                return left_old;
        }
 
-       if ((right_old = aci_getvalue(miro, getreg_right)) < 0) {
+       right_old = aci_getvalue(aci, getreg_right);
+       if (right_old < 0) {
                snd_printk(KERN_ERR "aci_getvalue(%d) failed: %d\n", getreg_right, right_old);
                return right_old;
        }
@@ -490,13 +525,15 @@ static int snd_miro_put_double(struct snd_kcontrol *kcontrol,
                        right_old = 0x80 - right_old;
 
                if (left >= 0) {
-                       if ((error = aci_setvalue(miro, setreg_left, left)) < 0) {
+                       error = aci_setvalue(aci, setreg_left, left);
+                       if (error < 0) {
                                snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                           left, error);
                                return error;
                        }
                } else {
-                       if ((error = aci_setvalue(miro, setreg_left, 0x80 - left)) < 0) {
+                       error = aci_setvalue(aci, setreg_left, 0x80 - left);
+                       if (error < 0) {
                                snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                           0x80 - left, error);
                                return error;
@@ -504,13 +541,15 @@ static int snd_miro_put_double(struct snd_kcontrol *kcontrol,
                }
 
                if (right >= 0) {
-                       if ((error = aci_setvalue(miro, setreg_right, right)) < 0) {
+                       error = aci_setvalue(aci, setreg_right, right);
+                       if (error < 0) {
                                snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                           right, error);
                                return error;
                        }
                } else {
-                       if ((error = aci_setvalue(miro, setreg_right, 0x80 - right)) < 0) {
+                       error = aci_setvalue(aci, setreg_right, 0x80 - right);
+                       if (error < 0) {
                                snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                           0x80 - right, error);
                                return error;
@@ -528,12 +567,14 @@ static int snd_miro_put_double(struct snd_kcontrol *kcontrol,
                left_old = 0x20 - left_old;
                right_old = 0x20 - right_old;
 
-               if ((error = aci_setvalue(miro, setreg_left, 0x20 - left)) < 0) {
+               error = aci_setvalue(aci, setreg_left, 0x20 - left);
+               if (error < 0) {
                        snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                   0x20 - left, error);
                        return error;
                }
-               if ((error = aci_setvalue(miro, setreg_right, 0x20 - right)) < 0) {
+               error = aci_setvalue(aci, setreg_right, 0x20 - right);
+               if (error < 0) {
                        snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n",
                                   0x20 - right, error);
                        return error;
@@ -631,11 +672,13 @@ static unsigned char aci_init_values[][2] __devinitdata = {
 static int __devinit snd_set_aci_init_values(struct snd_miro *miro)
 {
        int idx, error;
+       struct snd_miro_aci *aci = miro->aci;
 
        /* enable WSS on PCM1 */
 
-       if ((miro->aci_product == 'A') && wss) {
-               if ((error = aci_setvalue(miro, ACI_SET_WSS, wss)) < 0) {
+       if ((aci->aci_product == 'A') && wss) {
+               error = aci_setvalue(aci, ACI_SET_WSS, wss);
+               if (error < 0) {
                        snd_printk(KERN_ERR "enabling WSS mode failed\n");
                        return error;
                }
@@ -644,7 +687,8 @@ static int __devinit snd_set_aci_init_values(struct snd_miro *miro)
        /* enable IDE port */
 
        if (ide) {
-               if ((error = aci_setvalue(miro, ACI_SET_IDE, ide)) < 0) {
+               error = aci_setvalue(aci, ACI_SET_IDE, ide);
+               if (error < 0) {
                        snd_printk(KERN_ERR "enabling IDE port failed\n");
                        return error;
                }
@@ -652,32 +696,31 @@ static int __devinit snd_set_aci_init_values(struct snd_miro *miro)
 
        /* set common aci values */
 
-       for (idx = 0; idx < ARRAY_SIZE(aci_init_values); idx++)
-                if ((error = aci_setvalue(miro, aci_init_values[idx][0], 
-                                         aci_init_values[idx][1])) < 0) {
+       for (idx = 0; idx < ARRAY_SIZE(aci_init_values); idx++) {
+               error = aci_setvalue(aci, aci_init_values[idx][0],
+                                    aci_init_values[idx][1]);
+               if (error < 0) {
                        snd_printk(KERN_ERR "aci_setvalue(%d) failed: %d\n", 
                                   aci_init_values[idx][0], error);
                         return error;
                 }
-
-       miro->aci_amp = 0;
-       miro->aci_preamp = 0;
-       miro->aci_solomode = 1;
+       }
+       aci->aci_amp = 0;
+       aci->aci_preamp = 0;
+       aci->aci_solomode = 1;
 
        return 0;
 }
 
-static int __devinit snd_miro_mixer(struct snd_miro *miro)
+static int __devinit snd_miro_mixer(struct snd_card *card,
+                                   struct snd_miro *miro)
 {
-       struct snd_card *card;
        unsigned int idx;
        int err;
 
-       if (snd_BUG_ON(!miro || !miro->card))
+       if (snd_BUG_ON(!miro || !card))
                return -EINVAL;
 
-       card = miro->card;
-
        switch (miro->hardware) {
        case OPTi9XX_HW_82C924:
                strcpy(card->mixername, "ACI & OPTi924");
@@ -695,7 +738,8 @@ static int __devinit snd_miro_mixer(struct snd_miro *miro)
                        return err;
        }
 
-       if ((miro->aci_product == 'A') || (miro->aci_product == 'B')) {
+       if ((miro->aci->aci_product == 'A') ||
+           (miro->aci->aci_product == 'B')) {
                /* PCM1/PCM12 with power-amp and Line 2 */
                if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_miro_line_control[0], miro))) < 0)
                        return err;
@@ -703,16 +747,17 @@ static int __devinit snd_miro_mixer(struct snd_miro *miro)
                        return err;
        }
 
-       if ((miro->aci_product == 'B') || (miro->aci_product == 'C')) {
+       if ((miro->aci->aci_product == 'B') ||
+           (miro->aci->aci_product == 'C')) {
                /* PCM12/PCM20 with mic-preamp */
                if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_miro_preamp_control[0], miro))) < 0)
                        return err;
-               if (miro->aci_version >= 176)
+               if (miro->aci->aci_version >= 176)
                        if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_miro_capture_control[0], miro))) < 0)
                                return err;
        }
 
-       if (miro->aci_product == 'C') {
+       if (miro->aci->aci_product == 'C') {
                /* PCM20 with radio and 7 band equalizer */
                if ((err = snd_ctl_add(card, snd_ctl_new1(&snd_miro_radio_control[0], miro))) < 0)
                        return err;
@@ -758,17 +803,23 @@ static int __devinit snd_miro_init(struct snd_miro *chip,
        chip->mpu_port = -1;
        chip->mpu_irq = -1;
 
+       chip->pwd_reg = 3;
+
+#ifdef CONFIG_PNP
+       if (isapnp && chip->mc_base)
+               /* PnP resource gives the least 10 bits */
+               chip->mc_base |= 0xc00;
+       else
+#endif
+               chip->mc_base = 0xf8c;
+
        switch (hardware) {
        case OPTi9XX_HW_82C929:
-               chip->mc_base = 0xf8c;
                chip->password = 0xe3;
-               chip->pwd_reg = 3;
                break;
 
        case OPTi9XX_HW_82C924:
-               chip->mc_base = 0xf8c;
                chip->password = 0xe5;
-               chip->pwd_reg = 3;
                break;
 
        default:
@@ -850,14 +901,15 @@ static void snd_miro_proc_read(struct snd_info_entry * entry,
                               struct snd_info_buffer *buffer)
 {
        struct snd_miro *miro = (struct snd_miro *) entry->private_data;
+       struct snd_miro_aci *aci = miro->aci;
        char* model = "unknown";
 
        /* miroSOUND PCM1 pro, early PCM12 */
 
        if ((miro->hardware == OPTi9XX_HW_82C929) &&
-           (miro->aci_vendor == 'm') && 
-           (miro->aci_product == 'A')) {
-               switch(miro->aci_version) {
+           (aci->aci_vendor == 'm') &&
+           (aci->aci_product == 'A')) {
+               switch (aci->aci_version) {
                case 3:
                        model = "miroSOUND PCM1 pro";
                        break;
@@ -870,9 +922,9 @@ static void snd_miro_proc_read(struct snd_info_entry * entry,
        /* miroSOUND PCM12, PCM12 (Rev. E), PCM12 pnp */
 
        if ((miro->hardware == OPTi9XX_HW_82C924) &&
-           (miro->aci_vendor == 'm') && 
-           (miro->aci_product == 'B')) {
-               switch(miro->aci_version) {
+           (aci->aci_vendor == 'm') &&
+           (aci->aci_product == 'B')) {
+               switch (aci->aci_version) {
                case 4:
                        model = "miroSOUND PCM12";
                        break;
@@ -888,9 +940,9 @@ static void snd_miro_proc_read(struct snd_info_entry * entry,
        /* miroSOUND PCM20 radio */
 
        if ((miro->hardware == OPTi9XX_HW_82C924) &&
-           (miro->aci_vendor == 'm') && 
-           (miro->aci_product == 'C')) {
-               switch(miro->aci_version) {
+           (aci->aci_vendor == 'm') &&
+           (aci->aci_product == 'C')) {
+               switch (aci->aci_version) {
                case 7:
                        model = "miroSOUND PCM20 radio (Rev. E)";
                        break;
@@ -914,17 +966,17 @@ static void snd_miro_proc_read(struct snd_info_entry * entry,
 
        snd_iprintf(buffer, "ACI information:\n");
        snd_iprintf(buffer, "  vendor  : ");
-       switch(miro->aci_vendor) {
+       switch (aci->aci_vendor) {
        case 'm':
                snd_iprintf(buffer, "Miro\n");
                break;
        default:
-               snd_iprintf(buffer, "unknown (0x%x)\n", miro->aci_vendor);
+               snd_iprintf(buffer, "unknown (0x%x)\n", aci->aci_vendor);
                break;
        }
 
        snd_iprintf(buffer, "  product : ");
-       switch(miro->aci_product) {
+       switch (aci->aci_product) {
        case 'A':
                snd_iprintf(buffer, "miroSOUND PCM1 pro / (early) PCM12\n");
                break;
@@ -935,26 +987,27 @@ static void snd_miro_proc_read(struct snd_info_entry * entry,
                snd_iprintf(buffer, "miroSOUND PCM20 radio\n");
                break;
        default:
-               snd_iprintf(buffer, "unknown (0x%x)\n", miro->aci_product);
+               snd_iprintf(buffer, "unknown (0x%x)\n", aci->aci_product);
                break;
        }
 
        snd_iprintf(buffer, "  firmware: %d (0x%x)\n",
-                   miro->aci_version, miro->aci_version);
+                   aci->aci_version, aci->aci_version);
        snd_iprintf(buffer, "  port    : 0x%lx-0x%lx\n", 
-                   miro->aci_port, miro->aci_port+2);
+                   aci->aci_port, aci->aci_port+2);
        snd_iprintf(buffer, "  wss     : 0x%x\n", wss);
        snd_iprintf(buffer, "  ide     : 0x%x\n", ide);
-       snd_iprintf(buffer, "  solomode: 0x%x\n", miro->aci_solomode);
-       snd_iprintf(buffer, "  amp     : 0x%x\n", miro->aci_amp);
-       snd_iprintf(buffer, "  preamp  : 0x%x\n", miro->aci_preamp);
+       snd_iprintf(buffer, "  solomode: 0x%x\n", aci->aci_solomode);
+       snd_iprintf(buffer, "  amp     : 0x%x\n", aci->aci_amp);
+       snd_iprintf(buffer, "  preamp  : 0x%x\n", aci->aci_preamp);
 }
 
-static void __devinit snd_miro_proc_init(struct snd_miro * miro)
+static void __devinit snd_miro_proc_init(struct snd_card *card,
+                                        struct snd_miro *miro)
 {
        struct snd_info_entry *entry;
 
-       if (! snd_card_proc_new(miro->card, "miro", &entry))
+       if (!snd_card_proc_new(card, "miro", &entry))
                snd_info_set_text_ops(entry, miro, snd_miro_proc_read);
 }
 
@@ -971,37 +1024,40 @@ static int __devinit snd_miro_configure(struct snd_miro *chip)
        unsigned char mpu_irq_bits;
        unsigned long flags;
 
+       snd_miro_write_mask(chip, OPTi9XX_MC_REG(1), 0x80, 0x80);
+       snd_miro_write_mask(chip, OPTi9XX_MC_REG(2), 0x20, 0x20); /* OPL4 */
+       snd_miro_write_mask(chip, OPTi9XX_MC_REG(5), 0x02, 0x02);
+
        switch (chip->hardware) {
        case OPTi9XX_HW_82C924:
                snd_miro_write_mask(chip, OPTi9XX_MC_REG(6), 0x02, 0x02);
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(1), 0x80, 0x80);
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(2), 0x20, 0x20); /* OPL4 */
                snd_miro_write_mask(chip, OPTi9XX_MC_REG(3), 0xf0, 0xff);
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(5), 0x02, 0x02);
                break;
        case OPTi9XX_HW_82C929:
                /* untested init commands for OPTi929 */
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(1), 0x80, 0x80);
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(2), 0x20, 0x20); /* OPL4 */
                snd_miro_write_mask(chip, OPTi9XX_MC_REG(4), 0x00, 0x0c);
-               snd_miro_write_mask(chip, OPTi9XX_MC_REG(5), 0x02, 0x02);
                break;
        default:
                snd_printk(KERN_ERR "chip %d not supported\n", chip->hardware);
                return -EINVAL;
        }
 
-       switch (chip->wss_base) {
-       case 0x530:
+       /* PnP resource says it decodes only 10 bits of address */
+       switch (chip->wss_base & 0x3ff) {
+       case 0x130:
+               chip->wss_base = 0x530;
                wss_base_bits = 0x00;
                break;
-       case 0x604:
+       case 0x204:
+               chip->wss_base = 0x604;
                wss_base_bits = 0x03;
                break;
-       case 0xe80:
+       case 0x280:
+               chip->wss_base = 0xe80;
                wss_base_bits = 0x01;
                break;
-       case 0xf40:
+       case 0x340:
+               chip->wss_base = 0xf40;
                wss_base_bits = 0x02;
                break;
        default:
@@ -1119,75 +1175,92 @@ __skip_mpu:
        return 0;
 }
 
+static int __devinit snd_miro_opti_check(struct snd_miro *chip)
+{
+       unsigned char value;
+
+       chip->res_mc_base = request_region(chip->mc_base, chip->mc_base_size,
+                                          "OPTi9xx MC");
+       if (chip->res_mc_base == NULL)
+               return -ENOMEM;
+
+       value = snd_miro_read(chip, OPTi9XX_MC_REG(1));
+       if (value != 0xff && value != inb(chip->mc_base + OPTi9XX_MC_REG(1)))
+               if (value == snd_miro_read(chip, OPTi9XX_MC_REG(1)))
+                       return 0;
+
+       release_and_free_resource(chip->res_mc_base);
+       chip->res_mc_base = NULL;
+
+       return -ENODEV;
+}
+
 static int __devinit snd_card_miro_detect(struct snd_card *card,
                                          struct snd_miro *chip)
 {
        int i, err;
-       unsigned char value;
 
        for (i = OPTi9XX_HW_82C929; i <= OPTi9XX_HW_82C924; i++) {
 
                if ((err = snd_miro_init(chip, i)) < 0)
                        return err;
 
-               if ((chip->res_mc_base = request_region(chip->mc_base, chip->mc_base_size, "OPTi9xx MC")) == NULL)
-                       continue;
-
-               value = snd_miro_read(chip, OPTi9XX_MC_REG(1));
-               if ((value != 0xff) && (value != inb(chip->mc_base + 1)))
-                       if (value == snd_miro_read(chip, OPTi9XX_MC_REG(1)))
-                               return 1;
-
-               release_and_free_resource(chip->res_mc_base);
-               chip->res_mc_base = NULL;
-
+               err = snd_miro_opti_check(chip);
+               if (err == 0)
+                       return 1;
        }
 
        return -ENODEV;
 }
 
 static int __devinit snd_card_miro_aci_detect(struct snd_card *card,
-                                             struct snd_miro * miro)
+                                             struct snd_miro *miro)
 {
        unsigned char regval;
        int i;
+       struct snd_miro_aci *aci = &aci_device;
 
-       mutex_init(&miro->aci_mutex);
+       miro->aci = aci;
+
+       mutex_init(&aci->aci_mutex);
 
        /* get ACI port from OPTi9xx MC 4 */
 
-       miro->mc_base = 0xf8c;
        regval=inb(miro->mc_base + 4);
-       miro->aci_port = (regval & 0x10) ? 0x344: 0x354;
+       aci->aci_port = (regval & 0x10) ? 0x344 : 0x354;
 
-       if ((miro->res_aci_port = request_region(miro->aci_port, 3, "miro aci")) == NULL) {
+       miro->res_aci_port = request_region(aci->aci_port, 3, "miro aci");
+       if (miro->res_aci_port == NULL) {
                snd_printk(KERN_ERR "aci i/o area 0x%lx-0x%lx already used.\n", 
-                          miro->aci_port, miro->aci_port+2);
+                          aci->aci_port, aci->aci_port+2);
                return -ENOMEM;
        }
 
         /* force ACI into a known state */
        for (i = 0; i < 3; i++)
-               if (aci_cmd(miro, ACI_ERROR_OP, -1, -1) < 0) {
+               if (snd_aci_cmd(aci, ACI_ERROR_OP, -1, -1) < 0) {
                        snd_printk(KERN_ERR "can't force aci into known state.\n");
                        return -ENXIO;
                }
 
-       if ((miro->aci_vendor=aci_cmd(miro, ACI_READ_IDCODE, -1, -1)) < 0 ||
-           (miro->aci_product=aci_cmd(miro, ACI_READ_IDCODE, -1, -1)) < 0) {
-               snd_printk(KERN_ERR "can't read aci id on 0x%lx.\n", miro->aci_port);
+       aci->aci_vendor = snd_aci_cmd(aci, ACI_READ_IDCODE, -1, -1);
+       aci->aci_product = snd_aci_cmd(aci, ACI_READ_IDCODE, -1, -1);
+       if (aci->aci_vendor < 0 || aci->aci_product < 0) {
+               snd_printk(KERN_ERR "can't read aci id on 0x%lx.\n",
+                          aci->aci_port);
                return -ENXIO;
        }
 
-       if ((miro->aci_version=aci_cmd(miro, ACI_READ_VERSION, -1, -1)) < 0) {
+       aci->aci_version = snd_aci_cmd(aci, ACI_READ_VERSION, -1, -1);
+       if (aci->aci_version < 0) {
                snd_printk(KERN_ERR "can't read aci version on 0x%lx.\n", 
-                          miro->aci_port);
+                          aci->aci_port);
                return -ENXIO;
        }
 
-       if (aci_cmd(miro, ACI_INIT, -1, -1) < 0 ||
-           aci_cmd(miro, ACI_ERROR_OP, ACI_ERROR_OP, ACI_ERROR_OP) < 0 ||
-           aci_cmd(miro, ACI_ERROR_OP, ACI_ERROR_OP, ACI_ERROR_OP) < 0) {
+       if (snd_aci_cmd(aci, ACI_INIT, -1, -1) < 0 ||
+           snd_aci_cmd(aci, ACI_ERROR_OP, ACI_ERROR_OP, ACI_ERROR_OP) < 0 ||
+           snd_aci_cmd(aci, ACI_ERROR_OP, ACI_ERROR_OP, ACI_ERROR_OP) < 0) {
                snd_printk(KERN_ERR "can't initialize aci.\n"); 
                return -ENXIO;
        }
@@ -1198,65 +1271,39 @@ static int __devinit snd_card_miro_aci_detect(struct snd_card *card,
 static void snd_card_miro_free(struct snd_card *card)
 {
        struct snd_miro *miro = card->private_data;
-        
+
        release_and_free_resource(miro->res_aci_port);
+       if (miro->aci)
+               miro->aci->aci_port = 0;
        release_and_free_resource(miro->res_mc_base);
 }
 
-static int __devinit snd_miro_match(struct device *devptr, unsigned int n)
-{
-       return 1;
-}
-
-static int __devinit snd_miro_probe(struct device *devptr, unsigned int n)
+static int __devinit snd_miro_probe(struct snd_card *card)
 {
-       static long possible_ports[] = {0x530, 0xe80, 0xf40, 0x604, -1};
-       static long possible_mpu_ports[] = {0x330, 0x300, 0x310, 0x320, -1};
-       static int possible_irqs[] = {11, 9, 10, 7, -1};
-       static int possible_mpu_irqs[] = {10, 5, 9, 7, -1};
-       static int possible_dma1s[] = {3, 1, 0, -1};
-       static int possible_dma2s[][2] = {{1,-1}, {0,-1}, {-1,-1}, {0,-1}};
-
        int error;
-       struct snd_miro *miro;
+       struct snd_miro *miro = card->private_data;
        struct snd_wss *codec;
        struct snd_timer *timer;
-       struct snd_card *card;
        struct snd_pcm *pcm;
        struct snd_rawmidi *rmidi;
 
-       error = snd_card_create(index, id, THIS_MODULE,
-                               sizeof(struct snd_miro), &card);
-       if (error < 0)
-               return error;
-
-       card->private_free = snd_card_miro_free;
-       miro = card->private_data;
-       miro->card = card;
-
-       if ((error = snd_card_miro_aci_detect(card, miro)) < 0) {
-               snd_card_free(card);
-               snd_printk(KERN_ERR "unable to detect aci chip\n");
-               return -ENODEV;
+       if (!miro->res_mc_base) {
+               miro->res_mc_base = request_region(miro->mc_base,
+                                               miro->mc_base_size,
+                                               "miro (OPTi9xx MC)");
+               if (miro->res_mc_base == NULL) {
+                       snd_printk(KERN_ERR "request for OPTI9xx MC failed\n");
+                       return -ENOMEM;
+               }
        }
 
-       /* init proc interface */
-       snd_miro_proc_init(miro);
-
-       if ((error = snd_card_miro_detect(card, miro)) < 0) {
+       error = snd_card_miro_aci_detect(card, miro);
+       if (error < 0) {
                snd_card_free(card);
-               snd_printk(KERN_ERR "unable to detect OPTi9xx chip\n");
+               snd_printk(KERN_ERR "unable to detect aci chip\n");
                return -ENODEV;
        }
 
-       if (! miro->res_mc_base &&
-           (miro->res_mc_base = request_region(miro->mc_base, miro->mc_base_size,
-                                               "miro (OPTi9xx MC)")) == NULL) {
-               snd_card_free(card);
-               snd_printk(KERN_ERR "request for OPTI9xx MC failed\n");
-               return -ENOMEM;
-       }
-
        miro->wss_base = port;
        miro->mpu_port = mpu_port;
        miro->irq = irq;
@@ -1264,91 +1311,40 @@ static int __devinit snd_miro_probe(struct device *devptr, unsigned int n)
        miro->dma1 = dma1;
        miro->dma2 = dma2;
 
-       if (miro->wss_base == SNDRV_AUTO_PORT) {
-               if ((miro->wss_base = snd_legacy_find_free_ioport(possible_ports, 4)) < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free WSS port\n");
-                       return -EBUSY;
-               }
-       }
-
-       if (mpu_port == SNDRV_AUTO_PORT) {
-               mpu_port = snd_legacy_find_free_ioport(possible_mpu_ports, 2);
-               if (mpu_port < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free MPU401 port\n");
-                       return -EBUSY;
-               }
-       }
-       if (miro->irq == SNDRV_AUTO_IRQ) {
-               if ((miro->irq = snd_legacy_find_free_irq(possible_irqs)) < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free IRQ\n");
-                       return -EBUSY;
-               }
-       }
-       if (miro->mpu_irq == SNDRV_AUTO_IRQ) {
-               if ((miro->mpu_irq = snd_legacy_find_free_irq(possible_mpu_irqs)) < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free MPU401 IRQ\n");
-                       return -EBUSY;
-               }
-       }
-       if (miro->dma1 == SNDRV_AUTO_DMA) {
-               if ((miro->dma1 = snd_legacy_find_free_dma(possible_dma1s)) < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free DMA1\n");
-                       return -EBUSY;
-               }
-       }
-       if (miro->dma2 == SNDRV_AUTO_DMA) {
-               if ((miro->dma2 = snd_legacy_find_free_dma(possible_dma2s[miro->dma1 % 4])) < 0) {
-                       snd_card_free(card);
-                       snd_printk(KERN_ERR "unable to find a free DMA2\n");
-                       return -EBUSY;
-               }
-       }
+       /* init proc interface */
+       snd_miro_proc_init(card, miro);
 
        error = snd_miro_configure(miro);
-       if (error) {
-               snd_card_free(card);
+       if (error)
                return error;
-       }
 
        error = snd_wss_create(card, miro->wss_base + 4, -1,
-                               miro->irq, miro->dma1, miro->dma2,
-                               WSS_HW_AD1845, 0, &codec);
-       if (error < 0) {
-               snd_card_free(card);
+                              miro->irq, miro->dma1, miro->dma2,
+                              WSS_HW_DETECT, 0, &codec);
+       if (error < 0)
                return error;
-       }
 
        error = snd_wss_pcm(codec, 0, &pcm);
-       if (error < 0)  {
-               snd_card_free(card);
+       if (error < 0)
                return error;
-       }
+
        error = snd_wss_mixer(codec);
-       if (error < 0) {
-               snd_card_free(card);
+       if (error < 0)
                return error;
-       }
+
        error = snd_wss_timer(codec, 0, &timer);
-       if (error < 0) {
-               snd_card_free(card);
+       if (error < 0)
                return error;
-       }
 
        miro->pcm = pcm;
 
-       if ((error = snd_miro_mixer(miro)) < 0) {
-               snd_card_free(card);
+       error = snd_miro_mixer(card, miro);
+       if (error < 0)
                return error;
-       }
 
-       if (miro->aci_vendor == 'm') {
+       if (miro->aci->aci_vendor == 'm') {
                /* It looks like a miro sound card. */
-               switch (miro->aci_product) {
+               switch (miro->aci->aci_product) {
                case 'A':
                        sprintf(card->shortname, 
                                "miroSOUND PCM1 pro / PCM12");
@@ -1391,20 +1387,117 @@ static int __devinit snd_miro_probe(struct device *devptr, unsigned int n)
        if (fm_port > 0 && fm_port != SNDRV_AUTO_PORT) {
                struct snd_opl3 *opl3 = NULL;
                struct snd_opl4 *opl4;
+
                if (snd_opl4_create(card, fm_port, fm_port - 8,
                                    2, &opl3, &opl4) < 0)
                        snd_printk(KERN_WARNING "no OPL4 device at 0x%lx\n",
                                   fm_port);
        }
 
-       if ((error = snd_set_aci_init_values(miro)) < 0) {
-               snd_card_free(card);
+       error = snd_set_aci_init_values(miro);
+       if (error < 0)
                 return error;
+
+       return snd_card_register(card);
+}
+
+static int __devinit snd_miro_isa_match(struct device *devptr, unsigned int n)
+{
+#ifdef CONFIG_PNP
+       if (snd_miro_pnp_is_probed)
+               return 0;
+       if (isapnp)
+               return 0;
+#endif
+       return 1;
+}
+
+static int __devinit snd_miro_isa_probe(struct device *devptr, unsigned int n)
+{
+       static long possible_ports[] = {0x530, 0xe80, 0xf40, 0x604, -1};
+       static long possible_mpu_ports[] = {0x330, 0x300, 0x310, 0x320, -1};
+       static int possible_irqs[] = {11, 9, 10, 7, -1};
+       static int possible_mpu_irqs[] = {10, 5, 9, 7, -1};
+       static int possible_dma1s[] = {3, 1, 0, -1};
+       static int possible_dma2s[][2] = { {1, -1}, {0, -1}, {-1, -1},
+                                          {0, -1} };
+
+       int error;
+       struct snd_miro *miro;
+       struct snd_card *card;
+
+       error = snd_card_create(index, id, THIS_MODULE,
+                               sizeof(struct snd_miro), &card);
+       if (error < 0)
+               return error;
+
+       card->private_free = snd_card_miro_free;
+       miro = card->private_data;
+
+       error = snd_card_miro_detect(card, miro);
+       if (error < 0) {
+               snd_card_free(card);
+               snd_printk(KERN_ERR "unable to detect OPTi9xx chip\n");
+               return -ENODEV;
+       }
+
+       if (port == SNDRV_AUTO_PORT) {
+               port = snd_legacy_find_free_ioport(possible_ports, 4);
+               if (port < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR "unable to find a free WSS port\n");
+                       return -EBUSY;
+               }
+       }
+
+       if (mpu_port == SNDRV_AUTO_PORT) {
+               mpu_port = snd_legacy_find_free_ioport(possible_mpu_ports, 2);
+               if (mpu_port < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR
+                                  "unable to find a free MPU401 port\n");
+                       return -EBUSY;
+               }
+       }
+
+       if (irq == SNDRV_AUTO_IRQ) {
+               irq = snd_legacy_find_free_irq(possible_irqs);
+               if (irq < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR "unable to find a free IRQ\n");
+                       return -EBUSY;
+               }
+       }
+       if (mpu_irq == SNDRV_AUTO_IRQ) {
+               mpu_irq = snd_legacy_find_free_irq(possible_mpu_irqs);
+               if (mpu_irq < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR
+                                  "unable to find a free MPU401 IRQ\n");
+                       return -EBUSY;
+               }
+       }
+       if (dma1 == SNDRV_AUTO_DMA) {
+               dma1 = snd_legacy_find_free_dma(possible_dma1s);
+               if (dma1 < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR "unable to find a free DMA1\n");
+                       return -EBUSY;
+               }
+       }
+       if (dma2 == SNDRV_AUTO_DMA) {
+               dma2 = snd_legacy_find_free_dma(possible_dma2s[dma1 % 4]);
+               if (dma2 < 0) {
+                       snd_card_free(card);
+                       snd_printk(KERN_ERR "unable to find a free DMA2\n");
+                       return -EBUSY;
+               }
        }
 
        snd_card_set_dev(card, devptr);
 
-       if ((error = snd_card_register(card))) {
+       error = snd_miro_probe(card);
+       if (error < 0) {
                snd_card_free(card);
                return error;
        }
@@ -1413,7 +1506,8 @@ static int __devinit snd_miro_probe(struct device *devptr, unsigned int n)
        return 0;
 }
 
-static int __devexit snd_miro_remove(struct device *devptr, unsigned int dev)
+static int __devexit snd_miro_isa_remove(struct device *devptr,
+                                        unsigned int dev)
 {
        snd_card_free(dev_get_drvdata(devptr));
        dev_set_drvdata(devptr, NULL);
@@ -1423,23 +1517,164 @@ static int __devexit snd_miro_remove(struct device *devptr, unsigned int dev)
 #define DEV_NAME "miro"
 
 static struct isa_driver snd_miro_driver = {
-       .match          = snd_miro_match,
-       .probe          = snd_miro_probe,
-       .remove         = __devexit_p(snd_miro_remove),
+       .match          = snd_miro_isa_match,
+       .probe          = snd_miro_isa_probe,
+       .remove         = __devexit_p(snd_miro_isa_remove),
        /* FIXME: suspend/resume */
        .driver         = {
                .name   = DEV_NAME
        },
 };
 
+#ifdef CONFIG_PNP
+
+static int __devinit snd_card_miro_pnp(struct snd_miro *chip,
+                                       struct pnp_card_link *card,
+                                       const struct pnp_card_device_id *pid)
+{
+       struct pnp_dev *pdev;
+       int err;
+       struct pnp_dev *devmpu;
+       struct pnp_dev *devmc;
+
+       pdev = pnp_request_card_device(card, pid->devs[0].id, NULL);
+       if (pdev == NULL)
+               return -EBUSY;
+
+       devmpu = pnp_request_card_device(card, pid->devs[1].id, NULL);
+       if (devmpu == NULL)
+               return -EBUSY;
+
+       devmc = pnp_request_card_device(card, pid->devs[2].id, NULL);
+       if (devmc == NULL)
+               return -EBUSY;
+
+       err = pnp_activate_dev(pdev);
+       if (err < 0) {
+               snd_printk(KERN_ERR "AUDIO pnp configure failure: %d\n", err);
+               return err;
+       }
+
+       err = pnp_activate_dev(devmc);
+       if (err < 0) {
+               snd_printk(KERN_ERR "MC pnp configure failure: %d\n",
+                                   err);
+               return err;
+       }
+
+       port = pnp_port_start(pdev, 1);
+       fm_port = pnp_port_start(pdev, 2) + 8;
+
+       /*
+        * The MC(0) is never accessed and the miroSOUND PCM20 card does not
+        * include it in the PnP resource range. OPTI93x include it.
+        */
+       chip->mc_base = pnp_port_start(devmc, 0) - 1;
+       chip->mc_base_size = pnp_port_len(devmc, 0) + 1;
+
+       irq = pnp_irq(pdev, 0);
+       dma1 = pnp_dma(pdev, 0);
+       dma2 = pnp_dma(pdev, 1);
+
+       if (mpu_port > 0) {
+               err = pnp_activate_dev(devmpu);
+               if (err < 0) {
+                       snd_printk(KERN_ERR "MPU401 pnp configure failure\n");
+                       mpu_port = -1;
+                       return err;
+               }
+               mpu_port = pnp_port_start(devmpu, 0);
+               mpu_irq = pnp_irq(devmpu, 0);
+       }
+       return 0;
+}
+
+static int __devinit snd_miro_pnp_probe(struct pnp_card_link *pcard,
+                                       const struct pnp_card_device_id *pid)
+{
+       struct snd_card *card;
+       int err;
+       struct snd_miro *miro;
+
+       if (snd_miro_pnp_is_probed)
+               return -EBUSY;
+       if (!isapnp)
+               return -ENODEV;
+       err = snd_card_create(index, id, THIS_MODULE,
+                               sizeof(struct snd_miro), &card);
+       if (err < 0)
+               return err;
+
+       card->private_free = snd_card_miro_free;
+       miro = card->private_data;
+
+       err = snd_card_miro_pnp(miro, pcard, pid);
+       if (err) {
+               snd_card_free(card);
+               return err;
+       }
+
+       /* only miroSOUND PCM20 and PCM12 == OPTi924 */
+       err = snd_miro_init(miro, OPTi9XX_HW_82C924);
+       if (err) {
+               snd_card_free(card);
+               return err;
+       }
+
+       err = snd_miro_opti_check(miro);
+       if (err) {
+               snd_printk(KERN_ERR "OPTI chip not found\n");
+               snd_card_free(card);
+               return err;
+       }
+
+       snd_card_set_dev(card, &pcard->card->dev);
+       err = snd_miro_probe(card);
+       if (err < 0) {
+               snd_card_free(card);
+               return err;
+       }
+       pnp_set_card_drvdata(pcard, card);
+       snd_miro_pnp_is_probed = 1;
+       return 0;
+}
+
+static void __devexit snd_miro_pnp_remove(struct pnp_card_link * pcard)
+{
+       snd_card_free(pnp_get_card_drvdata(pcard));
+       pnp_set_card_drvdata(pcard, NULL);
+       snd_miro_pnp_is_probed = 0;
+}
+
+static struct pnp_card_driver miro_pnpc_driver = {
+       .flags          = PNP_DRIVER_RES_DISABLE,
+       .name           = "miro",
+       .id_table       = snd_miro_pnpids,
+       .probe          = snd_miro_pnp_probe,
+       .remove         = __devexit_p(snd_miro_pnp_remove),
+};
+#endif
+
 static int __init alsa_card_miro_init(void)
 {
+#ifdef CONFIG_PNP
+       pnp_register_card_driver(&miro_pnpc_driver);
+       if (snd_miro_pnp_is_probed)
+               return 0;
+       pnp_unregister_card_driver(&miro_pnpc_driver);
+#endif
        return isa_register_driver(&snd_miro_driver, 1);
 }
 
 static void __exit alsa_card_miro_exit(void)
 {
-       isa_unregister_driver(&snd_miro_driver);
+       if (!snd_miro_pnp_is_probed) {
+               isa_unregister_driver(&snd_miro_driver);
+               return;
+       }
+#ifdef CONFIG_PNP
+       pnp_unregister_card_driver(&miro_pnpc_driver);
+#endif
 }
 
 module_init(alsa_card_miro_init)