** Description changed:

  This bug has been around for a while. I have seen it in the forums circa
  2007, and I have provided a solution in 2009. Since the modified code
  never found its way into the mainstream kernels, I am issuing the fix as
  a bug report.
  
  The bug concerns the Soundblaster Audigy cards with the emu10k1 chip in
  both i386 and amd64 architectures, in any kernel (standard, RT etc).
  They have on-board joystick MIDI. There is an optional front-panel
- module, which provides its own MIDI UART. Under Liunux, the front MIDI
- does not work. People wuth dual-boot systems have reported that booting
+ module, which provides its own MIDI UART. Under Linux, the front MIDI
+ does not work. People with dual-boot systems have reported that booting
  Windows first, then rebooting into Linux makes the UART function
  properly.
  
  The cause for the bug lies in the fact that the front panel MIDI is
  routed through a microcontroller (resides in the front panel module),
- and this microcontroller needs a defined reset signal. Up to this point,
- the GPIO _levels_ are initialized correctly, but the reset pulse is
- never issued.
+ and this microcontroller needs a defined reset signal once after power-
+ up. Up to this point, the GPIO _levels_ are initialized correctly, but
+ the reset _pulse_ is never issued. Essentially, the emu10k1 chip's GPIO2
+ needs to be pulled high for a brief interval.
  
  The bug can be fixed in the file linux-
  source-3.13.0/sound/pci/emu10k1/emumpu401.c in the function
  snd_emu10k1_audigy_midi(). The proposed patch follows right after the
- present initialization of the second UART:
- 
+ present initialization of the second UART (note the added variable "val"
+ to store the GPIO bits):
  
  int snd_emu10k1_audigy_midi(struct snd_emu10k1 *emu)
  {
-       struct snd_emu10k1_midi *midi;
-       int err;
-       unsigned int val;               /* Needed in GPOUT2 pulse 
initialization */
+  struct snd_emu10k1_midi *midi;
+  int err;
+  unsigned int val;            /* Needed in GPOUT2 pulse initialization */
  
-       midi = &emu->midi;
-       if ((err = emu10k1_midi_init(emu, midi, 0, "Audigy MPU-401 (UART)")) < 
0)
-               return err;
+  midi = &emu->midi;
+  if ((err = emu10k1_midi_init(emu, midi, 0, "Audigy MPU-401 (UART)")) < 0)
+   return err;
  
-       midi->tx_enable = INTE_MIDITXENABLE;
-       midi->rx_enable = INTE_MIDIRXENABLE;
-       midi->port = A_MUDATA1;
-       midi->ipr_tx = IPR_MIDITRANSBUFEMPTY;
-       midi->ipr_rx = IPR_MIDIRECVBUFEMPTY;
-       midi->interrupt = snd_emu10k1_midi_interrupt;
+  midi->tx_enable = INTE_MIDITXENABLE;
+  midi->rx_enable = INTE_MIDIRXENABLE;
+  midi->port = A_MUDATA1;
+  midi->ipr_tx = IPR_MIDITRANSBUFEMPTY;
+  midi->ipr_rx = IPR_MIDIRECVBUFEMPTY;
+  midi->interrupt = snd_emu10k1_midi_interrupt;
  
-       midi = &emu->midi2;
-       if ((err = emu10k1_midi_init(emu, midi, 1, "Audigy MPU-401 #2")) < 0)
-               return err;
+  midi = &emu->midi2;
+  if ((err = emu10k1_midi_init(emu, midi, 1, "Audigy MPU-401 #2")) < 0)
+   return err;
  
-       midi->tx_enable = INTE_A_MIDITXENABLE2;
-       midi->rx_enable = INTE_A_MIDIRXENABLE2;
-       midi->port = A_MUDATA2;
-       midi->ipr_tx = IPR_A_MIDITRANSBUFEMPTY2;
-       midi->ipr_rx = IPR_A_MIDIRECVBUFEMPTY2;
-       midi->interrupt = snd_emu10k1_midi_interrupt2;
+  midi->tx_enable = INTE_A_MIDITXENABLE2;
+  midi->rx_enable = INTE_A_MIDIRXENABLE2;
+  midi->port = A_MUDATA2;
+  midi->ipr_tx = IPR_A_MIDITRANSBUFEMPTY2;
+  midi->ipr_rx = IPR_A_MIDIRECVBUFEMPTY2;
+  midi->interrupt = snd_emu10k1_midi_interrupt2;
  
-         /*** NEW CODE ***  Pulse reset line for the second UART */
+  /*** NEW CODE ***  Pulse reset line for the second UART */
  
-       val = inl(emu->port + A_IOCFG);
-       outl (val | A_IOCFG_GPOUT2, emu->port + A_IOCFG);
-       udelay(10);             /* udelay is a bad kludge, but remember that 
this is called only once on startup */
-       outl (val, emu->port + A_IOCFG);
+  val = inl(emu->port + A_IOCFG);
+  outl (val | A_IOCFG_GPOUT2, emu->port + A_IOCFG);
+  udelay(10);          /* udelay is a bad kludge, but remember that this is 
called only once on startup */
+  outl (val, emu->port + A_IOCFG);
  
-       return 0;
+  return 0;
  }
  
- 
- I suspect that my use of udelay would make a real kernel hacker cringe, but 
the level of GPOUT2 needs to remain high for a few microseconds to provide a 
stable RESET signal.
+ I suspect that my use of udelay would make a real kernel hacker cringe,
+ but the level of GPOUT2 needs to remain high for a few microseconds to
+ provide a stable RESET signal.
  
  Anyway, this fixes the problem. I have applied the patch for pretty much
  every major upgrade of the kernel. It is sufficient to compile the
  module snd-emu10k1.ko and replace it in the module tree.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1328746

Title:
  Soundblaster Audigy front panel MIDI fails to initialize

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1328746/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to