microblaze: linux-user support.
[qemu] / hw / sb16.c
index b2d20de..218c0ba 100644 (file)
--- a/hw/sb16.c
+++ b/hw/sb16.c
@@ -1,8 +1,8 @@
 /*
  * QEMU Soundblaster 16 emulation
- * 
- * Copyright (c) 2003 Vassili Karpov (malc)
- * 
+ *
+ * Copyright (c) 2003-2005 Vassili Karpov (malc)
+ *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
  * in the Software without restriction, including without limitation the rights
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
-#include "vl.h"
+#include "hw.h"
+#include "audiodev.h"
+#include "audio/audio.h"
+#include "isa.h"
+#include "qemu-timer.h"
 
-#define MIN(a, b) ((a)>(b)?(b):(a))
-#define LENOFA(a) ((int) (sizeof(a)/sizeof(a[0])))
+#define dolog(...) AUD_log ("sb16", __VA_ARGS__)
 
-#define DEREF(x) (void)x
-#define log(...) fprintf (stderr, "sb16: " __VA_ARGS__)
+/* #define DEBUG */
+/* #define DEBUG_SB16_MOST */
 
-/* #define DEBUG_SB16 */
-#ifdef DEBUG_SB16
-#define lwarn(...) fprintf (stderr, "sb16: " __VA_ARGS__)
-#define linfo(...) fprintf (stderr, "sb16: " __VA_ARGS__)
-#define ldebug(...) fprintf (stderr, "sb16: " __VA_ARGS__)
+#ifdef DEBUG
+#define ldebug(...) dolog (__VA_ARGS__)
 #else
-#define lwarn(...)
-#define linfo(...)
 #define ldebug(...)
 #endif
 
-#define IO_READ_PROTO(name) \
+#define IO_READ_PROTO(name)                             \
     uint32_t name (void *opaque, uint32_t nport)
-#define IO_WRITE_PROTO(name) \
+#define IO_WRITE_PROTO(name)                                    \
     void name (void *opaque, uint32_t nport, uint32_t val)
 
+static const char e3[] = "COPYRIGHT (C) CREATIVE TECHNOLOGY LTD, 1992.";
+
 static struct {
     int ver_lo;
     int ver_hi;
@@ -52,480 +52,1081 @@ static struct {
     int dma;
     int hdma;
     int port;
-    int mix_block;
-} sb = {5, 4, 5, 1, 5, 0x220, -1};
-
-static int mix_block, noirq;
+} conf = {5, 4, 5, 1, 5, 0x220};
 
 typedef struct SB16State {
+    QEMUSoundCard card;
+    qemu_irq *pic;
+    int irq;
+    int dma;
+    int hdma;
+    int port;
+    int ver;
+
     int in_index;
     int out_data_len;
     int fmt_stereo;
     int fmt_signed;
     int fmt_bits;
+    audfmt_e fmt;
     int dma_auto;
-    int dma_buffer_size;
+    int block_size;
     int fifo;
     int freq;
     int time_const;
     int speaker;
     int needed_bytes;
     int cmd;
-    int dma_pos;
     int use_hdma;
+    int highspeed;
+    int can_write;
 
     int v2x6;
 
-    uint8_t in_data[10];
-    uint8_t out_data[10];
+    uint8_t csp_param;
+    uint8_t csp_value;
+    uint8_t csp_mode;
+    uint8_t csp_regs[256];
+    uint8_t csp_index;
+    uint8_t csp_reg83[4];
+    int csp_reg83r;
+    int csp_reg83w;
+
+    uint8_t in2_data[10];
+    uint8_t out_data[50];
+    uint8_t test_reg;
+    uint8_t last_read_byte;
+    int nzero;
 
     int left_till_irq;
 
+    int dma_running;
+    int bytes_per_second;
+    int align;
+    int audio_free;
+    SWVoiceOut *voice;
+
+    QEMUTimer *aux_ts;
     /* mixer state */
     int mixer_nreg;
-    uint8_t mixer_regs[0x83];
+    uint8_t mixer_regs[256];
 } SB16State;
 
-/* XXX: suppress that and use a context */
-static struct SB16State dsp;
+static void SB_audio_callback (void *opaque, int free);
+
+static int magic_of_irq (int irq)
+{
+    switch (irq) {
+    case 5:
+        return 2;
+    case 7:
+        return 4;
+    case 9:
+        return 1;
+    case 10:
+        return 8;
+    default:
+        dolog ("bad irq %d\n", irq);
+        return 2;
+    }
+}
+
+static int irq_of_magic (int magic)
+{
+    switch (magic) {
+    case 1:
+        return 9;
+    case 2:
+        return 5;
+    case 4:
+        return 7;
+    case 8:
+        return 10;
+    default:
+        dolog ("bad irq magic %d\n", magic);
+        return -1;
+    }
+}
 
+#if 0
 static void log_dsp (SB16State *dsp)
 {
-    linfo ("%c:%c:%d:%c:dmabuf=%d:pos=%d:freq=%d:timeconst=%d:speaker=%d\n",
-           dsp->fmt_stereo ? 'S' : 'M',
-           dsp->fmt_signed ? 'S' : 'U',
-           dsp->fmt_bits,
-           dsp->dma_auto ? 'a' : 's',
-           dsp->dma_buffer_size,
-           dsp->dma_pos,
-           dsp->freq,
-           dsp->time_const,
-           dsp->speaker);
+    ldebug ("%s:%s:%d:%s:dmasize=%d:freq=%d:const=%d:speaker=%d\n",
+            dsp->fmt_stereo ? "Stereo" : "Mono",
+            dsp->fmt_signed ? "Signed" : "Unsigned",
+            dsp->fmt_bits,
+            dsp->dma_auto ? "Auto" : "Single",
+            dsp->block_size,
+            dsp->freq,
+            dsp->time_const,
+            dsp->speaker);
+}
+#endif
+
+static void speaker (SB16State *s, int on)
+{
+    s->speaker = on;
+    /* AUD_enable (s->voice, on); */
 }
 
-static void control (int hold)
+static void control (SB16State *s, int hold)
 {
-    linfo ("%d high %d\n", hold, dsp.use_hdma);
+    int dma = s->use_hdma ? s->hdma : s->dma;
+    s->dma_running = hold;
+
+    ldebug ("hold %d high %d dma %d\n", hold, s->use_hdma, dma);
+
     if (hold) {
-        if (dsp.use_hdma)
-            DMA_hold_DREQ (sb.hdma);
-        else
-            DMA_hold_DREQ (sb.dma);
+        DMA_hold_DREQ (dma);
+        AUD_set_active_out (s->voice, 1);
     }
     else {
-        if (dsp.use_hdma)
-            DMA_release_DREQ (sb.hdma);
-        else
-            DMA_release_DREQ (sb.dma);
+        DMA_release_DREQ (dma);
+        AUD_set_active_out (s->voice, 0);
     }
 }
 
-static void dma_cmd (uint8_t cmd, uint8_t d0, int dma_len)
+static void aux_timer (void *opaque)
 {
-    int bps;
-    audfmt_e fmt;
+    SB16State *s = opaque;
+    s->can_write = 1;
+    qemu_irq_raise (s->pic[s->irq]);
+}
+
+#define DMA8_AUTO 1
+#define DMA8_HIGH 2
+
+static void continue_dma8 (SB16State *s)
+{
+    if (s->freq > 0) {
+        struct audsettings as;
+
+        s->audio_free = 0;
+
+        as.freq = s->freq;
+        as.nchannels = 1 << s->fmt_stereo;
+        as.fmt = s->fmt;
+        as.endianness = 0;
+
+        s->voice = AUD_open_out (
+            &s->card,
+            s->voice,
+            "sb16",
+            s,
+            SB_audio_callback,
+            &as
+            );
+    }
+
+    control (s, 1);
+}
+
+static void dma_cmd8 (SB16State *s, int mask, int dma_len)
+{
+    s->fmt = AUD_FMT_U8;
+    s->use_hdma = 0;
+    s->fmt_bits = 8;
+    s->fmt_signed = 0;
+    s->fmt_stereo = (s->mixer_regs[0x0e] & 2) != 0;
+    if (-1 == s->time_const) {
+        if (s->freq <= 0)
+            s->freq = 11025;
+    }
+    else {
+        int tmp = (256 - s->time_const);
+        s->freq = (1000000 + (tmp / 2)) / tmp;
+    }
+
+    if (dma_len != -1) {
+        s->block_size = dma_len << s->fmt_stereo;
+    }
+    else {
+        /* This is apparently the only way to make both Act1/PL
+           and SecondReality/FC work
+
+           Act1 sets block size via command 0x48 and it's an odd number
+           SR does the same with even number
+           Both use stereo, and Creatives own documentation states that
+           0x48 sets block size in bytes less one.. go figure */
+        s->block_size &= ~s->fmt_stereo;
+    }
+
+    s->freq >>= s->fmt_stereo;
+    s->left_till_irq = s->block_size;
+    s->bytes_per_second = (s->freq << s->fmt_stereo);
+    /* s->highspeed = (mask & DMA8_HIGH) != 0; */
+    s->dma_auto = (mask & DMA8_AUTO) != 0;
+    s->align = (1 << s->fmt_stereo) - 1;
+
+    if (s->block_size & s->align) {
+        dolog ("warning: misaligned block size %d, alignment %d\n",
+               s->block_size, s->align + 1);
+    }
 
-    dsp.use_hdma = cmd < 0xc0;
-    dsp.fifo = (cmd >> 1) & 1;
-    dsp.dma_auto = (cmd >> 2) & 1;
+    ldebug ("freq %d, stereo %d, sign %d, bits %d, "
+            "dma %d, auto %d, fifo %d, high %d\n",
+            s->freq, s->fmt_stereo, s->fmt_signed, s->fmt_bits,
+            s->block_size, s->dma_auto, s->fifo, s->highspeed);
+
+    continue_dma8 (s);
+    speaker (s, 1);
+}
+
+static void dma_cmd (SB16State *s, uint8_t cmd, uint8_t d0, int dma_len)
+{
+    s->use_hdma = cmd < 0xc0;
+    s->fifo = (cmd >> 1) & 1;
+    s->dma_auto = (cmd >> 2) & 1;
+    s->fmt_signed = (d0 >> 4) & 1;
+    s->fmt_stereo = (d0 >> 5) & 1;
 
     switch (cmd >> 4) {
     case 11:
-        dsp.fmt_bits = 16;
+        s->fmt_bits = 16;
         break;
 
     case 12:
-        dsp.fmt_bits = 8;
+        s->fmt_bits = 8;
         break;
     }
 
-    dsp.fmt_signed = (d0 >> 4) & 1;
-    dsp.fmt_stereo = (d0 >> 5) & 1;
-
-    if (-1 != dsp.time_const) {
-        int tmp;
-
-        tmp = 256 - dsp.time_const;
-        dsp.freq = (1000000 + (tmp / 2)) / tmp;
+    if (-1 != s->time_const) {
+#if 1
+        int tmp = 256 - s->time_const;
+        s->freq = (1000000 + (tmp / 2)) / tmp;
+#else
+        /* s->freq = 1000000 / ((255 - s->time_const) << s->fmt_stereo); */
+        s->freq = 1000000 / ((255 - s->time_const));
+#endif
+        s->time_const = -1;
     }
-    bps = 1 << (16 == dsp.fmt_bits);
 
-    if (-1 != dma_len)
-        dsp.dma_buffer_size = (dma_len + 1) * bps;
+    s->block_size = dma_len + 1;
+    s->block_size <<= (s->fmt_bits == 16);
+    if (!s->dma_auto) {
+        /* It is clear that for DOOM and auto-init this value
+           shouldn't take stereo into account, while Miles Sound Systems
+           setsound.exe with single transfer mode wouldn't work without it
+           wonders of SB16 yet again */
+        s->block_size <<= s->fmt_stereo;
+    }
 
-    linfo ("frequency %d, stereo %d, signed %d, bits %d, size %d, auto %d\n",
-           dsp.freq, dsp.fmt_stereo, dsp.fmt_signed, dsp.fmt_bits,
-           dsp.dma_buffer_size, dsp.dma_auto);
+    ldebug ("freq %d, stereo %d, sign %d, bits %d, "
+            "dma %d, auto %d, fifo %d, high %d\n",
+            s->freq, s->fmt_stereo, s->fmt_signed, s->fmt_bits,
+            s->block_size, s->dma_auto, s->fifo, s->highspeed);
 
-    if (16 == dsp.fmt_bits) {
-        if (dsp.fmt_signed) {
-            fmt = AUD_FMT_S16;
+    if (16 == s->fmt_bits) {
+        if (s->fmt_signed) {
+            s->fmt = AUD_FMT_S16;
         }
         else {
-            fmt = AUD_FMT_U16;
+            s->fmt = AUD_FMT_U16;
         }
     }
     else {
-        if (dsp.fmt_signed) {
-            fmt = AUD_FMT_S8;
+        if (s->fmt_signed) {
+            s->fmt = AUD_FMT_S8;
         }
         else {
-            fmt = AUD_FMT_U8;
+            s->fmt = AUD_FMT_U8;
         }
     }
 
-    dsp.dma_pos = 0;
-    dsp.left_till_irq = dsp.dma_buffer_size;
+    s->left_till_irq = s->block_size;
 
-    if (sb.mix_block) {
-        mix_block = sb.mix_block;
+    s->bytes_per_second = (s->freq << s->fmt_stereo) << (s->fmt_bits == 16);
+    s->highspeed = 0;
+    s->align = (1 << (s->fmt_stereo + (s->fmt_bits == 16))) - 1;
+    if (s->block_size & s->align) {
+        dolog ("warning: misaligned block size %d, alignment %d\n",
+               s->block_size, s->align + 1);
     }
-    else {
-        int align;
 
-        align = bps << dsp.fmt_stereo;
-        mix_block = ((dsp.freq * align) / 100) & ~(align - 1);
+    if (s->freq) {
+        struct audsettings as;
+
+        s->audio_free = 0;
+
+        as.freq = s->freq;
+        as.nchannels = 1 << s->fmt_stereo;
+        as.fmt = s->fmt;
+        as.endianness = 0;
+
+        s->voice = AUD_open_out (
+            &s->card,
+            s->voice,
+            "sb16",
+            s,
+            SB_audio_callback,
+            &as
+            );
     }
 
-    AUD_reset (dsp.freq, 1 << dsp.fmt_stereo, fmt);
-    control (1);
-    dsp.speaker = 1;
+    control (s, 1);
+    speaker (s, 1);
 }
 
-static void command (SB16State *dsp, uint8_t cmd)
+static inline void dsp_out_data (SB16State *s, uint8_t val)
 {
-    linfo ("%#x\n", cmd);
+    ldebug ("outdata %#x\n", val);
+    if ((size_t) s->out_data_len < sizeof (s->out_data)) {
+        s->out_data[s->out_data_len++] = val;
+    }
+}
+
+static inline uint8_t dsp_get_data (SB16State *s)
+{
+    if (s->in_index) {
+        return s->in2_data[--s->in_index];
+    }
+    else {
+        dolog ("buffer underflow\n");
+        return 0;
+    }
+}
+
+static void command (SB16State *s, uint8_t cmd)
+{
+    ldebug ("command %#x\n", cmd);
 
     if (cmd > 0xaf && cmd < 0xd0) {
-        if (cmd & 8)
-            goto error;
+        if (cmd & 8) {
+            dolog ("ADC not yet supported (command %#x)\n", cmd);
+        }
 
         switch (cmd >> 4) {
         case 11:
         case 12:
             break;
         default:
-            log("%#x wrong bits", cmd);
-            goto error;
+            dolog ("%#x wrong bits\n", cmd);
         }
-        dsp->needed_bytes = 3;
+        s->needed_bytes = 3;
     }
     else {
+        s->needed_bytes = 0;
+
         switch (cmd) {
-        case 0x00:
         case 0x03:
-        case 0xe7:
-            /* IMS uses those when probing for sound devices */
-            return;
+            dsp_out_data (s, 0x10); /* s->csp_param); */
+            goto warn;
+
+        case 0x04:
+            s->needed_bytes = 1;
+            goto warn;
+
+        case 0x05:
+            s->needed_bytes = 2;
+            goto warn;
+
+        case 0x08:
+            /* __asm__ ("int3"); */
+            goto warn;
+
+        case 0x0e:
+            s->needed_bytes = 2;
+            goto warn;
+
+        case 0x09:
+            dsp_out_data (s, 0xf8);
+            goto warn;
+
+        case 0x0f:
+            s->needed_bytes = 1;
+            goto warn;
 
         case 0x10:
-            dsp->needed_bytes = 1;
-            break;
+            s->needed_bytes = 1;
+            goto warn;
 
         case 0x14:
-            dsp->needed_bytes = 2;
-            dsp->dma_buffer_size = 0;
+            s->needed_bytes = 2;
+            s->block_size = 0;
             break;
 
-        case 0x20:
-            dsp->out_data[dsp->out_data_len++] = 0xff;
+        case 0x1c:              /* Auto-Initialize DMA DAC, 8-bit */
+            dma_cmd8 (s, DMA8_AUTO, -1);
             break;
 
+        case 0x20:              /* Direct ADC, Juice/PL */
+            dsp_out_data (s, 0xff);
+            goto warn;
+
         case 0x35:
-            lwarn ("MIDI commands not implemented\n");
+            dolog ("0x35 - MIDI command not implemented\n");
             break;
 
         case 0x40:
-            dsp->freq = -1;
-            dsp->time_const = -1;
-            dsp->needed_bytes = 1;
+            s->freq = -1;
+            s->time_const = -1;
+            s->needed_bytes = 1;
             break;
 
         case 0x41:
-        case 0x42:
-            dsp->freq = -1;
-            dsp->time_const = -1;
-            dsp->needed_bytes = 2;
+            s->freq = -1;
+            s->time_const = -1;
+            s->needed_bytes = 2;
             break;
 
+        case 0x42:
+            s->freq = -1;
+            s->time_const = -1;
+            s->needed_bytes = 2;
+            goto warn;
+
+        case 0x45:
+            dsp_out_data (s, 0xaa);
+            goto warn;
+
         case 0x47:                /* Continue Auto-Initialize DMA 16bit */
             break;
 
         case 0x48:
-            dsp->needed_bytes = 2;
+            s->needed_bytes = 2;
             break;
 
-        case 0x27:                /* ????????? */
-        case 0x4e:
-            return;
+        case 0x74:
+            s->needed_bytes = 2; /* DMA DAC, 4-bit ADPCM */
+            dolog ("0x75 - DMA DAC, 4-bit ADPCM not implemented\n");
+            break;
+
+        case 0x75:              /* DMA DAC, 4-bit ADPCM Reference */
+            s->needed_bytes = 2;
+            dolog ("0x74 - DMA DAC, 4-bit ADPCM Reference not implemented\n");
+            break;
+
+        case 0x76:              /* DMA DAC, 2.6-bit ADPCM */
+            s->needed_bytes = 2;
+            dolog ("0x74 - DMA DAC, 2.6-bit ADPCM not implemented\n");
+            break;
+
+        case 0x77:              /* DMA DAC, 2.6-bit ADPCM Reference */
+            s->needed_bytes = 2;
+            dolog ("0x74 - DMA DAC, 2.6-bit ADPCM Reference not implemented\n");
+            break;
+
+        case 0x7d:
+            dolog ("0x7d - Autio-Initialize DMA DAC, 4-bit ADPCM Reference\n");
+            dolog ("not implemented\n");
+            break;
+
+        case 0x7f:
+            dolog (
+                "0x7d - Autio-Initialize DMA DAC, 2.6-bit ADPCM Reference\n"
+                );
+            dolog ("not implemented\n");
+            break;
 
         case 0x80:
-            cmd = -1;
+            s->needed_bytes = 2;
             break;
 
         case 0x90:
         case 0x91:
-            {
-                uint8_t d0;
-
-                d0 = 4;
-                if (dsp->fmt_signed) d0 |= 16;
-                if (dsp->fmt_stereo) d0 |= 32;
-                dma_cmd (cmd == 0x90 ? 0xc4 : 0xc0, d0, -1);
-                cmd = -1;
-                break;
-            }
+            dma_cmd8 (s, ((cmd & 1) == 0) | DMA8_HIGH, -1);
+            break;
 
-        case 0xd0:                /* XXX */
-            control (0);
-            return;
+        case 0xd0:              /* halt DMA operation. 8bit */
+            control (s, 0);
+            break;
 
-        case 0xd1:
-            dsp->speaker = 1;
+        case 0xd1:              /* speaker on */
+            speaker (s, 1);
             break;
 
-        case 0xd3:
-            dsp->speaker = 0;
-            return;
+        case 0xd3:              /* speaker off */
+            speaker (s, 0);
+            break;
 
-        case 0xd4:
-            control (1);
+        case 0xd4:              /* continue DMA operation. 8bit */
+            /* KQ6 (or maybe Sierras audblst.drv in general) resets
+               the frequency between halt/continue */
+            continue_dma8 (s);
             break;
 
-        case 0xd5:
-            control (0);
+        case 0xd5:              /* halt DMA operation. 16bit */
+            control (s, 0);
             break;
 
-        case 0xd6:
-            control (1);
+        case 0xd6:              /* continue DMA operation. 16bit */
+            control (s, 1);
             break;
 
-        case 0xd9:
-            control (0);
-            dsp->dma_auto = 0;
-            return;
+        case 0xd9:              /* exit auto-init DMA after this block. 16bit */
+            s->dma_auto = 0;
+            break;
 
-        case 0xda:
-            control (0);
-            dsp->dma_auto = 0;
+        case 0xda:              /* exit auto-init DMA after this block. 8bit */
+            s->dma_auto = 0;
             break;
 
-        case 0xe0:
-            dsp->needed_bytes = 1;
+        case 0xe0:              /* DSP identification */
+            s->needed_bytes = 1;
             break;
 
         case 0xe1:
-            dsp->out_data[dsp->out_data_len++] = sb.ver_lo;
-            dsp->out_data[dsp->out_data_len++] = sb.ver_hi;
-            return;
+            dsp_out_data (s, s->ver & 0xff);
+            dsp_out_data (s, s->ver >> 8);
+            break;
+
+        case 0xe2:
+            s->needed_bytes = 1;
+            goto warn;
+
+        case 0xe3:
+            {
+                int i;
+                for (i = sizeof (e3) - 1; i >= 0; --i)
+                    dsp_out_data (s, e3[i]);
+            }
+            break;
+
+        case 0xe4:              /* write test reg */
+            s->needed_bytes = 1;
+            break;
+
+        case 0xe7:
+            dolog ("Attempt to probe for ESS (0xe7)?\n");
+            break;
+
+        case 0xe8:              /* read test reg */
+            dsp_out_data (s, s->test_reg);
+            break;
 
         case 0xf2:
-            dsp->out_data[dsp->out_data_len++] = 0xaa;
-            dsp->mixer_regs[0x82] |= dsp->mixer_regs[0x80];
-            pic_set_irq (sb.irq, 1);
-            return;
+        case 0xf3:
+            dsp_out_data (s, 0xaa);
+            s->mixer_regs[0x82] |= (cmd == 0xf2) ? 1 : 2;
+            qemu_irq_raise (s->pic[s->irq]);
+            break;
+
+        case 0xf9:
+            s->needed_bytes = 1;
+            goto warn;
+
+        case 0xfa:
+            dsp_out_data (s, 0);
+            goto warn;
+
+        case 0xfc:              /* FIXME */
+            dsp_out_data (s, 0);
+            goto warn;
 
         default:
-            log("%#x is unknown", cmd);
-            goto error;
+            dolog ("Unrecognized command %#x\n", cmd);
+            break;
         }
     }
-    dsp->cmd = cmd;
-    return;
 
- error:
+    if (!s->needed_bytes) {
+        ldebug ("\n");
+    }
+
+ exit:
+    if (!s->needed_bytes) {
+        s->cmd = -1;
+    }
+    else {
+        s->cmd = cmd;
+    }
     return;
+
+ warn:
+    dolog ("warning: command %#x,%d is not truly understood yet\n",
+           cmd, s->needed_bytes);
+    goto exit;
+
 }
 
-static void complete (SB16State *dsp)
+static uint16_t dsp_get_lohi (SB16State *s)
 {
-    linfo ("complete command %#x, in_index %d, needed_bytes %d\n",
-           dsp->cmd, dsp->in_index, dsp->needed_bytes);
-
-    if (dsp->cmd > 0xaf && dsp->cmd < 0xd0) {
-        int d0, d1, d2;
+    uint8_t hi = dsp_get_data (s);
+    uint8_t lo = dsp_get_data (s);
+    return (hi << 8) | lo;
+}
 
-        d0 = dsp->in_data[0];
-        d1 = dsp->in_data[1];
-        d2 = dsp->in_data[2];
+static uint16_t dsp_get_hilo (SB16State *s)
+{
+    uint8_t lo = dsp_get_data (s);
+    uint8_t hi = dsp_get_data (s);
+    return (hi << 8) | lo;
+}
 
-        ldebug ("d0 = %d, d1 = %d, d2 = %d\n",
-                d0, d1, d2);
-        dma_cmd (dsp->cmd, d0, d1 + (d2 << 8));
+static void complete (SB16State *s)
+{
+    int d0, d1, d2;
+    ldebug ("complete command %#x, in_index %d, needed_bytes %d\n",
+            s->cmd, s->in_index, s->needed_bytes);
+
+    if (s->cmd > 0xaf && s->cmd < 0xd0) {
+        d2 = dsp_get_data (s);
+        d1 = dsp_get_data (s);
+        d0 = dsp_get_data (s);
+
+        if (s->cmd & 8) {
+            dolog ("ADC params cmd = %#x d0 = %d, d1 = %d, d2 = %d\n",
+                   s->cmd, d0, d1, d2);
+        }
+        else {
+            ldebug ("cmd = %#x d0 = %d, d1 = %d, d2 = %d\n",
+                    s->cmd, d0, d1, d2);
+            dma_cmd (s, s->cmd, d0, d1 + (d2 << 8));
+        }
     }
     else {
-        switch (dsp->cmd) {
+        switch (s->cmd) {
+        case 0x04:
+            s->csp_mode = dsp_get_data (s);
+            s->csp_reg83r = 0;
+            s->csp_reg83w = 0;
+            ldebug ("CSP command 0x04: mode=%#x\n", s->csp_mode);
+            break;
 
-        case 0x10:
+        case 0x05:
+            s->csp_param = dsp_get_data (s);
+            s->csp_value = dsp_get_data (s);
+            ldebug ("CSP command 0x05: param=%#x value=%#x\n",
+                    s->csp_param,
+                    s->csp_value);
             break;
 
-        case 0x14:
-            {
-                int d0, d1;
-                int save_left;
-                int save_pos;
+        case 0x0e:
+            d0 = dsp_get_data (s);
+            d1 = dsp_get_data (s);
+            ldebug ("write CSP register %d <- %#x\n", d1, d0);
+            if (d1 == 0x83) {
+                ldebug ("0x83[%d] <- %#x\n", s->csp_reg83r, d0);
+                s->csp_reg83[s->csp_reg83r % 4] = d0;
+                s->csp_reg83r += 1;
+            }
+            else {
+                s->csp_regs[d1] = d0;
+            }
+            break;
 
-                d0 = dsp->in_data[0];
-                d1 = dsp->in_data[1];
+        case 0x0f:
+            d0 = dsp_get_data (s);
+            ldebug ("read CSP register %#x -> %#x, mode=%#x\n",
+                    d0, s->csp_regs[d0], s->csp_mode);
+            if (d0 == 0x83) {
+                ldebug ("0x83[%d] -> %#x\n",
+                        s->csp_reg83w,
+                        s->csp_reg83[s->csp_reg83w % 4]);
+                dsp_out_data (s, s->csp_reg83[s->csp_reg83w % 4]);
+                s->csp_reg83w += 1;
+            }
+            else {
+                dsp_out_data (s, s->csp_regs[d0]);
+            }
+            break;
 
-                save_left = dsp->left_till_irq;
-                save_pos = dsp->dma_pos;
-                dma_cmd (0xc0, 0, d0 + (d1 << 8));
-                dsp->left_till_irq = save_left;
-                dsp->dma_pos = save_pos;
+        case 0x10:
+            d0 = dsp_get_data (s);
+            dolog ("cmd 0x10 d0=%#x\n", d0);
+            break;
 
-                linfo ("set buffer size data[%d, %d] %d pos %d\n",
-                       d0, d1, dsp->dma_buffer_size, dsp->dma_pos);
-                break;
-            }
+        case 0x14:
+            dma_cmd8 (s, 0, dsp_get_lohi (s) + 1);
+            break;
 
         case 0x40:
-            dsp->time_const = dsp->in_data[0];
-            linfo ("set time const %d\n", dsp->time_const);
+            s->time_const = dsp_get_data (s);
+            ldebug ("set time const %d\n", s->time_const);
             break;
 
+        case 0x42:              /* FT2 sets output freq with this, go figure */
+#if 0
+            dolog ("cmd 0x42 might not do what it think it should\n");
+#endif
         case 0x41:
-        case 0x42:
-            dsp->freq = dsp->in_data[1] + (dsp->in_data[0] << 8);
-            linfo ("set freq %#x, %#x = %d\n",
-                   dsp->in_data[1], dsp->in_data[0], dsp->freq);
+            s->freq = dsp_get_hilo (s);
+            ldebug ("set freq %d\n", s->freq);
             break;
 
         case 0x48:
-            dsp->dma_buffer_size = dsp->in_data[1] + (dsp->in_data[0] << 8);
-            linfo ("set dma len %#x, %#x = %d\n",
-                   dsp->in_data[1], dsp->in_data[0], dsp->dma_buffer_size);
+            s->block_size = dsp_get_lohi (s) + 1;
+            ldebug ("set dma block len %d\n", s->block_size);
+            break;
+
+        case 0x74:
+        case 0x75:
+        case 0x76:
+        case 0x77:
+            /* ADPCM stuff, ignore */
+            break;
+
+        case 0x80:
+            {
+                int freq, samples, bytes;
+                int64_t ticks;
+
+                freq = s->freq > 0 ? s->freq : 11025;
+                samples = dsp_get_lohi (s) + 1;
+                bytes = samples << s->fmt_stereo << (s->fmt_bits == 16);
+                ticks = (bytes * ticks_per_sec) / freq;
+                if (ticks < ticks_per_sec / 1024) {
+                    qemu_irq_raise (s->pic[s->irq]);
+                }
+                else {
+                    if (s->aux_ts) {
+                        qemu_mod_timer (
+                            s->aux_ts,
+                            qemu_get_clock (vm_clock) + ticks
+                            );
+                    }
+                }
+                ldebug ("mix silence %d %d %" PRId64 "\n", samples, bytes, ticks);
+            }
             break;
 
         case 0xe0:
-            dsp->out_data_len = 1;
-            linfo ("data = %#x\n", dsp->in_data[0]);
-            dsp->out_data[0] = dsp->in_data[0] ^ 0xff;
+            d0 = dsp_get_data (s);
+            s->out_data_len = 0;
+            ldebug ("E0 data = %#x\n", d0);
+            dsp_out_data (s, ~d0);
+            break;
+
+        case 0xe2:
+            d0 = dsp_get_data (s);
+            ldebug ("E2 = %#x\n", d0);
+            break;
+
+        case 0xe4:
+            s->test_reg = dsp_get_data (s);
+            break;
+
+        case 0xf9:
+            d0 = dsp_get_data (s);
+            ldebug ("command 0xf9 with %#x\n", d0);
+            switch (d0) {
+            case 0x0e:
+                dsp_out_data (s, 0xff);
+                break;
+
+            case 0x0f:
+                dsp_out_data (s, 0x07);
+                break;
+
+            case 0x37:
+                dsp_out_data (s, 0x38);
+                break;
+
+            default:
+                dsp_out_data (s, 0x00);
+                break;
+            }
             break;
 
         default:
-            log ("unrecognized command %#x", dsp->cmd);
+            dolog ("complete: unrecognized command %#x\n", s->cmd);
             return;
         }
     }
 
-    dsp->cmd = -1;
+    ldebug ("\n");
+    s->cmd = -1;
     return;
 }
 
+static void legacy_reset (SB16State *s)
+{
+    struct audsettings as;
+
+    s->freq = 11025;
+    s->fmt_signed = 0;
+    s->fmt_bits = 8;
+    s->fmt_stereo = 0;
+
+    as.freq = s->freq;
+    as.nchannels = 1;
+    as.fmt = AUD_FMT_U8;
+    as.endianness = 0;
+
+    s->voice = AUD_open_out (
+        &s->card,
+        s->voice,
+        "sb16",
+        s,
+        SB_audio_callback,
+        &as
+        );
+
+    /* Not sure about that... */
+    /* AUD_set_active_out (s->voice, 1); */
+}
+
+static void reset (SB16State *s)
+{
+    qemu_irq_lower (s->pic[s->irq]);
+    if (s->dma_auto) {
+        qemu_irq_raise (s->pic[s->irq]);
+        qemu_irq_lower (s->pic[s->irq]);
+    }
+
+    s->mixer_regs[0x82] = 0;
+    s->dma_auto = 0;
+    s->in_index = 0;
+    s->out_data_len = 0;
+    s->left_till_irq = 0;
+    s->needed_bytes = 0;
+    s->block_size = -1;
+    s->nzero = 0;
+    s->highspeed = 0;
+    s->v2x6 = 0;
+    s->cmd = -1;
+
+    dsp_out_data(s, 0xaa);
+    speaker (s, 0);
+    control (s, 0);
+    legacy_reset (s);
+}
+
 static IO_WRITE_PROTO (dsp_write)
 {
-    SB16State *dsp = opaque;
+    SB16State *s = opaque;
     int iport;
 
-    iport = nport - sb.port;
+    iport = nport - s->port;
 
+    ldebug ("write %#x <- %#x\n", nport, val);
     switch (iport) {
-    case 0x6:
-        if (0 == val)
-            dsp->v2x6 = 0;
-        else if ((1 == val) && (0 == dsp->v2x6)) {
-            dsp->v2x6 = 1;
-            dsp->out_data[dsp->out_data_len++] = 0xaa;
+    case 0x06:
+        switch (val) {
+        case 0x00:
+            if (s->v2x6 == 1) {
+                if (0 && s->highspeed) {
+                    s->highspeed = 0;
+                    qemu_irq_lower (s->pic[s->irq]);
+                    control (s, 0);
+                }
+                else {
+                    reset (s);
+                }
+            }
+            s->v2x6 = 0;
+            break;
+
+        case 0x01:
+        case 0x03:              /* FreeBSD kludge */
+            s->v2x6 = 1;
+            break;
+
+        case 0xc6:
+            s->v2x6 = 0;        /* Prince of Persia, csp.sys, diagnose.exe */
+            break;
+
+        case 0xb8:              /* Panic */
+            reset (s);
+            break;
+
+        case 0x39:
+            dsp_out_data (s, 0x38);
+            reset (s);
+            s->v2x6 = 0x39;
+            break;
+
+        default:
+            s->v2x6 = val;
+            break;
         }
-        else
-            dsp->v2x6 = ~0;
         break;
 
-    case 0xc:                   /* write data or command | write status */
-        if (0 == dsp->needed_bytes) {
-            command (dsp, val);
-            if (0 == dsp->needed_bytes) {
-                log_dsp (dsp);
+    case 0x0c:                  /* write data or command | write status */
+/*         if (s->highspeed) */
+/*             break; */
+
+        if (0 == s->needed_bytes) {
+            command (s, val);
+#if 0
+            if (0 == s->needed_bytes) {
+                log_dsp (s);
             }
+#endif
         }
         else {
-            dsp->in_data[dsp->in_index++] = val;
-            if (dsp->in_index == dsp->needed_bytes) {
-                dsp->needed_bytes = 0;
-                dsp->in_index = 0;
-                complete (dsp);
-                log_dsp (dsp);
+            if (s->in_index == sizeof (s->in2_data)) {
+                dolog ("in data overrun\n");
+            }
+            else {
+                s->in2_data[s->in_index++] = val;
+                if (s->in_index == s->needed_bytes) {
+                    s->needed_bytes = 0;
+                    complete (s);
+#if 0
+                    log_dsp (s);
+#endif
+                }
             }
         }
         break;
 
     default:
-        log ("(nport=%#x, val=%#x)", nport, val);
+        ldebug ("(nport=%#x, val=%#x)\n", nport, val);
         break;
     }
 }
 
 static IO_READ_PROTO (dsp_read)
 {
-    SB16State *dsp = opaque;
-    int iport, retval;
+    SB16State *s = opaque;
+    int iport, retval, ack = 0;
 
-    iport = nport - sb.port;
+    iport = nport - s->port;
 
     switch (iport) {
+    case 0x06:                  /* reset */
+        retval = 0xff;
+        break;
 
-    case 0x6:                   /* reset */
-        return 0;
-
-    case 0xa:                   /* read data */
-        if (dsp->out_data_len) {
-            retval = dsp->out_data[--dsp->out_data_len];
-        } else {
-            log("empty output buffer\n");
-            goto error;
+    case 0x0a:                  /* read data */
+        if (s->out_data_len) {
+            retval = s->out_data[--s->out_data_len];
+            s->last_read_byte = retval;
+        }
+        else {
+            if (s->cmd != -1) {
+                dolog ("empty output buffer for command %#x\n",
+                       s->cmd);
+            }
+            retval = s->last_read_byte;
+            /* goto error; */
         }
         break;
 
-    case 0xc:                   /* 0 can write */
-        retval = 0;
+    case 0x0c:                  /* 0 can write */
+        retval = s->can_write ? 0 : 0x80;
         break;
 
-    case 0xd:                   /* timer interrupt clear */
-        log("timer interrupt clear\n");
-        goto error;
+    case 0x0d:                  /* timer interrupt clear */
+        /* dolog ("timer interrupt clear\n"); */
+        retval = 0;
+        break;
 
-    case 0xe:                   /* data available status | irq 8 ack */
-        /* XXX drop pic irq line here? */
-        ldebug ("8 ack\n");
-        retval = (0 == dsp->out_data_len) ? 0 : 0x80;
-        dsp->mixer_regs[0x82] &= ~dsp->mixer_regs[0x80];
-        pic_set_irq (sb.irq, 0);
+    case 0x0e:                  /* data available status | irq 8 ack */
+        retval = (!s->out_data_len || s->highspeed) ? 0 : 0x80;
+        if (s->mixer_regs[0x82] & 1) {
+            ack = 1;
+            s->mixer_regs[0x82] &= 1;
+            qemu_irq_lower (s->pic[s->irq]);
+        }
         break;
 
-    case 0xf:                   /* irq 16 ack */
-        /* XXX drop pic irq line here? */
-        ldebug ("16 ack\n");
+    case 0x0f:                  /* irq 16 ack */
         retval = 0xff;
-        dsp->mixer_regs[0x82] &= ~dsp->mixer_regs[0x80];
-        pic_set_irq (sb.irq, 0);
+        if (s->mixer_regs[0x82] & 2) {
+            ack = 1;
+            s->mixer_regs[0x82] &= 2;
+            qemu_irq_lower (s->pic[s->irq]);
+        }
         break;
 
     default:
         goto error;
     }
 
-    if ((0xc != iport) && (0xe != iport)) {
-        ldebug ("nport=%#x iport %#x = %#x\n",
-                nport, iport, retval);
+    if (!ack) {
+        ldebug ("read %#x -> %#x\n", nport, retval);
     }
 
     return retval;
 
  error:
-    return 0;
+    dolog ("warning: dsp_read %#x error\n", nport);
+    return 0xff;
+}
+
+static void reset_mixer (SB16State *s)
+{
+    int i;
+
+    memset (s->mixer_regs, 0xff, 0x7f);
+    memset (s->mixer_regs + 0x83, 0xff, sizeof (s->mixer_regs) - 0x83);
+
+    s->mixer_regs[0x02] = 4;    /* master volume 3bits */
+    s->mixer_regs[0x06] = 4;    /* MIDI volume 3bits */
+    s->mixer_regs[0x08] = 0;    /* CD volume 3bits */
+    s->mixer_regs[0x0a] = 0;    /* voice volume 2bits */
+
+    /* d5=input filt, d3=lowpass filt, d1,d2=input source */
+    s->mixer_regs[0x0c] = 0;
+
+    /* d5=output filt, d1=stereo switch */
+    s->mixer_regs[0x0e] = 0;
+
+    /* voice volume L d5,d7, R d1,d3 */
+    s->mixer_regs[0x04] = (4 << 5) | (4 << 1);
+    /* master ... */
+    s->mixer_regs[0x22] = (4 << 5) | (4 << 1);
+    /* MIDI ... */
+    s->mixer_regs[0x26] = (4 << 5) | (4 << 1);
+
+    for (i = 0x30; i < 0x48; i++) {
+        s->mixer_regs[i] = 0x20;
+    }
 }
 
 static IO_WRITE_PROTO(mixer_write_indexb)
 {
-    SB16State *dsp = opaque;
-    dsp->mixer_nreg = val & 0xff;
+    SB16State *s = opaque;
+    (void) nport;
+    s->mixer_nreg = val;
 }
 
 static IO_WRITE_PROTO(mixer_write_datab)
 {
-    SB16State *dsp = opaque;
-    dsp->mixer_regs[dsp->mixer_nreg] = val;
+    SB16State *s = opaque;
+
+    (void) nport;
+    ldebug ("mixer_write [%#x] <- %#x\n", s->mixer_nreg, val);
+
+    switch (s->mixer_nreg) {
+    case 0x00:
+        reset_mixer (s);
+        break;
+
+    case 0x80:
+        {
+            int irq = irq_of_magic (val);
+            ldebug ("setting irq to %d (val=%#x)\n", irq, val);
+            if (irq > 0) {
+                s->irq = irq;
+            }
+        }
+        break;
+
+    case 0x81:
+        {
+            int dma, hdma;
+
+            dma = lsbindex (val & 0xf);
+            hdma = lsbindex (val & 0xf0);
+            if (dma != s->dma || hdma != s->hdma) {
+                dolog (
+                    "attempt to change DMA "
+                    "8bit %d(%d), 16bit %d(%d) (val=%#x)\n",
+                    dma, s->dma, hdma, s->hdma, val);
+            }
+#if 0
+            s->dma = dma;
+            s->hdma = hdma;
+#endif
+        }
+        break;
+
+    case 0x82:
+        dolog ("attempt to write into IRQ status register (val=%#x)\n",
+               val);
+        return;
+
+    default:
+        if (s->mixer_nreg >= 0x80) {
+            ldebug ("attempt to write mixer[%#x] <- %#x\n", s->mixer_nreg, val);
+        }
+        break;
+    }
+
+    s->mixer_regs[s->mixer_nreg] = val;
 }
 
 static IO_WRITE_PROTO(mixer_write_indexw)
@@ -536,183 +1137,315 @@ static IO_WRITE_PROTO(mixer_write_indexw)
 
 static IO_READ_PROTO(mixer_read)
 {
-    SB16State *dsp = opaque;
-    return dsp->mixer_regs[dsp->mixer_nreg];
-}
+    SB16State *s = opaque;
 
-void SB16_run (void)
-{
-    if (0 == dsp.speaker)
-        return;
-
-    AUD_run ();
+    (void) nport;
+#ifndef DEBUG_SB16_MOST
+    if (s->mixer_nreg != 0x82) {
+        ldebug ("mixer_read[%#x] -> %#x\n",
+                s->mixer_nreg, s->mixer_regs[s->mixer_nreg]);
+    }
+#else
+    ldebug ("mixer_read[%#x] -> %#x\n",
+            s->mixer_nreg, s->mixer_regs[s->mixer_nreg]);
+#endif
+    return s->mixer_regs[s->mixer_nreg];
 }
 
-static int write_audio (uint32_t addr, int len, int size)
+static int write_audio (SB16State *s, int nchan, int dma_pos,
+                        int dma_len, int len)
 {
     int temp, net;
     uint8_t tmpbuf[4096];
 
-    temp = size;
-
+    temp = len;
     net = 0;
 
     while (temp) {
-        int left_till_end;
-        int to_copy;
+        int left = dma_len - dma_pos;
         int copied;
+        size_t to_copy;
 
-        left_till_end = len - dsp.dma_pos;
+        to_copy = audio_MIN (temp, left);
+        if (to_copy > sizeof (tmpbuf)) {
+            to_copy = sizeof (tmpbuf);
+        }
 
-        to_copy = MIN (temp, left_till_end);
-        if (to_copy > sizeof(tmpbuf))
-            to_copy = sizeof(tmpbuf);
-        cpu_physical_memory_read(addr + dsp.dma_pos, tmpbuf, to_copy);
-        copied = AUD_write (tmpbuf, to_copy);
+        copied = DMA_read_memory (nchan, tmpbuf, dma_pos, to_copy);
+        copied = AUD_write (s->voice, tmpbuf, copied);
 
         temp -= copied;
-        dsp.dma_pos += copied;
-
-        if (dsp.dma_pos == len) {
-            dsp.dma_pos = 0;
-        }
-
+        dma_pos = (dma_pos + copied) % dma_len;
         net += copied;
 
-        if (copied != to_copy)
-            return net;
+        if (!copied) {
+            break;
+        }
     }
 
     return net;
 }
 
-static int SB_read_DMA (void *opaque, target_ulong addr, int size)
+static int SB_read_DMA (void *opaque, int nchan, int dma_pos, int dma_len)
 {
-    SB16State *dsp = opaque;
-    int free, till, copy, written;
+    SB16State *s = opaque;
+    int till, copy, written, free;
 
-    if (0 == dsp->speaker)
-        return 0;
-
-    if (dsp->left_till_irq < 0) {
-        dsp->left_till_irq += dsp->dma_buffer_size;
-        return dsp->dma_pos;
+    if (s->block_size <= 0) {
+        dolog ("invalid block size=%d nchan=%d dma_pos=%d dma_len=%d\n",
+               s->block_size, nchan, dma_pos, dma_len);
+        return dma_pos;
     }
 
-    free = AUD_get_free ();
-
-    if ((free <= 0) || (0 == size)) {
-        return dsp->dma_pos;
+    if (s->left_till_irq < 0) {
+        s->left_till_irq = s->block_size;
     }
 
-    if (mix_block > 0) {
-        copy = MIN (free, mix_block);
+    if (s->voice) {
+        free = s->audio_free & ~s->align;
+        if ((free <= 0) || !dma_len) {
+            return dma_pos;
+        }
     }
     else {
-        copy = free;
+        free = dma_len;
     }
 
-    till = dsp->left_till_irq;
+    copy = free;
+    till = s->left_till_irq;
+
+#ifdef DEBUG_SB16_MOST
+    dolog ("pos:%06d %d till:%d len:%d\n",
+           dma_pos, free, till, dma_len);
+#endif
 
-    ldebug ("addr:%#010x free:%d till:%d size:%d\n",
-            addr, free, till, size);
     if (till <= copy) {
-        if (0 == dsp->dma_auto) {
+        if (0 == s->dma_auto) {
             copy = till;
         }
     }
 
-    written = write_audio (addr, size, copy);
-    dsp->left_till_irq -= written;
-    AUD_adjust_estimate (free - written);
-
-    if (dsp->left_till_irq <= 0) {
-        dsp->mixer_regs[0x82] |= dsp->mixer_regs[0x80];
-        if (0 == noirq) {
-            ldebug ("request irq\n");
-            pic_set_irq(sb.irq, 1);
-        }
+    written = write_audio (s, nchan, dma_pos, dma_len, copy);
+    dma_pos = (dma_pos + written) % dma_len;
+    s->left_till_irq -= written;
 
-        if (0 == dsp->dma_auto) {
-            control (0);
+    if (s->left_till_irq <= 0) {
+        s->mixer_regs[0x82] |= (nchan & 4) ? 2 : 1;
+        qemu_irq_raise (s->pic[s->irq]);
+        if (0 == s->dma_auto) {
+            control (s, 0);
+            speaker (s, 0);
         }
     }
 
-    ldebug ("pos %5d free %5d size %5d till % 5d copy %5d dma size %5d\n",
-            dsp->dma_pos, free, size, dsp->left_till_irq, copy,
-            dsp->dma_buffer_size);
+#ifdef DEBUG_SB16_MOST
+    ldebug ("pos %5d free %5d size %5d till % 5d copy %5d written %5d size %5d\n",
+            dma_pos, free, dma_len, s->left_till_irq, copy, written,
+            s->block_size);
+#endif
 
-    if (dsp->left_till_irq <= 0) {
-        dsp->left_till_irq += dsp->dma_buffer_size;
+    while (s->left_till_irq <= 0) {
+        s->left_till_irq = s->block_size + s->left_till_irq;
     }
 
-    return dsp->dma_pos;
+    return dma_pos;
 }
 
-static int magic_of_irq (int irq)
+static void SB_audio_callback (void *opaque, int free)
 {
-    switch (irq) {
-    case 2:
-        return 1;
-    case 5:
-        return 2;
-    case 7:
-        return 4;
-    case 10:
-        return 8;
-    default:
-        log ("bad irq %d\n", irq);
-        return 2;
-    }
+    SB16State *s = opaque;
+    s->audio_free = free;
 }
 
-static int irq_of_magic (int magic)
+static void SB_save (QEMUFile *f, void *opaque)
 {
-    switch (magic) {
-    case 1:
-        return 2;
-    case 2:
-        return 5;
-    case 4:
-        return 7;
-    case 8:
-        return 10;
-    default:
-        log ("bad irq magic %d\n", magic);
-        return 2;
+    SB16State *s = opaque;
+
+    qemu_put_be32 (f, s->irq);
+    qemu_put_be32 (f, s->dma);
+    qemu_put_be32 (f, s->hdma);
+    qemu_put_be32 (f, s->port);
+    qemu_put_be32 (f, s->ver);
+    qemu_put_be32 (f, s->in_index);
+    qemu_put_be32 (f, s->out_data_len);
+    qemu_put_be32 (f, s->fmt_stereo);
+    qemu_put_be32 (f, s->fmt_signed);
+    qemu_put_be32 (f, s->fmt_bits);
+    qemu_put_be32s (f, &s->fmt);
+    qemu_put_be32 (f, s->dma_auto);
+    qemu_put_be32 (f, s->block_size);
+    qemu_put_be32 (f, s->fifo);
+    qemu_put_be32 (f, s->freq);
+    qemu_put_be32 (f, s->time_const);
+    qemu_put_be32 (f, s->speaker);
+    qemu_put_be32 (f, s->needed_bytes);
+    qemu_put_be32 (f, s->cmd);
+    qemu_put_be32 (f, s->use_hdma);
+    qemu_put_be32 (f, s->highspeed);
+    qemu_put_be32 (f, s->can_write);
+    qemu_put_be32 (f, s->v2x6);
+
+    qemu_put_8s (f, &s->csp_param);
+    qemu_put_8s (f, &s->csp_value);
+    qemu_put_8s (f, &s->csp_mode);
+    qemu_put_8s (f, &s->csp_param);
+    qemu_put_buffer (f, s->csp_regs, 256);
+    qemu_put_8s (f, &s->csp_index);
+    qemu_put_buffer (f, s->csp_reg83, 4);
+    qemu_put_be32 (f, s->csp_reg83r);
+    qemu_put_be32 (f, s->csp_reg83w);
+
+    qemu_put_buffer (f, s->in2_data, sizeof (s->in2_data));
+    qemu_put_buffer (f, s->out_data, sizeof (s->out_data));
+    qemu_put_8s (f, &s->test_reg);
+    qemu_put_8s (f, &s->last_read_byte);
+
+    qemu_put_be32 (f, s->nzero);
+    qemu_put_be32 (f, s->left_till_irq);
+    qemu_put_be32 (f, s->dma_running);
+    qemu_put_be32 (f, s->bytes_per_second);
+    qemu_put_be32 (f, s->align);
+
+    qemu_put_be32 (f, s->mixer_nreg);
+    qemu_put_buffer (f, s->mixer_regs, 256);
+}
+
+static int SB_load (QEMUFile *f, void *opaque, int version_id)
+{
+    SB16State *s = opaque;
+
+    if (version_id != 1) {
+        return -EINVAL;
     }
+
+    s->irq=qemu_get_be32 (f);
+    s->dma=qemu_get_be32 (f);
+    s->hdma=qemu_get_be32 (f);
+    s->port=qemu_get_be32 (f);
+    s->ver=qemu_get_be32 (f);
+    s->in_index=qemu_get_be32 (f);
+    s->out_data_len=qemu_get_be32 (f);
+    s->fmt_stereo=qemu_get_be32 (f);
+    s->fmt_signed=qemu_get_be32 (f);
+    s->fmt_bits=qemu_get_be32 (f);
+    qemu_get_be32s (f, &s->fmt);
+    s->dma_auto=qemu_get_be32 (f);
+    s->block_size=qemu_get_be32 (f);
+    s->fifo=qemu_get_be32 (f);
+    s->freq=qemu_get_be32 (f);
+    s->time_const=qemu_get_be32 (f);
+    s->speaker=qemu_get_be32 (f);
+    s->needed_bytes=qemu_get_be32 (f);
+    s->cmd=qemu_get_be32 (f);
+    s->use_hdma=qemu_get_be32 (f);
+    s->highspeed=qemu_get_be32 (f);
+    s->can_write=qemu_get_be32 (f);
+    s->v2x6=qemu_get_be32 (f);
+
+    qemu_get_8s (f, &s->csp_param);
+    qemu_get_8s (f, &s->csp_value);
+    qemu_get_8s (f, &s->csp_mode);
+    qemu_get_8s (f, &s->csp_param);
+    qemu_get_buffer (f, s->csp_regs, 256);
+    qemu_get_8s (f, &s->csp_index);
+    qemu_get_buffer (f, s->csp_reg83, 4);
+    s->csp_reg83r=qemu_get_be32 (f);
+    s->csp_reg83w=qemu_get_be32 (f);
+
+    qemu_get_buffer (f, s->in2_data, sizeof (s->in2_data));
+    qemu_get_buffer (f, s->out_data, sizeof (s->out_data));
+    qemu_get_8s (f, &s->test_reg);
+    qemu_get_8s (f, &s->last_read_byte);
+
+    s->nzero=qemu_get_be32 (f);
+    s->left_till_irq=qemu_get_be32 (f);
+    s->dma_running=qemu_get_be32 (f);
+    s->bytes_per_second=qemu_get_be32 (f);
+    s->align=qemu_get_be32 (f);
+
+    s->mixer_nreg=qemu_get_be32 (f);
+    qemu_get_buffer (f, s->mixer_regs, 256);
+
+    if (s->voice) {
+        AUD_close_out (&s->card, s->voice);
+        s->voice = NULL;
+    }
+
+    if (s->dma_running) {
+        if (s->freq) {
+            struct audsettings as;
+
+            s->audio_free = 0;
+
+            as.freq = s->freq;
+            as.nchannels = 1 << s->fmt_stereo;
+            as.fmt = s->fmt;
+            as.endianness = 0;
+
+            s->voice = AUD_open_out (
+                &s->card,
+                s->voice,
+                "sb16",
+                s,
+                SB_audio_callback,
+                &as
+                );
+        }
+
+        control (s, 1);
+        speaker (s, s->speaker);
+    }
+    return 0;
 }
 
-void SB16_init (void)
+int SB16_init (qemu_irq *pic)
 {
-    SB16State *s = &dsp;
+    SB16State *s;
     int i;
     static const uint8_t dsp_write_ports[] = {0x6, 0xc};
     static const uint8_t dsp_read_ports[] = {0x6, 0xa, 0xc, 0xd, 0xe, 0xf};
 
-    s->mixer_regs[0x0e] = ~0;
-    s->mixer_regs[0x80] = magic_of_irq (sb.irq);
-    s->mixer_regs[0x81] = 0x20 | (sb.dma << 1);
+    s = qemu_mallocz (sizeof (*s));
 
-    DEREF (irq_of_magic);
+    s->cmd = -1;
+    s->pic = pic;
+    s->irq = conf.irq;
+    s->dma = conf.dma;
+    s->hdma = conf.hdma;
+    s->port = conf.port;
+    s->ver = conf.ver_lo | (conf.ver_hi << 8);
 
-    for (i = 0x30; i < 0x48; i++) {
-        s->mixer_regs[i] = 0x20;
+    s->mixer_regs[0x80] = magic_of_irq (s->irq);
+    s->mixer_regs[0x81] = (1 << s->dma) | (1 << s->hdma);
+    s->mixer_regs[0x82] = 2 << 5;
+
+    s->csp_regs[5] = 1;
+    s->csp_regs[9] = 0xf8;
+
+    reset_mixer (s);
+    s->aux_ts = qemu_new_timer (vm_clock, aux_timer, s);
+    if (!s->aux_ts) {
+        dolog ("warning: Could not create auxiliary timer\n");
     }
 
-    for (i = 0; i < LENOFA (dsp_write_ports); i++) {
-        register_ioport_write (sb.port + dsp_write_ports[i], 1, 1, dsp_write, s);
+    for (i = 0; i < ARRAY_SIZE (dsp_write_ports); i++) {
+        register_ioport_write (s->port + dsp_write_ports[i], 1, 1, dsp_write, s);
     }
 
-    for (i = 0; i < LENOFA (dsp_read_ports); i++) {
-        register_ioport_read (sb.port + dsp_read_ports[i], 1, 1, dsp_read, s);
+    for (i = 0; i < ARRAY_SIZE (dsp_read_ports); i++) {
+        register_ioport_read (s->port + dsp_read_ports[i], 1, 1, dsp_read, s);
     }
 
-    register_ioport_write (sb.port + 0x4, 1, 1, mixer_write_indexb, s);
-    register_ioport_write (sb.port + 0x4, 1, 2, mixer_write_indexw, s);
-    register_ioport_read (sb.port + 0x5, 1, 1, mixer_read, s);
-    register_ioport_write (sb.port + 0x5, 1, 1, mixer_write_datab, s);
+    register_ioport_write (s->port + 0x4, 1, 1, mixer_write_indexb, s);
+    register_ioport_write (s->port + 0x4, 1, 2, mixer_write_indexw, s);
+    register_ioport_read (s->port + 0x5, 1, 1, mixer_read, s);
+    register_ioport_write (s->port + 0x5, 1, 1, mixer_write_datab, s);
 
-    DMA_register_channel (sb.hdma, SB_read_DMA, s);
-    DMA_register_channel (sb.dma, SB_read_DMA, s);
+    DMA_register_channel (s->hdma, SB_read_DMA, s);
+    DMA_register_channel (s->dma, SB_read_DMA, s);
+    s->can_write = 1;
+
+    register_savevm ("sb16", 0, 1, SB_save, SB_load, s);
+    AUD_register_card ("sb16", &s->card);
+    return 0;
 }