vserver 2.0 rc7
[linux-2.6.git] / sound / pci / emu10k1 / emufx.c
index 06d2792..b9fa2e8 100644 (file)
@@ -468,10 +468,13 @@ int snd_emu10k1_fx8010_unregister_irq_handler(emu10k1_t *emu,
 static void snd_emu10k1_write_op(emu10k1_fx8010_code_t *icode, unsigned int *ptr,
                                 u32 op, u32 r, u32 a, u32 x, u32 y)
 {
+       u_int32_t *code;
        snd_assert(*ptr < 512, return);
+       code = (u_int32_t *)icode->code + (*ptr) * 2;
        set_bit(*ptr, icode->code_valid);
-       icode->code[(*ptr)   * 2 + 0] = ((x & 0x3ff) << 10) | (y & 0x3ff);
-       icode->code[(*ptr)++ * 2 + 1] = ((op & 0x0f) << 20) | ((r & 0x3ff) << 10) | (a & 0x3ff);
+       code[0] = ((x & 0x3ff) << 10) | (y & 0x3ff);
+       code[1] = ((op & 0x0f) << 20) | ((r & 0x3ff) << 10) | (a & 0x3ff);
+       (*ptr)++;
 }
 
 #define OP(icode, ptr, op, r, a, x, y) \
@@ -480,10 +483,13 @@ static void snd_emu10k1_write_op(emu10k1_fx8010_code_t *icode, unsigned int *ptr
 static void snd_emu10k1_audigy_write_op(emu10k1_fx8010_code_t *icode, unsigned int *ptr,
                                        u32 op, u32 r, u32 a, u32 x, u32 y)
 {
+       u_int32_t *code;
        snd_assert(*ptr < 1024, return);
+       code = (u_int32_t *)icode->code + (*ptr) * 2;
        set_bit(*ptr, icode->code_valid);
-       icode->code[(*ptr)   * 2 + 0] = ((x & 0x7ff) << 12) | (y & 0x7ff);
-       icode->code[(*ptr)++ * 2 + 1] = ((op & 0x0f) << 24) | ((r & 0x7ff) << 12) | (a & 0x7ff);
+       code[0] = ((x & 0x7ff) << 12) | (y & 0x7ff);
+       code[1] = ((op & 0x0f) << 24) | ((r & 0x7ff) << 12) | (a & 0x7ff);
+       (*ptr)++;
 }
 
 #define A_OP(icode, ptr, op, r, a, x, y) \
@@ -628,7 +634,8 @@ static int snd_emu10k1_verify_controls(emu10k1_t *emu, emu10k1_fx8010_code_t *ic
        snd_ctl_elem_id_t __user *_id;
        snd_ctl_elem_id_t id;
        emu10k1_fx8010_control_gpr_t __user *_gctl;
-       emu10k1_fx8010_control_gpr_t gctl;
+       emu10k1_fx8010_control_gpr_t *gctl;
+       int err;
        
        for (i = 0, _id = icode->gpr_del_controls;
             i < icode->gpr_del_control_count; i++, _id++) {
@@ -637,29 +644,42 @@ static int snd_emu10k1_verify_controls(emu10k1_t *emu, emu10k1_fx8010_code_t *ic
                if (snd_emu10k1_look_for_ctl(emu, &id) == NULL)
                        return -ENOENT;
        }
+       gctl = kmalloc(sizeof(*gctl), GFP_KERNEL);
+       if (! gctl)
+               return -ENOMEM;
+       err = 0;
        for (i = 0, _gctl = icode->gpr_add_controls;
             i < icode->gpr_add_control_count; i++, _gctl++) {
-               if (copy_from_user(&gctl, _gctl, sizeof(gctl)))
-                       return -EFAULT;
-               if (snd_emu10k1_look_for_ctl(emu, &gctl.id))
+               if (copy_from_user(gctl, _gctl, sizeof(*gctl))) {
+                       err = -EFAULT;
+                       goto __error;
+               }
+               if (snd_emu10k1_look_for_ctl(emu, &gctl->id))
                        continue;
                down_read(&emu->card->controls_rwsem);
-               if (snd_ctl_find_id(emu->card, &gctl.id) != NULL) {
+               if (snd_ctl_find_id(emu->card, &gctl->id) != NULL) {
                        up_read(&emu->card->controls_rwsem);
-                       return -EEXIST;
+                       err = -EEXIST;
+                       goto __error;
                }
                up_read(&emu->card->controls_rwsem);
-               if (gctl.id.iface != SNDRV_CTL_ELEM_IFACE_MIXER &&
-                   gctl.id.iface != SNDRV_CTL_ELEM_IFACE_PCM)
-                       return -EINVAL;
+               if (gctl->id.iface != SNDRV_CTL_ELEM_IFACE_MIXER &&
+                   gctl->id.iface != SNDRV_CTL_ELEM_IFACE_PCM) {
+                       err = -EINVAL;
+                       goto __error;
+               }
        }
        for (i = 0, _gctl = icode->gpr_list_controls;
             i < icode->gpr_list_control_count; i++, _gctl++) {
                /* FIXME: we need to check the WRITE access */
-               if (copy_from_user(&gctl, _gctl, sizeof(gctl)))
-                       return -EFAULT;
+               if (copy_from_user(gctl, _gctl, sizeof(*gctl))) {
+                       err = -EFAULT;
+                       goto __error;
+               }
        }
-       return 0;
+ __error:
+       kfree(gctl);
+       return err;
 }
 
 static void snd_emu10k1_ctl_private_free(snd_kcontrol_t *kctl)
@@ -676,52 +696,59 @@ static int snd_emu10k1_add_controls(emu10k1_t *emu, emu10k1_fx8010_code_t *icode
 {
        unsigned int i, j;
        emu10k1_fx8010_control_gpr_t __user *_gctl;
-       emu10k1_fx8010_control_gpr_t gctl;
-       snd_emu10k1_fx8010_ctl_t *ctl, nctl;
+       emu10k1_fx8010_control_gpr_t *gctl;
+       snd_emu10k1_fx8010_ctl_t *ctl, *nctl;
        snd_kcontrol_new_t knew;
        snd_kcontrol_t *kctl;
        snd_ctl_elem_value_t *val;
        int err = 0;
 
        val = (snd_ctl_elem_value_t *)kmalloc(sizeof(*val), GFP_KERNEL);
-       if (!val)
-               return -ENOMEM;
+       gctl = kmalloc(sizeof(*gctl), GFP_KERNEL);
+       nctl = kmalloc(sizeof(*nctl), GFP_KERNEL);
+       if (!val || !gctl || !nctl) {
+               err = -ENOMEM;
+               goto __error;
+       }
+
        for (i = 0, _gctl = icode->gpr_add_controls;
             i < icode->gpr_add_control_count; i++, _gctl++) {
-               if (copy_from_user(&gctl, _gctl, sizeof(gctl))) {
+               if (copy_from_user(gctl, _gctl, sizeof(*gctl))) {
                        err = -EFAULT;
                        goto __error;
                }
-               snd_runtime_check(gctl.id.iface == SNDRV_CTL_ELEM_IFACE_MIXER ||
-                                 gctl.id.iface == SNDRV_CTL_ELEM_IFACE_PCM, err = -EINVAL; goto __error);
-               snd_runtime_check(gctl.id.name[0] != '\0', err = -EINVAL; goto __error);
-               ctl = snd_emu10k1_look_for_ctl(emu, &gctl.id);
+               snd_runtime_check(gctl->id.iface == SNDRV_CTL_ELEM_IFACE_MIXER ||
+                                 gctl->id.iface == SNDRV_CTL_ELEM_IFACE_PCM, err = -EINVAL; goto __error);
+               snd_runtime_check(gctl->id.name[0] != '\0', err = -EINVAL; goto __error);
+               ctl = snd_emu10k1_look_for_ctl(emu, &gctl->id);
                memset(&knew, 0, sizeof(knew));
-               knew.iface = gctl.id.iface;
-               knew.name = gctl.id.name;
-               knew.index = gctl.id.index;
-               knew.device = gctl.id.device;
-               knew.subdevice = gctl.id.subdevice;
+               knew.iface = gctl->id.iface;
+               knew.name = gctl->id.name;
+               knew.index = gctl->id.index;
+               knew.device = gctl->id.device;
+               knew.subdevice = gctl->id.subdevice;
                knew.info = snd_emu10k1_gpr_ctl_info;
                knew.get = snd_emu10k1_gpr_ctl_get;
                knew.put = snd_emu10k1_gpr_ctl_put;
-               memset(&nctl, 0, sizeof(nctl));
-               nctl.vcount = gctl.vcount;
-               nctl.count = gctl.count;
+               memset(nctl, 0, sizeof(*nctl));
+               nctl->vcount = gctl->vcount;
+               nctl->count = gctl->count;
                for (j = 0; j < 32; j++) {
-                       nctl.gpr[j] = gctl.gpr[j];
-                       nctl.value[j] = ~gctl.value[j]; /* inverted, we want to write new value in gpr_ctl_put() */
-                       val->value.integer.value[j] = gctl.value[j];
+                       nctl->gpr[j] = gctl->gpr[j];
+                       nctl->value[j] = ~gctl->value[j];       /* inverted, we want to write new value in gpr_ctl_put() */
+                       val->value.integer.value[j] = gctl->value[j];
                }
-               nctl.min = gctl.min;
-               nctl.max = gctl.max;
-               nctl.translation = gctl.translation;
+               nctl->min = gctl->min;
+               nctl->max = gctl->max;
+               nctl->translation = gctl->translation;
                if (ctl == NULL) {
                        ctl = (snd_emu10k1_fx8010_ctl_t *)kmalloc(sizeof(*ctl), GFP_KERNEL);
-                       if (ctl == NULL)
-                               continue;
+                       if (ctl == NULL) {
+                               err = -ENOMEM;
+                               goto __error;
+                       }
                        knew.private_value = (unsigned long)ctl;
-                       memcpy(ctl, &nctl, sizeof(nctl));
+                       *ctl = *nctl;
                        if ((err = snd_ctl_add(emu->card, kctl = snd_ctl_new1(&knew, emu))) < 0) {
                                kfree(ctl);
                                goto __error;
@@ -731,15 +758,17 @@ static int snd_emu10k1_add_controls(emu10k1_t *emu, emu10k1_fx8010_code_t *icode
                        list_add_tail(&ctl->list, &emu->fx8010.gpr_ctl);
                } else {
                        /* overwrite */
-                       nctl.list = ctl->list;
-                       nctl.kcontrol = ctl->kcontrol;
-                       memcpy(ctl, &nctl, sizeof(nctl));
+                       nctl->list = ctl->list;
+                       nctl->kcontrol = ctl->kcontrol;
+                       *ctl = *nctl;
                        snd_ctl_notify(emu->card, SNDRV_CTL_EVENT_MASK_VALUE |
                                                  SNDRV_CTL_EVENT_MASK_INFO, &ctl->kcontrol->id);
                }
                snd_emu10k1_gpr_ctl_put(ctl->kcontrol, val);
        }
       __error:
+       kfree(nctl);
+       kfree(gctl);
        kfree(val);
        return err;
 }
@@ -768,40 +797,47 @@ static int snd_emu10k1_list_controls(emu10k1_t *emu, emu10k1_fx8010_code_t *icod
 {
        unsigned int i = 0, j;
        unsigned int total = 0;
-       emu10k1_fx8010_control_gpr_t gctl;
+       emu10k1_fx8010_control_gpr_t *gctl;
        emu10k1_fx8010_control_gpr_t __user *_gctl;
        snd_emu10k1_fx8010_ctl_t *ctl;
        snd_ctl_elem_id_t *id;
        struct list_head *list;
 
+       gctl = kmalloc(sizeof(*gctl), GFP_KERNEL);
+       if (! gctl)
+               return -ENOMEM;
+
        _gctl = icode->gpr_list_controls;       
        list_for_each(list, &emu->fx8010.gpr_ctl) {
                ctl = emu10k1_gpr_ctl(list);
                total++;
                if (_gctl && i < icode->gpr_list_control_count) {
-                       memset(&gctl, 0, sizeof(gctl));
+                       memset(gctl, 0, sizeof(*gctl));
                        id = &ctl->kcontrol->id;
-                       gctl.id.iface = id->iface;
-                       strlcpy(gctl.id.name, id->name, sizeof(gctl.id.name));
-                       gctl.id.index = id->index;
-                       gctl.id.device = id->device;
-                       gctl.id.subdevice = id->subdevice;
-                       gctl.vcount = ctl->vcount;
-                       gctl.count = ctl->count;
+                       gctl->id.iface = id->iface;
+                       strlcpy(gctl->id.name, id->name, sizeof(gctl->id.name));
+                       gctl->id.index = id->index;
+                       gctl->id.device = id->device;
+                       gctl->id.subdevice = id->subdevice;
+                       gctl->vcount = ctl->vcount;
+                       gctl->count = ctl->count;
                        for (j = 0; j < 32; j++) {
-                               gctl.gpr[j] = ctl->gpr[j];
-                               gctl.value[j] = ctl->value[j];
+                               gctl->gpr[j] = ctl->gpr[j];
+                               gctl->value[j] = ctl->value[j];
                        }
-                       gctl.min = ctl->min;
-                       gctl.max = ctl->max;
-                       gctl.translation = ctl->translation;
-                       if (copy_to_user(_gctl, &gctl, sizeof(gctl)))
+                       gctl->min = ctl->min;
+                       gctl->max = ctl->max;
+                       gctl->translation = ctl->translation;
+                       if (copy_to_user(_gctl, gctl, sizeof(*gctl))) {
+                               kfree(gctl);
                                return -EFAULT;
+                       }
                        _gctl++;
                        i++;
                }
        }
        icode->gpr_list_control_total = total;
+       kfree(gctl);
        return 0;
 }
 
@@ -994,17 +1030,19 @@ static int __devinit _snd_emu10k1_audigy_init_efx(emu10k1_t *emu)
        u32 ptr;
        emu10k1_fx8010_code_t *icode = NULL;
        emu10k1_fx8010_control_gpr_t *controls = NULL, *ctl;
+       u32 *gpr_map;
        mm_segment_t seg;
 
        spin_lock_init(&emu->fx8010.irq_lock);
        INIT_LIST_HEAD(&emu->fx8010.gpr_ctl);
 
        if ((icode = kcalloc(1, sizeof(*icode), GFP_KERNEL)) == NULL ||
-           (icode->gpr_map = kcalloc(512 + 256 + 256 + 2 * 1024, sizeof(u_int32_t), GFP_KERNEL)) == NULL ||
+           (icode->gpr_map = (u_int32_t __user *)kcalloc(512 + 256 + 256 + 2 * 1024, sizeof(u_int32_t), GFP_KERNEL)) == NULL ||
            (controls = kcalloc(SND_EMU10K1_GPR_CONTROLS, sizeof(*controls), GFP_KERNEL)) == NULL) {
                err = -ENOMEM;
                goto __err;
        }
+       gpr_map = (u32 *)icode->gpr_map;
 
        icode->tram_data_map = icode->gpr_map + 512;
        icode->tram_addr_map = icode->tram_data_map + 256;
@@ -1065,10 +1103,10 @@ static int __devinit _snd_emu10k1_audigy_init_efx(emu10k1_t *emu)
        snd_emu10k1_init_stereo_control(&controls[nctl++], "Wave Playback Volume", gpr, 100);
        gpr += 2;
 
-       /* Music Playback */
+       /* Synth Playback */
        A_OP(icode, &ptr, iMAC0, A_GPR(stereo_mix+0), A_GPR(stereo_mix+0), A_GPR(gpr), A_FXBUS(FXBUS_MIDI_LEFT));
        A_OP(icode, &ptr, iMAC0, A_GPR(stereo_mix+1), A_GPR(stereo_mix+1), A_GPR(gpr+1), A_FXBUS(FXBUS_MIDI_RIGHT));
-       snd_emu10k1_init_stereo_control(&controls[nctl++], "Music Playback Volume", gpr, 100);
+       snd_emu10k1_init_stereo_control(&controls[nctl++], "Synth Playback Volume", gpr, 100);
        gpr += 2;
 
        /* Wave (PCM) Capture */
@@ -1077,10 +1115,10 @@ static int __devinit _snd_emu10k1_audigy_init_efx(emu10k1_t *emu)
        snd_emu10k1_init_stereo_control(&controls[nctl++], "PCM Capture Volume", gpr, 0);
        gpr += 2;
 
-       /* Music Capture */
+       /* Synth Capture */
        A_OP(icode, &ptr, iMAC0, A_GPR(capture+0), A_GPR(capture+0), A_GPR(gpr), A_FXBUS(FXBUS_MIDI_LEFT));
        A_OP(icode, &ptr, iMAC0, A_GPR(capture+1), A_GPR(capture+1), A_GPR(gpr+1), A_FXBUS(FXBUS_MIDI_RIGHT));
-       snd_emu10k1_init_stereo_control(&controls[nctl++], "Music Capture Volume", gpr, 0);
+       snd_emu10k1_init_stereo_control(&controls[nctl++], "Synth Capture Volume", gpr, 0);
        gpr += 2;
 
        /*
@@ -1331,8 +1369,9 @@ A_OP(icode, &ptr, iMAC0, A_GPR(var), A_GPR(var), A_GPR(vol), A_EXTIN(input))
        /* A_PUT_STEREO_OUTPUT(A_EXTOUT_FRONT_L, A_EXTOUT_FRONT_R, playback + SND_EMU10K1_PLAYBACK_CHANNELS); */
 
        /* IEC958 Optical Raw Playback Switch */ 
-       icode->gpr_map[gpr++] = 0x1008;
-       icode->gpr_map[gpr++] = 0xffff0000;
+       gpr_map[gpr++] = 0;
+       gpr_map[gpr++] = 0x1008;
+       gpr_map[gpr++] = 0xffff0000;
        for (z = 0; z < 2; z++) {
                A_OP(icode, &ptr, iMAC0, A_GPR(tmp + 2), A_FXBUS(FXBUS_PT_LEFT + z), A_C_00000000, A_C_00000000);
                A_OP(icode, &ptr, iSKIP, A_GPR_COND, A_GPR_COND, A_GPR(gpr - 2), A_C_00000001);
@@ -1341,7 +1380,14 @@ A_OP(icode, &ptr, iMAC0, A_GPR(var), A_GPR(var), A_GPR(vol), A_EXTIN(input))
                A_SWITCH(icode, &ptr, tmp + 0, tmp + 2, gpr + z);
                A_SWITCH_NEG(icode, &ptr, tmp + 1, gpr + z);
                A_SWITCH(icode, &ptr, tmp + 1, playback + SND_EMU10K1_PLAYBACK_CHANNELS + z, tmp + 1);
-               A_OP(icode, &ptr, iACC3, A_EXTOUT(A_EXTOUT_FRONT_L + z), A_GPR(tmp + 0), A_GPR(tmp + 1), A_C_00000000);
+               if ((z==1) && (emu->card_capabilities->spdif_bug)) {
+                       /* Due to a SPDIF output bug on some Audigy cards, this code delays the Right channel by 1 sample */
+                       snd_printk("Installing spdif_bug patch: %s\n", emu->card_capabilities->name);
+                       A_OP(icode, &ptr, iACC3, A_EXTOUT(A_EXTOUT_FRONT_L + z), A_GPR(gpr - 3), A_C_00000000, A_C_00000000);
+                       A_OP(icode, &ptr, iACC3, A_GPR(gpr - 3), A_GPR(tmp + 0), A_GPR(tmp + 1), A_C_00000000);
+               } else {
+                       A_OP(icode, &ptr, iACC3, A_EXTOUT(A_EXTOUT_FRONT_L + z), A_GPR(tmp + 0), A_GPR(tmp + 1), A_C_00000000);
+               }
        }
        snd_emu10k1_init_stereo_onoff_control(controls + nctl++, "IEC958 Optical Raw Playback Switch", gpr, 0);
        gpr += 2;
@@ -1358,6 +1404,11 @@ A_OP(icode, &ptr, iMAC0, A_GPR(var), A_GPR(var), A_GPR(vol), A_EXTIN(input))
        A_PUT_OUTPUT(A_EXTOUT_ADC_CAP_R, capture+1);
 #endif
 
+       /* EFX capture - capture the 16 EXTINs */
+       for (z = 0; z < 16; z++) {
+               A_OP(icode, &ptr, iACC3, A_FXBUS2(z), A_C_00000000, A_C_00000000, A_EXTIN(z));
+       }
+       
        /*
         * ok, set up done..
         */
@@ -1373,16 +1424,14 @@ A_OP(icode, &ptr, iMAC0, A_GPR(var), A_GPR(var), A_GPR(vol), A_EXTIN(input))
 
        seg = snd_enter_user();
        icode->gpr_add_control_count = nctl;
-       icode->gpr_add_controls = controls;
+       icode->gpr_add_controls = (emu10k1_fx8010_control_gpr_t __user *)controls;
        err = snd_emu10k1_icode_poke(emu, icode);
        snd_leave_user(seg);
 
  __err:
-       if (controls != NULL)
-               kfree(controls);
+       kfree(controls);
        if (icode != NULL) {
-               if (icode->gpr_map != NULL)
-                       kfree(icode->gpr_map);
+               kfree((void *)icode->gpr_map);
                kfree(icode);
        }
        return err;
@@ -1448,6 +1497,7 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
        emu10k1_fx8010_code_t *icode;
        emu10k1_fx8010_pcm_t *ipcm = NULL;
        emu10k1_fx8010_control_gpr_t *controls = NULL, *ctl;
+       u32 *gpr_map;
        mm_segment_t seg;
 
        spin_lock_init(&emu->fx8010.irq_lock);
@@ -1455,12 +1505,13 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
 
        if ((icode = kcalloc(1, sizeof(*icode), GFP_KERNEL)) == NULL)
                return -ENOMEM;
-       if ((icode->gpr_map = kcalloc(256 + 160 + 160 + 2 * 512, sizeof(u_int32_t), GFP_KERNEL)) == NULL ||
+       if ((icode->gpr_map = (u_int32_t __user *)kcalloc(256 + 160 + 160 + 2 * 512, sizeof(u_int32_t), GFP_KERNEL)) == NULL ||
             (controls = kcalloc(SND_EMU10K1_GPR_CONTROLS, sizeof(emu10k1_fx8010_control_gpr_t), GFP_KERNEL)) == NULL ||
            (ipcm = kcalloc(1, sizeof(*ipcm), GFP_KERNEL)) == NULL) {
                err = -ENOMEM;
                goto __err;
        }
+       gpr_map = (u32 *)icode->gpr_map;
 
        icode->tram_data_map = icode->gpr_map + 256;
        icode->tram_addr_map = icode->tram_data_map + 160;
@@ -1515,19 +1566,19 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
        ipcm->etram[0] = 0;
        ipcm->etram[1] = 1;
 
-       icode->gpr_map[gpr + 0] = 0xfffff000;
-       icode->gpr_map[gpr + 1] = 0xffff0000;
-       icode->gpr_map[gpr + 2] = 0x70000000;
-       icode->gpr_map[gpr + 3] = 0x00000007;
-       icode->gpr_map[gpr + 4] = 0x001f << 11;
-       icode->gpr_map[gpr + 5] = 0x001c << 11;
-       icode->gpr_map[gpr + 6] = (0x22  - 0x01) - 1;   /* skip at 01 to 22 */
-       icode->gpr_map[gpr + 7] = (0x22  - 0x06) - 1;   /* skip at 06 to 22 */
-       icode->gpr_map[gpr + 8] = 0x2000000 + (2<<11);
-       icode->gpr_map[gpr + 9] = 0x4000000 + (2<<11);
-       icode->gpr_map[gpr + 10] = 1<<11;
-       icode->gpr_map[gpr + 11] = (0x24 - 0x0a) - 1;   /* skip at 0a to 24 */
-       icode->gpr_map[gpr + 12] = 0;
+       gpr_map[gpr + 0] = 0xfffff000;
+       gpr_map[gpr + 1] = 0xffff0000;
+       gpr_map[gpr + 2] = 0x70000000;
+       gpr_map[gpr + 3] = 0x00000007;
+       gpr_map[gpr + 4] = 0x001f << 11;
+       gpr_map[gpr + 5] = 0x001c << 11;
+       gpr_map[gpr + 6] = (0x22  - 0x01) - 1;  /* skip at 01 to 22 */
+       gpr_map[gpr + 7] = (0x22  - 0x06) - 1;  /* skip at 06 to 22 */
+       gpr_map[gpr + 8] = 0x2000000 + (2<<11);
+       gpr_map[gpr + 9] = 0x4000000 + (2<<11);
+       gpr_map[gpr + 10] = 1<<11;
+       gpr_map[gpr + 11] = (0x24 - 0x0a) - 1;  /* skip at 0a to 24 */
+       gpr_map[gpr + 12] = 0;
 
        /* if the trigger flag is not set, skip */
        /* 00: */ OP(icode, &ptr, iMAC0, C_00000000, GPR(ipcm->gpr_trigger), C_00000000, C_00000000);
@@ -1608,19 +1659,19 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
        snd_emu10k1_init_stereo_onoff_control(controls + i++, "Wave Capture Switch", gpr + 2, 0);
        gpr += 4;
 
-       /* Music Playback Volume */
+       /* Synth Playback Volume */
        for (z = 0; z < 2; z++)
                VOLUME_ADD(icode, &ptr, playback + z, 2 + z, gpr + z);
-       snd_emu10k1_init_stereo_control(controls + i++, "Music Playback Volume", gpr, 100);
+       snd_emu10k1_init_stereo_control(controls + i++, "Synth Playback Volume", gpr, 100);
        gpr += 2;
 
-       /* Music Capture Volume + Switch */
+       /* Synth Capture Volume + Switch */
        for (z = 0; z < 2; z++) {
                SWITCH(icode, &ptr, tmp + 0, 2 + z, gpr + 2 + z);
                VOLUME_ADD(icode, &ptr, capture + z, tmp + 0, gpr + z);
        }
-       snd_emu10k1_init_stereo_control(controls + i++, "Music Capture Volume", gpr, 0);
-       snd_emu10k1_init_stereo_onoff_control(controls + i++, "Music Capture Switch", gpr + 2, 0);
+       snd_emu10k1_init_stereo_control(controls + i++, "Synth Capture Volume", gpr, 0);
+       snd_emu10k1_init_stereo_onoff_control(controls + i++, "Synth Capture Switch", gpr + 2, 0);
        gpr += 4;
 
        /* Surround Digital Playback Volume (renamed later without Digital) */
@@ -1930,6 +1981,24 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
        if (emu->fx8010.extout_mask & (1<<EXTOUT_MIC_CAP))
                OP(icode, &ptr, iACC3, EXTOUT(EXTOUT_MIC_CAP), GPR(capture + 2), C_00000000, C_00000000);
 
+       /* EFX capture - capture the 16 EXTINS */
+       OP(icode, &ptr, iACC3, FXBUS2(14), C_00000000, C_00000000, EXTIN(0));
+       OP(icode, &ptr, iACC3, FXBUS2(15), C_00000000, C_00000000, EXTIN(1));
+       OP(icode, &ptr, iACC3, FXBUS2(0), C_00000000, C_00000000, EXTIN(2));
+       OP(icode, &ptr, iACC3, FXBUS2(3), C_00000000, C_00000000, EXTIN(3));
+       /* Dont connect anything to FXBUS2 1 and 2.  These are shared with 
+        * Center/LFE on the SBLive 5.1.  The kX driver only changes the 
+        * routing when it detects an SBLive 5.1.
+        *
+        * Since only 14 of the 16 EXTINs are used, this is not a big problem.  
+        * We route AC97L and R to FX capture 14 and 15, SPDIF CD in to FX capture 
+        * 0 and 3, then the rest of the EXTINs to the corresponding FX capture 
+        * channel.
+        */
+       for (z = 4; z < 14; z++) {
+               OP(icode, &ptr, iACC3, FXBUS2(z), C_00000000, C_00000000, EXTIN(z));
+       }
+
        if (gpr > tmp) {
                snd_BUG();
                err = -EIO;
@@ -1949,19 +2018,16 @@ static int __devinit _snd_emu10k1_init_efx(emu10k1_t *emu)
                goto __err;
        seg = snd_enter_user();
        icode->gpr_add_control_count = i;
-       icode->gpr_add_controls = controls;
+       icode->gpr_add_controls = (emu10k1_fx8010_control_gpr_t __user *)controls;
        err = snd_emu10k1_icode_poke(emu, icode);
        snd_leave_user(seg);
        if (err >= 0)
                err = snd_emu10k1_ipcm_poke(emu, ipcm);
       __err:
-       if (ipcm != NULL)
-               kfree(ipcm);
-       if (controls != NULL)
-               kfree(controls);
+       kfree(ipcm);
+       kfree(controls);
        if (icode != NULL) {
-               if (icode->gpr_map != NULL)
-                       kfree(icode->gpr_map);
+               kfree((void *)icode->gpr_map);
                kfree(icode);
        }
        return err;
@@ -2064,7 +2130,7 @@ static int snd_emu10k1_fx8010_info(emu10k1_t *emu, emu10k1_fx8010_info_t *info)
        memset(info, 0, sizeof(info));
        info->card = emu->card_type;
        info->internal_tram_size = emu->fx8010.itram_size;
-       info->external_tram_size = emu->fx8010.etram_pages.bytes;
+       info->external_tram_size = emu->fx8010.etram_pages.bytes / 2;
        fxbus = fxbuses;
        extin = emu->audigy ? audigy_ins : creative_ins;
        extout = emu->audigy ? audigy_outs : creative_outs;