Merge to Fedora kernel-2.6.17-1.2187_FC5 patched with stable patch-2.6.17.13-vs2...
[linux-2.6.git] / sound / core / timer.c
index 35c5d05..4585600 100644 (file)
 #include <sound/driver.h>
 #include <linux/delay.h>
 #include <linux/init.h>
+#include <linux/smp_lock.h>
 #include <linux/slab.h>
 #include <linux/time.h>
+#include <linux/mutex.h>
+#include <linux/moduleparam.h>
+#include <linux/string.h>
 #include <sound/core.h>
 #include <sound/timer.h>
 #include <sound/control.h>
 #include <linux/kerneld.h>
 #endif
 
-#if !defined(CONFIG_SND_RTCTIMER) && !defined(CONFIG_SND_RTCTIMER_MODULE)
-#define DEFAULT_TIMER_LIMIT 1
-#else
+#if defined(CONFIG_SND_HPET) || defined(CONFIG_SND_HPET_MODULE)
+#define DEFAULT_TIMER_LIMIT 3
+#elif defined(CONFIG_SND_RTCTIMER) || defined(CONFIG_SND_RTCTIMER_MODULE)
 #define DEFAULT_TIMER_LIMIT 2
+#else
+#define DEFAULT_TIMER_LIMIT 1
 #endif
 
 static int timer_limit = DEFAULT_TIMER_LIMIT;
 MODULE_AUTHOR("Jaroslav Kysela <perex@suse.cz>, Takashi Iwai <tiwai@suse.de>");
 MODULE_DESCRIPTION("ALSA timer interface");
 MODULE_LICENSE("GPL");
-MODULE_CLASSES("{sound}");
-MODULE_PARM(timer_limit, "i");
+module_param(timer_limit, int, 0444);
 MODULE_PARM_DESC(timer_limit, "Maximum global timers in system.");
 
-typedef struct {
-       snd_timer_instance_t *timeri;
-       int tread;                      /* enhanced read with timestamps and events */
+struct snd_timer_user {
+       struct snd_timer_instance *timeri;
+       int tread;              /* enhanced read with timestamps and events */
        unsigned long ticks;
        unsigned long overrun;
        int qhead;
        int qtail;
        int qused;
        int queue_size;
-       snd_timer_read_t *queue;
-       snd_timer_tread_t *tqueue;
+       struct snd_timer_read *queue;
+       struct snd_timer_tread *tqueue;
        spinlock_t qlock;
        unsigned long last_resolution;
        unsigned int filter;
        struct timespec tstamp;         /* trigger tstamp */
        wait_queue_head_t qchange_sleep;
        struct fasync_struct *fasync;
-} snd_timer_user_t;
+       struct mutex tread_sem;
+};
 
 /* list of timers */
 static LIST_HEAD(snd_timer_list);
@@ -75,28 +81,29 @@ static LIST_HEAD(snd_timer_list);
 static LIST_HEAD(snd_timer_slave_list);
 
 /* lock for slave active lists */
-static spinlock_t slave_active_lock = SPIN_LOCK_UNLOCKED;
+static DEFINE_SPINLOCK(slave_active_lock);
 
-static DECLARE_MUTEX(register_mutex);
+static DEFINE_MUTEX(register_mutex);
 
-static int snd_timer_free(snd_timer_t *timer);
-static int snd_timer_dev_free(snd_device_t *device);
-static int snd_timer_dev_register(snd_device_t *device);
-static int snd_timer_dev_unregister(snd_device_t *device);
+static int snd_timer_free(struct snd_timer *timer);
+static int snd_timer_dev_free(struct snd_device *device);
+static int snd_timer_dev_register(struct snd_device *device);
+static int snd_timer_dev_unregister(struct snd_device *device);
 
-static void snd_timer_reschedule(snd_timer_t * timer, unsigned long ticks_left);
+static void snd_timer_reschedule(struct snd_timer * timer, unsigned long ticks_left);
 
 /*
  * create a timer instance with the given owner string.
  * when timer is not NULL, increments the module counter
  */
-static snd_timer_instance_t *snd_timer_instance_new(char *owner, snd_timer_t *timer)
+static struct snd_timer_instance *snd_timer_instance_new(char *owner,
+                                                        struct snd_timer *timer)
 {
-       snd_timer_instance_t *timeri;
-       timeri = snd_kcalloc(sizeof(snd_timer_instance_t), GFP_KERNEL);
+       struct snd_timer_instance *timeri;
+       timeri = kzalloc(sizeof(*timeri), GFP_KERNEL);
        if (timeri == NULL)
                return NULL;
-       timeri->owner = snd_kmalloc_strdup(owner, GFP_KERNEL);
+       timeri->owner = kstrdup(owner, GFP_KERNEL);
        if (! timeri->owner) {
                kfree(timeri);
                return NULL;
@@ -108,7 +115,7 @@ static snd_timer_instance_t *snd_timer_instance_new(char *owner, snd_timer_t *ti
        INIT_LIST_HEAD(&timeri->slave_active_head);
 
        timeri->timer = timer;
-       if (timer && timer->card && !try_module_get(timer->card->module)) {
+       if (timer && !try_module_get(timer->module)) {
                kfree(timeri->owner);
                kfree(timeri);
                return NULL;
@@ -120,13 +127,13 @@ static snd_timer_instance_t *snd_timer_instance_new(char *owner, snd_timer_t *ti
 /*
  * find a timer instance from the given timer id
  */
-static snd_timer_t *snd_timer_find(snd_timer_id_t *tid)
+static struct snd_timer *snd_timer_find(struct snd_timer_id *tid)
 {
-       snd_timer_t *timer = NULL;
+       struct snd_timer *timer = NULL;
        struct list_head *p;
 
        list_for_each(p, &snd_timer_list) {
-               timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+               timer = list_entry(p, struct snd_timer, device_list);
 
                if (timer->tmr_class != tid->dev_class)
                        continue;
@@ -146,7 +153,7 @@ static snd_timer_t *snd_timer_find(snd_timer_id_t *tid)
 
 #ifdef CONFIG_KMOD
 
-static void snd_timer_request(snd_timer_id_t *tid)
+static void snd_timer_request(struct snd_timer_id *tid)
 {
        if (! current->fs->root)
                return;
@@ -173,21 +180,22 @@ static void snd_timer_request(snd_timer_id_t *tid)
  *
  * call this with register_mutex down.
  */
-static void snd_timer_check_slave(snd_timer_instance_t *slave)
+static void snd_timer_check_slave(struct snd_timer_instance *slave)
 {
-       snd_timer_t *timer;
-       snd_timer_instance_t *master;
+       struct snd_timer *timer;
+       struct snd_timer_instance *master;
        struct list_head *p, *q;
 
        /* FIXME: it's really dumb to look up all entries.. */
        list_for_each(p, &snd_timer_list) {
-               timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+               timer = list_entry(p, struct snd_timer, device_list);
                list_for_each(q, &timer->open_list_head) {
-                       master = (snd_timer_instance_t *)list_entry(q, snd_timer_instance_t, open_list);
+                       master = list_entry(q, struct snd_timer_instance, open_list);
                        if (slave->slave_class == master->slave_class &&
                            slave->slave_id == master->slave_id) {
                                list_del(&slave->open_list);
-                               list_add_tail(&slave->open_list, &master->slave_list_head);
+                               list_add_tail(&slave->open_list,
+                                             &master->slave_list_head);
                                spin_lock_irq(&slave_active_lock);
                                slave->master = master;
                                slave->timer = master->timer;
@@ -204,14 +212,14 @@ static void snd_timer_check_slave(snd_timer_instance_t *slave)
  *
  * call this with register_mutex down.
  */
-static void snd_timer_check_master(snd_timer_instance_t *master)
+static void snd_timer_check_master(struct snd_timer_instance *master)
 {
-       snd_timer_instance_t *slave;
+       struct snd_timer_instance *slave;
        struct list_head *p, *n;
 
        /* check all pending slaves */
        list_for_each_safe(p, n, &snd_timer_slave_list) {
-               slave = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, open_list);
+               slave = list_entry(p, struct snd_timer_instance, open_list);
                if (slave->slave_class == master->slave_class &&
                    slave->slave_id == master->slave_id) {
                        list_del(p);
@@ -220,7 +228,8 @@ static void snd_timer_check_master(snd_timer_instance_t *master)
                        slave->master = master;
                        slave->timer = master->timer;
                        if (slave->flags & SNDRV_TIMER_IFLG_RUNNING)
-                               list_add_tail(&slave->active_list, &master->slave_active_head);
+                               list_add_tail(&slave->active_list,
+                                             &master->slave_active_head);
                        spin_unlock_irq(&slave_active_lock);
                }
        }
@@ -230,13 +239,13 @@ static void snd_timer_check_master(snd_timer_instance_t *master)
  * open a timer instance
  * when opening a master, the slave id must be here given.
  */
-int snd_timer_open(snd_timer_instance_t **ti,
-                  char *owner, snd_timer_id_t *tid,
+int snd_timer_open(struct snd_timer_instance **ti,
+                  char *owner, struct snd_timer_id *tid,
                   unsigned int slave_id)
 {
-       snd_timer_t *timer;
-       snd_timer_instance_t *timeri = NULL;
-       
+       struct snd_timer *timer;
+       struct snd_timer_instance *timeri = NULL;
+
        if (tid->dev_class == SNDRV_TIMER_CLASS_SLAVE) {
                /* open a slave instance */
                if (tid->dev_sclass <= SNDRV_TIMER_SCLASS_NONE ||
@@ -244,65 +253,72 @@ int snd_timer_open(snd_timer_instance_t **ti,
                        snd_printd("invalid slave class %i\n", tid->dev_sclass);
                        return -EINVAL;
                }
-               down(&register_mutex);
+               mutex_lock(&register_mutex);
                timeri = snd_timer_instance_new(owner, NULL);
+               if (!timeri) {
+                       mutex_unlock(&register_mutex);
+                       return -ENOMEM;
+               }
                timeri->slave_class = tid->dev_sclass;
                timeri->slave_id = tid->device;
                timeri->flags |= SNDRV_TIMER_IFLG_SLAVE;
                list_add_tail(&timeri->open_list, &snd_timer_slave_list);
                snd_timer_check_slave(timeri);
-               up(&register_mutex);
+               mutex_unlock(&register_mutex);
                *ti = timeri;
                return 0;
        }
 
        /* open a master instance */
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        timer = snd_timer_find(tid);
 #ifdef CONFIG_KMOD
        if (timer == NULL) {
-               up(&register_mutex);
+               mutex_unlock(&register_mutex);
                snd_timer_request(tid);
-               down(&register_mutex);
+               mutex_lock(&register_mutex);
                timer = snd_timer_find(tid);
        }
 #endif
-       if (timer) {
-               if (!list_empty(&timer->open_list_head)) {
-                       timeri = (snd_timer_instance_t *)list_entry(timer->open_list_head.next, snd_timer_instance_t, open_list);
-                       if (timeri->flags & SNDRV_TIMER_IFLG_EXCLUSIVE) {
-                               up(&register_mutex);
-                               return -EBUSY;
-                       }
-               }
-               timeri = snd_timer_instance_new(owner, timer);
-               if (timeri) {
-                       timeri->slave_class = tid->dev_sclass;
-                       timeri->slave_id = slave_id;
-                       if (list_empty(&timer->open_list_head) && timer->hw.open)
-                               timer->hw.open(timer);
-                       list_add_tail(&timeri->open_list, &timer->open_list_head);
-                       snd_timer_check_master(timeri);
-               }
-       } else {
-               up(&register_mutex);
+       if (!timer) {
+               mutex_unlock(&register_mutex);
                return -ENODEV;
        }
-       up(&register_mutex);
+       if (!list_empty(&timer->open_list_head)) {
+               timeri = list_entry(timer->open_list_head.next,
+                                   struct snd_timer_instance, open_list);
+               if (timeri->flags & SNDRV_TIMER_IFLG_EXCLUSIVE) {
+                       mutex_unlock(&register_mutex);
+                       return -EBUSY;
+               }
+       }
+       timeri = snd_timer_instance_new(owner, timer);
+       if (!timeri) {
+               mutex_unlock(&register_mutex);
+               return -ENOMEM;
+       }
+       timeri->slave_class = tid->dev_sclass;
+       timeri->slave_id = slave_id;
+       if (list_empty(&timer->open_list_head) && timer->hw.open)
+               timer->hw.open(timer);
+       list_add_tail(&timeri->open_list, &timer->open_list_head);
+       snd_timer_check_master(timeri);
+       mutex_unlock(&register_mutex);
        *ti = timeri;
        return 0;
 }
 
-static int _snd_timer_stop(snd_timer_instance_t * timeri, int keep_flag, enum sndrv_timer_event event);
+static int _snd_timer_stop(struct snd_timer_instance *timeri,
+                          int keep_flag, int event);
 
 /*
  * close a timer instance
  */
-int snd_timer_close(snd_timer_instance_t * timeri)
+int snd_timer_close(struct snd_timer_instance *timeri)
 {
-       snd_timer_t *timer = NULL;
+       struct snd_timer *timer = NULL;
        struct list_head *p, *n;
-       snd_timer_instance_t *slave;
+       struct snd_timer_instance *slave;
 
        snd_assert(timeri != NULL, return -ENXIO);
 
@@ -318,9 +334,9 @@ int snd_timer_close(snd_timer_instance_t * timeri)
                        spin_lock_irq(&slave_active_lock);
                }
                spin_unlock_irq(&slave_active_lock);
-               down(&register_mutex);
+               mutex_lock(&register_mutex);
                list_del(&timeri->open_list);
-               up(&register_mutex);
+               mutex_unlock(&register_mutex);
        } else {
                timer = timeri->timer;
                /* wait, until the active callback is finished */
@@ -331,13 +347,14 @@ int snd_timer_close(snd_timer_instance_t * timeri)
                        spin_lock_irq(&timer->lock);
                }
                spin_unlock_irq(&timer->lock);
-               down(&register_mutex);
+               mutex_lock(&register_mutex);
                list_del(&timeri->open_list);
-               if (timer && list_empty(&timer->open_list_head) && timer->hw.close)
+               if (timer && list_empty(&timer->open_list_head) &&
+                   timer->hw.close)
                        timer->hw.close(timer);
                /* remove slave links */
                list_for_each_safe(p, n, &timeri->slave_list_head) {
-                       slave = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, open_list);
+                       slave = list_entry(p, struct snd_timer_instance, open_list);
                        spin_lock_irq(&slave_active_lock);
                        _snd_timer_stop(slave, 1, SNDRV_TIMER_EVENT_RESOLUTION);
                        list_del(p);
@@ -346,21 +363,20 @@ int snd_timer_close(snd_timer_instance_t * timeri)
                        slave->timer = NULL;
                        spin_unlock_irq(&slave_active_lock);
                }
-               up(&register_mutex);
+               mutex_unlock(&register_mutex);
        }
        if (timeri->private_free)
                timeri->private_free(timeri);
-       if (timeri->owner)
-               kfree(timeri->owner);
+       kfree(timeri->owner);
        kfree(timeri);
-       if (timer && timer->card)
-               module_put(timer->card->module);
+       if (timer)
+               module_put(timer->module);
        return 0;
 }
 
-unsigned long snd_timer_resolution(snd_timer_instance_t * timeri)
+unsigned long snd_timer_resolution(struct snd_timer_instance *timeri)
 {
-       snd_timer_t * timer;
+       struct snd_timer * timer;
 
        if (timeri == NULL)
                return 0;
@@ -372,18 +388,20 @@ unsigned long snd_timer_resolution(snd_timer_instance_t * timeri)
        return 0;
 }
 
-static void snd_timer_notify1(snd_timer_instance_t *ti, enum sndrv_timer_event event)
+static void snd_timer_notify1(struct snd_timer_instance *ti, int event)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        unsigned long flags;
        unsigned long resolution = 0;
-       snd_timer_instance_t *ts;
+       struct snd_timer_instance *ts;
        struct list_head *n;
        struct timespec tstamp;
 
-       snd_timestamp_now(&tstamp, 1);
-       snd_assert(event >= SNDRV_TIMER_EVENT_START && event <= SNDRV_TIMER_EVENT_PAUSE, return);
-       if (event == SNDRV_TIMER_EVENT_START || event == SNDRV_TIMER_EVENT_CONTINUE)
+       getnstimeofday(&tstamp);
+       snd_assert(event >= SNDRV_TIMER_EVENT_START &&
+                  event <= SNDRV_TIMER_EVENT_PAUSE, return);
+       if (event == SNDRV_TIMER_EVENT_START ||
+           event == SNDRV_TIMER_EVENT_CONTINUE)
                resolution = snd_timer_resolution(ti);
        if (ti->ccallback)
                ti->ccallback(ti, SNDRV_TIMER_EVENT_START, &tstamp, resolution);
@@ -396,14 +414,15 @@ static void snd_timer_notify1(snd_timer_instance_t *ti, enum sndrv_timer_event e
                return;
        spin_lock_irqsave(&timer->lock, flags);
        list_for_each(n, &ti->slave_active_head) {
-               ts = (snd_timer_instance_t *)list_entry(n, snd_timer_instance_t, active_list);
+               ts = list_entry(n, struct snd_timer_instance, active_list);
                if (ts->ccallback)
                        ts->ccallback(ti, event + 100, &tstamp, resolution);
        }
        spin_unlock_irqrestore(&timer->lock, flags);
 }
 
-static int snd_timer_start1(snd_timer_t *timer, snd_timer_instance_t *timeri, unsigned long sticks)
+static int snd_timer_start1(struct snd_timer *timer, struct snd_timer_instance *timeri,
+                           unsigned long sticks)
 {
        list_del(&timeri->active_list);
        list_add_tail(&timeri->active_list, &timer->active_list_head);
@@ -423,24 +442,25 @@ static int snd_timer_start1(snd_timer_t *timer, snd_timer_instance_t *timeri, un
        }
 }
 
-static int snd_timer_start_slave(snd_timer_instance_t *timeri)
+static int snd_timer_start_slave(struct snd_timer_instance *timeri)
 {
        unsigned long flags;
 
        spin_lock_irqsave(&slave_active_lock, flags);
        timeri->flags |= SNDRV_TIMER_IFLG_RUNNING;
        if (timeri->master)
-               list_add_tail(&timeri->active_list, &timeri->master->slave_active_head);
+               list_add_tail(&timeri->active_list,
+                             &timeri->master->slave_active_head);
        spin_unlock_irqrestore(&slave_active_lock, flags);
        return 1; /* delayed start */
 }
 
 /*
  *  start the timer instance
- */ 
-int snd_timer_start(snd_timer_instance_t * timeri, unsigned int ticks)
+ */
+int snd_timer_start(struct snd_timer_instance *timeri, unsigned int ticks)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        int result = -EINVAL;
        unsigned long flags;
 
@@ -463,9 +483,10 @@ int snd_timer_start(snd_timer_instance_t * timeri, unsigned int ticks)
        return result;
 }
 
-static int _snd_timer_stop(snd_timer_instance_t * timeri, int keep_flag, enum sndrv_timer_event event)
+static int _snd_timer_stop(struct snd_timer_instance * timeri,
+                          int keep_flag, int event)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        unsigned long flags;
 
        snd_assert(timeri != NULL, return -ENXIO);
@@ -497,7 +518,8 @@ static int _snd_timer_stop(snd_timer_instance_t * timeri, int keep_flag, enum sn
                }
        }
        if (!keep_flag)
-               timeri->flags &= ~(SNDRV_TIMER_IFLG_RUNNING|SNDRV_TIMER_IFLG_START);
+               timeri->flags &=
+                       ~(SNDRV_TIMER_IFLG_RUNNING | SNDRV_TIMER_IFLG_START);
        spin_unlock_irqrestore(&timer->lock, flags);
       __end:
        if (event != SNDRV_TIMER_EVENT_RESOLUTION)
@@ -510,9 +532,9 @@ static int _snd_timer_stop(snd_timer_instance_t * timeri, int keep_flag, enum sn
  *
  * do not call this from the timer callback!
  */
-int snd_timer_stop(snd_timer_instance_t * timeri)
+int snd_timer_stop(struct snd_timer_instance *timeri)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        unsigned long flags;
        int err;
 
@@ -530,9 +552,9 @@ int snd_timer_stop(snd_timer_instance_t * timeri)
 /*
  * start again..  the tick is kept.
  */
-int snd_timer_continue(snd_timer_instance_t * timeri)
+int snd_timer_continue(struct snd_timer_instance *timeri)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        int result = -EINVAL;
        unsigned long flags;
 
@@ -556,7 +578,7 @@ int snd_timer_continue(snd_timer_instance_t * timeri)
 /*
  * pause.. remember the ticks left
  */
-int snd_timer_pause(snd_timer_instance_t * timeri)
+int snd_timer_pause(struct snd_timer_instance * timeri)
 {
        return _snd_timer_stop(timeri, 0, SNDRV_TIMER_EVENT_PAUSE);
 }
@@ -567,14 +589,14 @@ int snd_timer_pause(snd_timer_instance_t * timeri)
  * start pending instances and check the scheduling ticks.
  * when the scheduling ticks is changed set CHANGE flag to reprogram the timer.
  */
-static void snd_timer_reschedule(snd_timer_t * timer, unsigned long ticks_left)
+static void snd_timer_reschedule(struct snd_timer * timer, unsigned long ticks_left)
 {
-       snd_timer_instance_t *ti;
+       struct snd_timer_instance *ti;
        unsigned long ticks = ~0UL;
        struct list_head *p;
 
        list_for_each(p, &timer->active_list_head) {
-               ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, active_list);
+               ti = list_entry(p, struct snd_timer_instance, active_list);
                if (ti->flags & SNDRV_TIMER_IFLG_START) {
                        ti->flags &= ~SNDRV_TIMER_IFLG_START;
                        ti->flags |= SNDRV_TIMER_IFLG_RUNNING;
@@ -602,20 +624,21 @@ static void snd_timer_reschedule(snd_timer_t * timer, unsigned long ticks_left)
  */
 static void snd_timer_tasklet(unsigned long arg)
 {
-       snd_timer_t *timer = (snd_timer_t *) arg;
-       snd_timer_instance_t *ti;
+       struct snd_timer *timer = (struct snd_timer *) arg;
+       struct snd_timer_instance *ti;
        struct list_head *p;
        unsigned long resolution, ticks;
+       unsigned long flags;
 
-       spin_lock(&timer->lock);
+       spin_lock_irqsave(&timer->lock, flags);
        /* now process all callbacks */
        while (!list_empty(&timer->sack_list_head)) {
                p = timer->sack_list_head.next;         /* get first item */
-               ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, ack_list);
+               ti = list_entry(p, struct snd_timer_instance, ack_list);
 
                /* remove from ack_list and make empty */
                list_del_init(p);
-               
+
                ticks = ti->pticks;
                ti->pticks = 0;
                resolution = ti->resolution;
@@ -627,7 +650,7 @@ static void snd_timer_tasklet(unsigned long arg)
                spin_lock(&timer->lock);
                ti->flags &= ~SNDRV_TIMER_IFLG_CALLBACK;
        }
-       spin_unlock(&timer->lock);
+       spin_unlock_irqrestore(&timer->lock, flags);
 }
 
 /*
@@ -636,17 +659,18 @@ static void snd_timer_tasklet(unsigned long arg)
  * ticks_left is usually equal to timer->sticks.
  *
  */
-void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
+void snd_timer_interrupt(struct snd_timer * timer, unsigned long ticks_left)
 {
-       snd_timer_instance_t *ti, *ts;
+       struct snd_timer_instance *ti, *ts;
        unsigned long resolution, ticks;
-       struct list_head *p, *q, *n;
+       struct list_head *p, *q, *n, *ack_list_head;
+       unsigned long flags;
        int use_tasklet = 0;
 
        if (timer == NULL)
                return;
 
-       spin_lock(&timer->lock);
+       spin_lock_irqsave(&timer->lock, flags);
 
        /* remember the current resolution */
        if (timer->hw.c_resolution)
@@ -655,11 +679,12 @@ void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
                resolution = timer->hw.resolution;
 
        /* loop for all active instances
-        * here we cannot use list_for_each because the active_list of a processed
-        * instance is relinked to done_list_head before callback is called.
+        * Here we cannot use list_for_each because the active_list of a
+        * processed instance is relinked to done_list_head before the callback
+        * is called.
         */
        list_for_each_safe(p, n, &timer->active_list_head) {
-               ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, active_list);
+               ti = list_entry(p, struct snd_timer_instance, active_list);
                if (!(ti->flags & SNDRV_TIMER_IFLG_RUNNING))
                        continue;
                ti->pticks += ticks_left;
@@ -677,26 +702,19 @@ void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
                        if (--timer->running)
                                list_del(p);
                }
-               if (list_empty(&ti->ack_list)) {
-                       if ((timer->hw.flags & SNDRV_TIMER_HW_TASKLET) ||
-                           (ti->flags & SNDRV_TIMER_IFLG_FAST)) {
-                               list_add_tail(&ti->ack_list, &timer->ack_list_head);
-                       } else {
-                               list_add_tail(&ti->ack_list, &timer->sack_list_head);
-                       }
-               }
+               if ((timer->hw.flags & SNDRV_TIMER_HW_TASKLET) ||
+                   (ti->flags & SNDRV_TIMER_IFLG_FAST))
+                       ack_list_head = &timer->ack_list_head;
+               else
+                       ack_list_head = &timer->sack_list_head;
+               if (list_empty(&ti->ack_list))
+                       list_add_tail(&ti->ack_list, ack_list_head);
                list_for_each(q, &ti->slave_active_head) {
-                       ts = (snd_timer_instance_t *)list_entry(q, snd_timer_instance_t, active_list);
+                       ts = list_entry(q, struct snd_timer_instance, active_list);
                        ts->pticks = ti->pticks;
                        ts->resolution = resolution;
-                       if (list_empty(&ts->ack_list)) {
-                               if ((timer->hw.flags & SNDRV_TIMER_HW_TASKLET) ||
-                                   (ti->flags & SNDRV_TIMER_IFLG_FAST)) {
-                                       list_add_tail(&ts->ack_list, &timer->ack_list_head);
-                               } else {
-                                       list_add_tail(&ts->ack_list, &timer->sack_list_head);
-                               }
-                       }
+                       if (list_empty(&ts->ack_list))
+                               list_add_tail(&ts->ack_list, ack_list_head);
                }
        }
        if (timer->flags & SNDRV_TIMER_FLG_RESCHED)
@@ -719,11 +737,11 @@ void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
        /* now process all fast callbacks */
        while (!list_empty(&timer->ack_list_head)) {
                p = timer->ack_list_head.next;          /* get first item */
-               ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, ack_list);
-               
+               ti = list_entry(p, struct snd_timer_instance, ack_list);
+
                /* remove from ack_list and make empty */
                list_del_init(p);
-               
+
                ticks = ti->pticks;
                ti->pticks = 0;
 
@@ -737,7 +755,7 @@ void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
 
        /* do we have any slow callbacks? */
        use_tasklet = !list_empty(&timer->sack_list_head);
-       spin_unlock(&timer->lock);
+       spin_unlock_irqrestore(&timer->lock, flags);
 
        if (use_tasklet)
                tasklet_hi_schedule(&timer->task_queue);
@@ -747,11 +765,12 @@ void snd_timer_interrupt(snd_timer_t * timer, unsigned long ticks_left)
 
  */
 
-int snd_timer_new(snd_card_t *card, char *id, snd_timer_id_t *tid, snd_timer_t ** rtimer)
+int snd_timer_new(struct snd_card *card, char *id, struct snd_timer_id *tid,
+                 struct snd_timer **rtimer)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        int err;
-       static snd_device_ops_t ops = {
+       static struct snd_device_ops ops = {
                .dev_free = snd_timer_dev_free,
                .dev_register = snd_timer_dev_register,
                .dev_unregister = snd_timer_dev_unregister
@@ -760,9 +779,11 @@ int snd_timer_new(snd_card_t *card, char *id, snd_timer_id_t *tid, snd_timer_t *
        snd_assert(tid != NULL, return -EINVAL);
        snd_assert(rtimer != NULL, return -EINVAL);
        *rtimer = NULL;
-       timer = snd_magic_kcalloc(snd_timer_t, 0, GFP_KERNEL);
-       if (timer == NULL)
+       timer = kzalloc(sizeof(*timer), GFP_KERNEL);
+       if (timer == NULL) {
+               snd_printk(KERN_ERR "timer: cannot allocate\n");
                return -ENOMEM;
+       }
        timer->tmr_class = tid->dev_class;
        timer->card = card;
        timer->tmr_device = tid->device;
@@ -775,9 +796,12 @@ int snd_timer_new(snd_card_t *card, char *id, snd_timer_id_t *tid, snd_timer_t *
        INIT_LIST_HEAD(&timer->ack_list_head);
        INIT_LIST_HEAD(&timer->sack_list_head);
        spin_lock_init(&timer->lock);
-       tasklet_init(&timer->task_queue, snd_timer_tasklet, (unsigned long)timer);
+       tasklet_init(&timer->task_queue, snd_timer_tasklet,
+                    (unsigned long)timer);
        if (card != NULL) {
-               if ((err = snd_device_new(card, SNDRV_DEV_TIMER, timer, &ops)) < 0) {
+               timer->module = card->module;
+               err = snd_device_new(card, SNDRV_DEV_TIMER, timer, &ops);
+               if (err < 0) {
                        snd_timer_free(timer);
                        return err;
                }
@@ -786,35 +810,36 @@ int snd_timer_new(snd_card_t *card, char *id, snd_timer_id_t *tid, snd_timer_t *
        return 0;
 }
 
-static int snd_timer_free(snd_timer_t *timer)
+static int snd_timer_free(struct snd_timer *timer)
 {
        snd_assert(timer != NULL, return -ENXIO);
        if (timer->private_free)
                timer->private_free(timer);
-       snd_magic_kfree(timer);
+       kfree(timer);
        return 0;
 }
 
-int snd_timer_dev_free(snd_device_t *device)
+static int snd_timer_dev_free(struct snd_device *device)
 {
-       snd_timer_t *timer = snd_magic_cast(snd_timer_t, device->device_data, return -ENXIO);
+       struct snd_timer *timer = device->device_data;
        return snd_timer_free(timer);
 }
 
-int snd_timer_dev_register(snd_device_t *dev)
+static int snd_timer_dev_register(struct snd_device *dev)
 {
-       snd_timer_t *timer = snd_magic_cast(snd_timer_t, dev->device_data, return -ENXIO);
-       snd_timer_t *timer1;
+       struct snd_timer *timer = dev->device_data;
+       struct snd_timer *timer1;
        struct list_head *p;
 
-       snd_assert(timer != NULL && timer->hw.start != NULL && timer->hw.stop != NULL, return -ENXIO);
+       snd_assert(timer != NULL && timer->hw.start != NULL &&
+                  timer->hw.stop != NULL, return -ENXIO);
        if (!(timer->hw.flags & SNDRV_TIMER_HW_SLAVE) &&
            !timer->hw.resolution && timer->hw.c_resolution == NULL)
                return -EINVAL;
 
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        list_for_each(p, &snd_timer_list) {
-               timer1 = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+               timer1 = list_entry(p, struct snd_timer, device_list);
                if (timer1->tmr_class > timer->tmr_class)
                        break;
                if (timer1->tmr_class < timer->tmr_class)
@@ -834,62 +859,66 @@ int snd_timer_dev_register(snd_device_t *dev)
                if (timer1->tmr_subdevice < timer->tmr_subdevice)
                        continue;
                /* conflicts.. */
-               up(&register_mutex);
+               mutex_unlock(&register_mutex);
                return -EBUSY;
        }
        list_add_tail(&timer->device_list, p);
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return 0;
 }
 
-int snd_timer_unregister(snd_timer_t *timer)
+static int snd_timer_unregister(struct snd_timer *timer)
 {
        struct list_head *p, *n;
-       snd_timer_instance_t *ti;
+       struct snd_timer_instance *ti;
 
        snd_assert(timer != NULL, return -ENXIO);
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        if (! list_empty(&timer->open_list_head)) {
                snd_printk(KERN_WARNING "timer 0x%lx is busy?\n", (long)timer);
                list_for_each_safe(p, n, &timer->open_list_head) {
                        list_del_init(p);
-                       ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, open_list);
+                       ti = list_entry(p, struct snd_timer_instance, open_list);
                        ti->timer = NULL;
                }
        }
        list_del(&timer->device_list);
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return snd_timer_free(timer);
 }
 
-static int snd_timer_dev_unregister(snd_device_t *device)
+static int snd_timer_dev_unregister(struct snd_device *device)
 {
-       snd_timer_t *timer = snd_magic_cast(snd_timer_t, device->device_data, return -ENXIO);
+       struct snd_timer *timer = device->device_data;
        return snd_timer_unregister(timer);
 }
 
-void snd_timer_notify(snd_timer_t *timer, enum sndrv_timer_event event, struct timespec *tstamp)
+void snd_timer_notify(struct snd_timer *timer, int event, struct timespec *tstamp)
 {
        unsigned long flags;
        unsigned long resolution = 0;
-       snd_timer_instance_t *ti, *ts;
+       struct snd_timer_instance *ti, *ts;
        struct list_head *p, *n;
 
-       snd_runtime_check(timer->hw.flags & SNDRV_TIMER_HW_SLAVE, return);      
-       snd_assert(event >= SNDRV_TIMER_EVENT_MSTART && event <= SNDRV_TIMER_EVENT_MPAUSE, return);
+       if (! (timer->hw.flags & SNDRV_TIMER_HW_SLAVE))
+               return;
+       snd_assert(event >= SNDRV_TIMER_EVENT_MSTART &&
+                  event <= SNDRV_TIMER_EVENT_MRESUME, return);
        spin_lock_irqsave(&timer->lock, flags);
-       if (event == SNDRV_TIMER_EVENT_MSTART || event == SNDRV_TIMER_EVENT_MCONTINUE) {
+       if (event == SNDRV_TIMER_EVENT_MSTART ||
+           event == SNDRV_TIMER_EVENT_MCONTINUE ||
+           event == SNDRV_TIMER_EVENT_MRESUME) {
                if (timer->hw.c_resolution)
                        resolution = timer->hw.c_resolution(timer);
                else
                        resolution = timer->hw.resolution;
        }
        list_for_each(p, &timer->active_list_head) {
-               ti = (snd_timer_instance_t *)list_entry(p, snd_timer_instance_t, active_list);
+               ti = list_entry(p, struct snd_timer_instance, active_list);
                if (ti->ccallback)
                        ti->ccallback(ti, event, tstamp, resolution);
                list_for_each(n, &ti->slave_active_head) {
-                       ts = (snd_timer_instance_t *)list_entry(n, snd_timer_instance_t, active_list);
+                       ts = list_entry(n, struct snd_timer_instance, active_list);
                        if (ts->ccallback)
                                ts->ccallback(ts, event, tstamp, resolution);
                }
@@ -900,10 +929,10 @@ void snd_timer_notify(snd_timer_t *timer, enum sndrv_timer_event event, struct t
 /*
  * exported functions for global timers
  */
-int snd_timer_global_new(char *id, int device, snd_timer_t **rtimer)
+int snd_timer_global_new(char *id, int device, struct snd_timer **rtimer)
 {
-       snd_timer_id_t tid;
-       
+       struct snd_timer_id tid;
+
        tid.dev_class = SNDRV_TIMER_CLASS_GLOBAL;
        tid.dev_sclass = SNDRV_TIMER_SCLASS_NONE;
        tid.card = -1;
@@ -912,26 +941,26 @@ int snd_timer_global_new(char *id, int device, snd_timer_t **rtimer)
        return snd_timer_new(NULL, id, &tid, rtimer);
 }
 
-int snd_timer_global_free(snd_timer_t *timer)
+int snd_timer_global_free(struct snd_timer *timer)
 {
        return snd_timer_free(timer);
 }
 
-int snd_timer_global_register(snd_timer_t *timer)
+int snd_timer_global_register(struct snd_timer *timer)
 {
-       snd_device_t dev;
+       struct snd_device dev;
 
        memset(&dev, 0, sizeof(dev));
        dev.device_data = timer;
        return snd_timer_dev_register(&dev);
 }
 
-int snd_timer_global_unregister(snd_timer_t *timer)
+int snd_timer_global_unregister(struct snd_timer *timer)
 {
        return snd_timer_unregister(timer);
 }
 
-/* 
+/*
  *  System timer
  */
 
@@ -943,14 +972,9 @@ struct snd_timer_system_private {
        unsigned long correction;
 };
 
-unsigned int snd_timer_system_resolution(void)
-{
-       return 1000000000L / HZ;
-}
-
 static void snd_timer_s_function(unsigned long data)
 {
-       snd_timer_t *timer = (snd_timer_t *)data;
+       struct snd_timer *timer = (struct snd_timer *)data;
        struct snd_timer_system_private *priv = timer->private_data;
        unsigned long jiff = jiffies;
        if (time_after(jiff, priv->last_expires))
@@ -958,7 +982,7 @@ static void snd_timer_s_function(unsigned long data)
        snd_timer_interrupt(timer, (long)jiff - (long)priv->last_jiffies);
 }
 
-static int snd_timer_s_start(snd_timer_t * timer)
+static int snd_timer_s_start(struct snd_timer * timer)
 {
        struct snd_timer_system_private *priv;
        unsigned long njiff;
@@ -977,7 +1001,7 @@ static int snd_timer_s_start(snd_timer_t * timer)
        return 0;
 }
 
-static int snd_timer_s_stop(snd_timer_t * timer)
+static int snd_timer_s_stop(struct snd_timer * timer)
 {
        struct snd_timer_system_private *priv;
        unsigned long jiff;
@@ -992,7 +1016,7 @@ static int snd_timer_s_stop(snd_timer_t * timer)
        return 0;
 }
 
-static struct _snd_timer_hardware snd_timer_system =
+static struct snd_timer_hardware snd_timer_system =
 {
        .flags =        SNDRV_TIMER_HW_FIRST | SNDRV_TIMER_HW_TASKLET,
        .resolution =   1000000000L / HZ,
@@ -1001,23 +1025,23 @@ static struct _snd_timer_hardware snd_timer_system =
        .stop =         snd_timer_s_stop
 };
 
-static void snd_timer_free_system(snd_timer_t *timer)
+static void snd_timer_free_system(struct snd_timer *timer)
 {
-       if (timer->private_data)
-               kfree(timer->private_data);
+       kfree(timer->private_data);
 }
 
 static int snd_timer_register_system(void)
 {
-       snd_timer_t *timer;
+       struct snd_timer *timer;
        struct snd_timer_system_private *priv;
        int err;
 
-       if ((err = snd_timer_global_new("system", SNDRV_TIMER_GLOBAL_SYSTEM, &timer)) < 0)
+       err = snd_timer_global_new("system", SNDRV_TIMER_GLOBAL_SYSTEM, &timer);
+       if (err < 0)
                return err;
        strcpy(timer->name, "system timer");
        timer->hw = snd_timer_system;
-       priv = (struct snd_timer_system_private *) snd_kcalloc(sizeof(struct snd_timer_system_private), GFP_KERNEL);
+       priv = kzalloc(sizeof(*priv), GFP_KERNEL);
        if (priv == NULL) {
                snd_timer_free(timer);
                return -ENOMEM;
@@ -1030,65 +1054,101 @@ static int snd_timer_register_system(void)
        return snd_timer_global_register(timer);
 }
 
+#ifdef CONFIG_PROC_FS
 /*
  *  Info interface
  */
 
-static void snd_timer_proc_read(snd_info_entry_t *entry,
-                               snd_info_buffer_t * buffer)
+static void snd_timer_proc_read(struct snd_info_entry *entry,
+                               struct snd_info_buffer *buffer)
 {
        unsigned long flags;
-       snd_timer_t *timer;
-       snd_timer_instance_t *ti;
+       struct snd_timer *timer;
+       struct snd_timer_instance *ti;
        struct list_head *p, *q;
 
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        list_for_each(p, &snd_timer_list) {
-               timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+               timer = list_entry(p, struct snd_timer, device_list);
                switch (timer->tmr_class) {
                case SNDRV_TIMER_CLASS_GLOBAL:
                        snd_iprintf(buffer, "G%i: ", timer->tmr_device);
                        break;
                case SNDRV_TIMER_CLASS_CARD:
-                       snd_iprintf(buffer, "C%i-%i: ", timer->card->number, timer->tmr_device);
+                       snd_iprintf(buffer, "C%i-%i: ",
+                                   timer->card->number, timer->tmr_device);
                        break;
                case SNDRV_TIMER_CLASS_PCM:
-                       snd_iprintf(buffer, "P%i-%i-%i: ", timer->card->number, timer->tmr_device, timer->tmr_subdevice);
+                       snd_iprintf(buffer, "P%i-%i-%i: ", timer->card->number,
+                                   timer->tmr_device, timer->tmr_subdevice);
                        break;
                default:
-                       snd_iprintf(buffer, "?%i-%i-%i-%i: ", timer->tmr_class, timer->card ? timer->card->number : -1, timer->tmr_device, timer->tmr_subdevice);
+                       snd_iprintf(buffer, "?%i-%i-%i-%i: ", timer->tmr_class,
+                                   timer->card ? timer->card->number : -1,
+                                   timer->tmr_device, timer->tmr_subdevice);
                }
                snd_iprintf(buffer, "%s :", timer->name);
                if (timer->hw.resolution)
-                       snd_iprintf(buffer, " %lu.%03luus (%lu ticks)", timer->hw.resolution / 1000, timer->hw.resolution % 1000, timer->hw.ticks);
+                       snd_iprintf(buffer, " %lu.%03luus (%lu ticks)",
+                                   timer->hw.resolution / 1000,
+                                   timer->hw.resolution % 1000,
+                                   timer->hw.ticks);
                if (timer->hw.flags & SNDRV_TIMER_HW_SLAVE)
                        snd_iprintf(buffer, " SLAVE");
                snd_iprintf(buffer, "\n");
                spin_lock_irqsave(&timer->lock, flags);
                list_for_each(q, &timer->open_list_head) {
-                       ti = (snd_timer_instance_t *)list_entry(q, snd_timer_instance_t, open_list);
-                       snd_iprintf(buffer, "  Client %s : %s : lost interrupts %li\n",
-                                       ti->owner ? ti->owner : "unknown",
-                                       ti->flags & (SNDRV_TIMER_IFLG_START|SNDRV_TIMER_IFLG_RUNNING) ? "running" : "stopped",
-                                       ti->lost);
+                       ti = list_entry(q, struct snd_timer_instance, open_list);
+                       snd_iprintf(buffer, "  Client %s : %s\n",
+                                   ti->owner ? ti->owner : "unknown",
+                                   ti->flags & (SNDRV_TIMER_IFLG_START |
+                                                SNDRV_TIMER_IFLG_RUNNING)
+                                   ? "running" : "stopped");
                }
                spin_unlock_irqrestore(&timer->lock, flags);
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
 }
 
+static struct snd_info_entry *snd_timer_proc_entry = NULL;
+
+static void __init snd_timer_proc_init(void)
+{
+       struct snd_info_entry *entry;
+
+       entry = snd_info_create_module_entry(THIS_MODULE, "timers", NULL);
+       if (entry != NULL) {
+               entry->c.text.read_size = SNDRV_TIMER_DEVICES * 128;
+               entry->c.text.read = snd_timer_proc_read;
+               if (snd_info_register(entry) < 0) {
+                       snd_info_free_entry(entry);
+                       entry = NULL;
+               }
+       }
+       snd_timer_proc_entry = entry;
+}
+
+static void __exit snd_timer_proc_done(void)
+{
+       snd_info_unregister(snd_timer_proc_entry);
+}
+#else /* !CONFIG_PROC_FS */
+#define snd_timer_proc_init()
+#define snd_timer_proc_done()
+#endif
+
 /*
  *  USER SPACE interface
  */
 
-static void snd_timer_user_interrupt(snd_timer_instance_t *timeri,
+static void snd_timer_user_interrupt(struct snd_timer_instance *timeri,
                                     unsigned long resolution,
                                     unsigned long ticks)
 {
-       snd_timer_user_t *tu = snd_magic_cast(snd_timer_user_t, timeri->callback_data, return);
-       snd_timer_read_t *r;
+       struct snd_timer_user *tu = timeri->callback_data;
+       struct snd_timer_read *r;
        int prev;
-       
+
        spin_lock(&tu->qlock);
        if (tu->qused > 0) {
                prev = tu->qtail == 0 ? tu->queue_size - 1 : tu->qtail - 1;
@@ -1113,25 +1173,28 @@ static void snd_timer_user_interrupt(snd_timer_instance_t *timeri,
        wake_up(&tu->qchange_sleep);
 }
 
-static void snd_timer_user_append_to_tqueue(snd_timer_user_t *tu, snd_timer_tread_t *tread)
+static void snd_timer_user_append_to_tqueue(struct snd_timer_user *tu,
+                                           struct snd_timer_tread *tread)
 {
        if (tu->qused >= tu->queue_size) {
                tu->overrun++;
        } else {
-               memcpy(&tu->queue[tu->qtail++], tread, sizeof(*tread));
+               memcpy(&tu->tqueue[tu->qtail++], tread, sizeof(*tread));
+               tu->qtail %= tu->queue_size;
                tu->qused++;
        }
 }
 
-static void snd_timer_user_ccallback(snd_timer_instance_t *timeri,
-                                    enum sndrv_timer_event event,
+static void snd_timer_user_ccallback(struct snd_timer_instance *timeri,
+                                    int event,
                                     struct timespec *tstamp,
                                     unsigned long resolution)
 {
-       snd_timer_user_t *tu = snd_magic_cast(snd_timer_user_t, timeri->callback_data, return);
-       snd_timer_tread_t r1;
+       struct snd_timer_user *tu = timeri->callback_data;
+       struct snd_timer_tread r1;
 
-       if (event >= SNDRV_TIMER_EVENT_START && event <= SNDRV_TIMER_EVENT_PAUSE)
+       if (event >= SNDRV_TIMER_EVENT_START &&
+           event <= SNDRV_TIMER_EVENT_PAUSE)
                tu->tstamp = *tstamp;
        if ((tu->filter & (1 << event)) == 0 || !tu->tread)
                return;
@@ -1141,26 +1204,30 @@ static void snd_timer_user_ccallback(snd_timer_instance_t *timeri,
        spin_lock(&tu->qlock);
        snd_timer_user_append_to_tqueue(tu, &r1);
        spin_unlock(&tu->qlock);
+       kill_fasync(&tu->fasync, SIGIO, POLL_IN);
+       wake_up(&tu->qchange_sleep);
 }
 
-static void snd_timer_user_tinterrupt(snd_timer_instance_t *timeri,
+static void snd_timer_user_tinterrupt(struct snd_timer_instance *timeri,
                                      unsigned long resolution,
                                      unsigned long ticks)
 {
-       snd_timer_user_t *tu = snd_magic_cast(snd_timer_user_t, timeri->callback_data, return);
-       snd_timer_tread_t *r, r1;
+       struct snd_timer_user *tu = timeri->callback_data;
+       struct snd_timer_tread *r, r1;
        struct timespec tstamp;
        int prev, append = 0;
 
-       snd_timestamp_zero(&tstamp);
+       memset(&tstamp, 0, sizeof(tstamp));
        spin_lock(&tu->qlock);
-       if ((tu->filter & ((1 << SNDRV_TIMER_EVENT_RESOLUTION)|(1 << SNDRV_TIMER_EVENT_TICK))) == 0) {
+       if ((tu->filter & ((1 << SNDRV_TIMER_EVENT_RESOLUTION) |
+                          (1 << SNDRV_TIMER_EVENT_TICK))) == 0) {
                spin_unlock(&tu->qlock);
                return;
        }
        if (tu->last_resolution != resolution || ticks > 0)
-               snd_timestamp_now(&tstamp, 1);
-       if ((tu->filter & (1 << SNDRV_TIMER_EVENT_RESOLUTION)) && tu->last_resolution != resolution) {
+               getnstimeofday(&tstamp);
+       if ((tu->filter & (1 << SNDRV_TIMER_EVENT_RESOLUTION)) &&
+           tu->last_resolution != resolution) {
                r1.event = SNDRV_TIMER_EVENT_RESOLUTION;
                r1.tstamp = tstamp;
                r1.val = resolution;
@@ -1197,18 +1264,20 @@ static void snd_timer_user_tinterrupt(snd_timer_instance_t *timeri,
 
 static int snd_timer_user_open(struct inode *inode, struct file *file)
 {
-       snd_timer_user_t *tu;
-       
-       tu = snd_magic_kcalloc(snd_timer_user_t, 0, GFP_KERNEL);
+       struct snd_timer_user *tu;
+
+       tu = kzalloc(sizeof(*tu), GFP_KERNEL);
        if (tu == NULL)
                return -ENOMEM;
        spin_lock_init(&tu->qlock);
        init_waitqueue_head(&tu->qchange_sleep);
+       mutex_init(&tu->tread_sem);
        tu->ticks = 1;
        tu->queue_size = 128;
-       tu->queue = (snd_timer_read_t *)kmalloc(tu->queue_size * sizeof(snd_timer_read_t), GFP_KERNEL);
+       tu->queue = kmalloc(tu->queue_size * sizeof(struct snd_timer_read),
+                           GFP_KERNEL);
        if (tu->queue == NULL) {
-               snd_magic_kfree(tu);
+               kfree(tu);
                return -ENOMEM;
        }
        file->private_data = tu;
@@ -1217,24 +1286,22 @@ static int snd_timer_user_open(struct inode *inode, struct file *file)
 
 static int snd_timer_user_release(struct inode *inode, struct file *file)
 {
-       snd_timer_user_t *tu;
+       struct snd_timer_user *tu;
 
        if (file->private_data) {
-               tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+               tu = file->private_data;
                file->private_data = NULL;
                fasync_helper(-1, file, 0, &tu->fasync);
                if (tu->timeri)
                        snd_timer_close(tu->timeri);
-               if (tu->queue)
-                       kfree(tu->queue);
-               if (tu->tqueue)
-                       kfree(tu->tqueue);
-               snd_magic_kfree(tu);
+               kfree(tu->queue);
+               kfree(tu->tqueue);
+               kfree(tu);
        }
        return 0;
 }
 
-static void snd_timer_user_zero_id(snd_timer_id_t *id)
+static void snd_timer_user_zero_id(struct snd_timer_id *id)
 {
        id->dev_class = SNDRV_TIMER_CLASS_NONE;
        id->dev_sclass = SNDRV_TIMER_SCLASS_NONE;
@@ -1243,7 +1310,7 @@ static void snd_timer_user_zero_id(snd_timer_id_t *id)
        id->subdevice = -1;
 }
 
-static void snd_timer_user_copy_id(snd_timer_id_t *id, snd_timer_t *timer)
+static void snd_timer_user_copy_id(struct snd_timer_id *id, struct snd_timer *timer)
 {
        id->dev_class = timer->tmr_class;
        id->dev_sclass = SNDRV_TIMER_SCLASS_NONE;
@@ -1252,20 +1319,21 @@ static void snd_timer_user_copy_id(snd_timer_id_t *id, snd_timer_t *timer)
        id->subdevice = timer->tmr_subdevice;
 }
 
-static int snd_timer_user_next_device(snd_timer_id_t *_tid)
+static int snd_timer_user_next_device(struct snd_timer_id __user *_tid)
 {
-       snd_timer_id_t id;
-       snd_timer_t *timer;
+       struct snd_timer_id id;
+       struct snd_timer *timer;
        struct list_head *p;
-       
+
        if (copy_from_user(&id, _tid, sizeof(id)))
                return -EFAULT;
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        if (id.dev_class < 0) {         /* first item */
                if (list_empty(&snd_timer_list))
                        snd_timer_user_zero_id(&id);
                else {
-                       timer = (snd_timer_t *)list_entry(snd_timer_list.next, snd_timer_t, device_list);
+                       timer = list_entry(snd_timer_list.next,
+                                          struct snd_timer, device_list);
                        snd_timer_user_copy_id(&id, timer);
                }
        } else {
@@ -1273,7 +1341,7 @@ static int snd_timer_user_next_device(snd_timer_id_t *_tid)
                case SNDRV_TIMER_CLASS_GLOBAL:
                        id.device = id.device < 0 ? 0 : id.device + 1;
                        list_for_each(p, &snd_timer_list) {
-                               timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+                               timer = list_entry(p, struct snd_timer, device_list);
                                if (timer->tmr_class > SNDRV_TIMER_CLASS_GLOBAL) {
                                        snd_timer_user_copy_id(&id, timer);
                                        break;
@@ -1297,12 +1365,16 @@ static int snd_timer_user_next_device(snd_timer_id_t *_tid)
                                        if (id.device < 0) {
                                                id.device = 0;
                                        } else {
-                                               id.subdevice = id.subdevice < 0 ? 0 : id.subdevice + 1;
+                                               if (id.subdevice < 0) {
+                                                       id.subdevice = 0;
+                                               } else {
+                                                       id.subdevice++;
+                                               }
                                        }
                                }
                        }
                        list_for_each(p, &snd_timer_list) {
-                               timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+                               timer = list_entry(p, struct snd_timer, device_list);
                                if (timer->tmr_class > id.dev_class) {
                                        snd_timer_user_copy_id(&id, timer);
                                        break;
@@ -1337,81 +1409,92 @@ static int snd_timer_user_next_device(snd_timer_id_t *_tid)
                        snd_timer_user_zero_id(&id);
                }
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        if (copy_to_user(_tid, &id, sizeof(*_tid)))
                return -EFAULT;
        return 0;
-} 
+}
 
-static int snd_timer_user_ginfo(struct file *file, snd_timer_ginfo_t *_ginfo)
+static int snd_timer_user_ginfo(struct file *file,
+                               struct snd_timer_ginfo __user *_ginfo)
 {
-       snd_timer_ginfo_t ginfo;
-       snd_timer_id_t tid;
-       snd_timer_t *t;
+       struct snd_timer_ginfo *ginfo;
+       struct snd_timer_id tid;
+       struct snd_timer *t;
        struct list_head *p;
        int err = 0;
 
-       if (copy_from_user(&ginfo, _ginfo, sizeof(ginfo)))
+       ginfo = kmalloc(sizeof(*ginfo), GFP_KERNEL);
+       if (! ginfo)
+               return -ENOMEM;
+       if (copy_from_user(ginfo, _ginfo, sizeof(*ginfo))) {
+               kfree(ginfo);
                return -EFAULT;
-       tid = ginfo.tid;
-       memset(&ginfo, 0, sizeof(ginfo));
-       ginfo.tid = tid;
-       down(&register_mutex);
+       }
+       tid = ginfo->tid;
+       memset(ginfo, 0, sizeof(*ginfo));
+       ginfo->tid = tid;
+       mutex_lock(&register_mutex);
        t = snd_timer_find(&tid);
        if (t != NULL) {
-               ginfo.card = t->card ? t->card->number : -1;
+               ginfo->card = t->card ? t->card->number : -1;
                if (t->hw.flags & SNDRV_TIMER_HW_SLAVE)
-                       ginfo.flags |= SNDRV_TIMER_FLG_SLAVE;
-               strlcpy(ginfo.id, t->id, sizeof(ginfo.id));
-               strlcpy(ginfo.name, t->name, sizeof(ginfo.name));
-               ginfo.resolution = t->hw.resolution;
+                       ginfo->flags |= SNDRV_TIMER_FLG_SLAVE;
+               strlcpy(ginfo->id, t->id, sizeof(ginfo->id));
+               strlcpy(ginfo->name, t->name, sizeof(ginfo->name));
+               ginfo->resolution = t->hw.resolution;
                if (t->hw.resolution_min > 0) {
-                       ginfo.resolution_min = t->hw.resolution_min;
-                       ginfo.resolution_max = t->hw.resolution_max;
+                       ginfo->resolution_min = t->hw.resolution_min;
+                       ginfo->resolution_max = t->hw.resolution_max;
                }
                list_for_each(p, &t->open_list_head) {
-                       ginfo.clients++;
+                       ginfo->clients++;
                }
        } else {
                err = -ENODEV;
        }
-       up(&register_mutex);
-       if (err >= 0 && copy_to_user(_ginfo, &ginfo, sizeof(ginfo)))
+       mutex_unlock(&register_mutex);
+       if (err >= 0 && copy_to_user(_ginfo, ginfo, sizeof(*ginfo)))
                err = -EFAULT;
+       kfree(ginfo);
        return err;
 }
 
-static int snd_timer_user_gparams(struct file *file, snd_timer_gparams_t *_gparams)
+static int snd_timer_user_gparams(struct file *file,
+                                 struct snd_timer_gparams __user *_gparams)
 {
-       snd_timer_gparams_t gparams;
-       snd_timer_t *t;
+       struct snd_timer_gparams gparams;
+       struct snd_timer *t;
        int err;
 
        if (copy_from_user(&gparams, _gparams, sizeof(gparams)))
                return -EFAULT;
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        t = snd_timer_find(&gparams.tid);
-       if (t != NULL) {
-               if (list_empty(&t->open_list_head)) {
-                       if (t->hw.set_period)
-                               err = t->hw.set_period(t, gparams.period_num, gparams.period_den);
-                       else
-                               err = -ENOSYS;
-               } else {
-                       err = -EBUSY;
-               }
-       } else {
+       if (!t) {
                err = -ENODEV;
+               goto _error;
+       }
+       if (!list_empty(&t->open_list_head)) {
+               err = -EBUSY;
+               goto _error;
        }
-       up(&register_mutex);
+       if (!t->hw.set_period) {
+               err = -ENOSYS;
+               goto _error;
+       }
+       err = t->hw.set_period(t, gparams.period_num, gparams.period_den);
+_error:
+       mutex_unlock(&register_mutex);
        return err;
 }
 
-static int snd_timer_user_gstatus(struct file *file, snd_timer_gstatus_t *_gstatus)
+static int snd_timer_user_gstatus(struct file *file,
+                                 struct snd_timer_gstatus __user *_gstatus)
 {
-       snd_timer_gstatus_t gstatus;
-       snd_timer_id_t tid;
-       snd_timer_t *t;
+       struct snd_timer_gstatus gstatus;
+       struct snd_timer_id tid;
+       struct snd_timer *t;
        int err = 0;
 
        if (copy_from_user(&gstatus, _gstatus, sizeof(gstatus)))
@@ -1419,7 +1502,7 @@ static int snd_timer_user_gstatus(struct file *file, snd_timer_gstatus_t *_gstat
        tid = gstatus.tid;
        memset(&gstatus, 0, sizeof(gstatus));
        gstatus.tid = tid;
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        t = snd_timer_find(&tid);
        if (t != NULL) {
                if (t->hw.c_resolution)
@@ -1427,7 +1510,8 @@ static int snd_timer_user_gstatus(struct file *file, snd_timer_gstatus_t *_gstat
                else
                        gstatus.resolution = t->hw.resolution;
                if (t->hw.precise_resolution) {
-                       t->hw.precise_resolution(t, &gstatus.resolution_num, &gstatus.resolution_den);
+                       t->hw.precise_resolution(t, &gstatus.resolution_num,
+                                                &gstatus.resolution_den);
                } else {
                        gstatus.resolution_num = gstatus.resolution;
                        gstatus.resolution_den = 1000000000uL;
@@ -1435,92 +1519,108 @@ static int snd_timer_user_gstatus(struct file *file, snd_timer_gstatus_t *_gstat
        } else {
                err = -ENODEV;
        }
-       up(&register_mutex);
-       if (err >= 0 && copy_from_user(_gstatus, &gstatus, sizeof(gstatus)))
+       mutex_unlock(&register_mutex);
+       if (err >= 0 && copy_to_user(_gstatus, &gstatus, sizeof(gstatus)))
                err = -EFAULT;
        return err;
 }
 
-static int snd_timer_user_tselect(struct file *file, snd_timer_select_t *_tselect)
+static int snd_timer_user_tselect(struct file *file,
+                                 struct snd_timer_select __user *_tselect)
 {
-       snd_timer_user_t *tu;
-       snd_timer_select_t tselect;
+       struct snd_timer_user *tu;
+       struct snd_timer_select tselect;
        char str[32];
-       int err;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
-       if (tu->timeri)
+       int err = 0;
+
+       tu = file->private_data;
+       mutex_lock(&tu->tread_sem);
+       if (tu->timeri) {
                snd_timer_close(tu->timeri);
-       if (copy_from_user(&tselect, _tselect, sizeof(tselect)))
-               return -EFAULT;
+               tu->timeri = NULL;
+       }
+       if (copy_from_user(&tselect, _tselect, sizeof(tselect))) {
+               err = -EFAULT;
+               goto __err;
+       }
        sprintf(str, "application %i", current->pid);
        if (tselect.id.dev_class != SNDRV_TIMER_CLASS_SLAVE)
                tselect.id.dev_sclass = SNDRV_TIMER_SCLASS_APPLICATION;
-       if ((err = snd_timer_open(&tu->timeri, str, &tselect.id, current->pid)) < 0)
-               return err;
+       err = snd_timer_open(&tu->timeri, str, &tselect.id, current->pid);
+       if (err < 0)
+               goto __err;
 
-       if (tu->queue) {
-               kfree(tu->queue);
-               tu->queue = NULL;
-       }
-       if (tu->tqueue) {
-               kfree(tu->tqueue);
-               tu->tqueue = NULL;
-       }
+       kfree(tu->queue);
+       tu->queue = NULL;
+       kfree(tu->tqueue);
+       tu->tqueue = NULL;
        if (tu->tread) {
-               tu->tqueue = (snd_timer_tread_t *)kmalloc(tu->queue_size * sizeof(snd_timer_tread_t), GFP_KERNEL);
-               if (tu->tqueue == NULL) {
-                       snd_timer_close(tu->timeri);
-                       return -ENOMEM;
-               }
+               tu->tqueue = kmalloc(tu->queue_size * sizeof(struct snd_timer_tread),
+                                    GFP_KERNEL);
+               if (tu->tqueue == NULL)
+                       err = -ENOMEM;
        } else {
-               tu->queue = (snd_timer_read_t *)kmalloc(tu->queue_size * sizeof(snd_timer_read_t), GFP_KERNEL);
-               if (tu->queue == NULL) {
-                       snd_timer_close(tu->timeri);
-                       return -ENOMEM;
-               }
+               tu->queue = kmalloc(tu->queue_size * sizeof(struct snd_timer_read),
+                                   GFP_KERNEL);
+               if (tu->queue == NULL)
+                       err = -ENOMEM;
        }
-       
-       tu->timeri->flags |= SNDRV_TIMER_IFLG_FAST;
-       tu->timeri->callback = tu->tread ? snd_timer_user_tinterrupt : snd_timer_user_interrupt;
-       tu->timeri->ccallback = snd_timer_user_ccallback;
-       tu->timeri->callback_data = (void *)tu;
-       return 0;
+
+       if (err < 0) {
+               snd_timer_close(tu->timeri);
+               tu->timeri = NULL;
+       } else {
+               tu->timeri->flags |= SNDRV_TIMER_IFLG_FAST;
+               tu->timeri->callback = tu->tread
+                       ? snd_timer_user_tinterrupt : snd_timer_user_interrupt;
+               tu->timeri->ccallback = snd_timer_user_ccallback;
+               tu->timeri->callback_data = (void *)tu;
+       }
+
+      __err:
+       mutex_unlock(&tu->tread_sem);
+       return err;
 }
 
-static int snd_timer_user_info(struct file *file, snd_timer_info_t *_info)
+static int snd_timer_user_info(struct file *file,
+                              struct snd_timer_info __user *_info)
 {
-       snd_timer_user_t *tu;
-       snd_timer_info_t info;
-       snd_timer_t *t;
+       struct snd_timer_user *tu;
+       struct snd_timer_info *info;
+       struct snd_timer *t;
+       int err = 0;
 
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        t = tu->timeri->timer;
        snd_assert(t != NULL, return -ENXIO);
-       memset(&info, 0, sizeof(info));
-       info.card = t->card ? t->card->number : -1;
+
+       info = kzalloc(sizeof(*info), GFP_KERNEL);
+       if (! info)
+               return -ENOMEM;
+       info->card = t->card ? t->card->number : -1;
        if (t->hw.flags & SNDRV_TIMER_HW_SLAVE)
-               info.flags |= SNDRV_TIMER_FLG_SLAVE;
-       strlcpy(info.id, t->id, sizeof(info.id));
-       strlcpy(info.name, t->name, sizeof(info.name));
-       info.resolution = t->hw.resolution;
-       if (copy_to_user(_info, &info, sizeof(*_info)))
-               return -EFAULT;
-       return 0;
+               info->flags |= SNDRV_TIMER_FLG_SLAVE;
+       strlcpy(info->id, t->id, sizeof(info->id));
+       strlcpy(info->name, t->name, sizeof(info->name));
+       info->resolution = t->hw.resolution;
+       if (copy_to_user(_info, info, sizeof(*_info)))
+               err = -EFAULT;
+       kfree(info);
+       return err;
 }
 
-static int snd_timer_user_params(struct file *file, snd_timer_params_t *_params)
+static int snd_timer_user_params(struct file *file,
+                                struct snd_timer_params __user *_params)
 {
-       unsigned long flags;
-       snd_timer_user_t *tu;
-       snd_timer_params_t params;
-       snd_timer_t *t;
-       snd_timer_read_t *tr;
-       snd_timer_tread_t *ttr;
+       struct snd_timer_user *tu;
+       struct snd_timer_params params;
+       struct snd_timer *t;
+       struct snd_timer_read *tr;
+       struct snd_timer_tread *ttr;
        int err;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        t = tu->timeri->timer;
        snd_assert(t != NULL, return -ENXIO);
@@ -1530,7 +1630,8 @@ static int snd_timer_user_params(struct file *file, snd_timer_params_t *_params)
                err = -EINVAL;
                goto _end;
        }
-       if (params.queue_size > 0 && (params.queue_size < 32 || params.queue_size > 1024)) {
+       if (params.queue_size > 0 &&
+           (params.queue_size < 32 || params.queue_size > 1024)) {
                err = -EINVAL;
                goto _end;
        }
@@ -1540,36 +1641,42 @@ static int snd_timer_user_params(struct file *file, snd_timer_params_t *_params)
                              (1<<SNDRV_TIMER_EVENT_STOP)|
                              (1<<SNDRV_TIMER_EVENT_CONTINUE)|
                              (1<<SNDRV_TIMER_EVENT_PAUSE)|
+                             (1<<SNDRV_TIMER_EVENT_SUSPEND)|
+                             (1<<SNDRV_TIMER_EVENT_RESUME)|
                              (1<<SNDRV_TIMER_EVENT_MSTART)|
                              (1<<SNDRV_TIMER_EVENT_MSTOP)|
                              (1<<SNDRV_TIMER_EVENT_MCONTINUE)|
-                             (1<<SNDRV_TIMER_EVENT_MPAUSE))) {
+                             (1<<SNDRV_TIMER_EVENT_MPAUSE)|
+                             (1<<SNDRV_TIMER_EVENT_MSUSPEND)|
+                             (1<<SNDRV_TIMER_EVENT_MRESUME))) {
                err = -EINVAL;
                goto _end;
        }
        snd_timer_stop(tu->timeri);
-       spin_lock_irqsave(&t->lock, flags);
-       if (params.flags & SNDRV_TIMER_PSFLG_AUTO) {
+       spin_lock_irq(&t->lock);
+       tu->timeri->flags &= ~(SNDRV_TIMER_IFLG_AUTO|
+                              SNDRV_TIMER_IFLG_EXCLUSIVE|
+                              SNDRV_TIMER_IFLG_EARLY_EVENT);
+       if (params.flags & SNDRV_TIMER_PSFLG_AUTO)
                tu->timeri->flags |= SNDRV_TIMER_IFLG_AUTO;
-       } else {
-               tu->timeri->flags &= ~SNDRV_TIMER_IFLG_AUTO;
-       }
-       if (params.flags & SNDRV_TIMER_PSFLG_EXCLUSIVE) {
+       if (params.flags & SNDRV_TIMER_PSFLG_EXCLUSIVE)
                tu->timeri->flags |= SNDRV_TIMER_IFLG_EXCLUSIVE;
-       } else {
-               tu->timeri->flags &= ~SNDRV_TIMER_IFLG_EXCLUSIVE;
-       }
-       spin_unlock_irqrestore(&t->lock, flags);
-       if (params.queue_size > 0 && (unsigned int)tu->queue_size != params.queue_size) {
+       if (params.flags & SNDRV_TIMER_PSFLG_EARLY_EVENT)
+               tu->timeri->flags |= SNDRV_TIMER_IFLG_EARLY_EVENT;
+       spin_unlock_irq(&t->lock);
+       if (params.queue_size > 0 &&
+           (unsigned int)tu->queue_size != params.queue_size) {
                if (tu->tread) {
-                       ttr = (snd_timer_tread_t *)kmalloc(params.queue_size * sizeof(snd_timer_tread_t), GFP_KERNEL);
+                       ttr = kmalloc(params.queue_size * sizeof(*ttr),
+                                     GFP_KERNEL);
                        if (ttr) {
                                kfree(tu->tqueue);
                                tu->queue_size = params.queue_size;
                                tu->tqueue = ttr;
                        }
                } else {
-                       tr = (snd_timer_read_t *)kmalloc(params.queue_size * sizeof(snd_timer_read_t), GFP_KERNEL);
+                       tr = kmalloc(params.queue_size * sizeof(*tr),
+                                    GFP_KERNEL);
                        if (tr) {
                                kfree(tu->queue);
                                tu->queue_size = params.queue_size;
@@ -1577,6 +1684,23 @@ static int snd_timer_user_params(struct file *file, snd_timer_params_t *_params)
                        }
                }
        }
+       tu->qhead = tu->qtail = tu->qused = 0;
+       if (tu->timeri->flags & SNDRV_TIMER_IFLG_EARLY_EVENT) {
+               if (tu->tread) {
+                       struct snd_timer_tread tread;
+                       tread.event = SNDRV_TIMER_EVENT_EARLY;
+                       tread.tstamp.tv_sec = 0;
+                       tread.tstamp.tv_nsec = 0;
+                       tread.val = 0;
+                       snd_timer_user_append_to_tqueue(tu, &tread);
+               } else {
+                       struct snd_timer_read *r = &tu->queue[0];
+                       r->resolution = 0;
+                       r->ticks = 0;
+                       tu->qused++;
+                       tu->qtail++;
+               }
+       }
        tu->filter = params.filter;
        tu->ticks = params.ticks;
        err = 0;
@@ -1586,22 +1710,22 @@ static int snd_timer_user_params(struct file *file, snd_timer_params_t *_params)
        return err;
 }
 
-static int snd_timer_user_status(struct file *file, snd_timer_status_t *_status)
+static int snd_timer_user_status(struct file *file,
+                                struct snd_timer_status __user *_status)
 {
-       unsigned long flags;
-       snd_timer_user_t *tu;
-       snd_timer_status_t status;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       struct snd_timer_user *tu;
+       struct snd_timer_status status;
+
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        memset(&status, 0, sizeof(status));
        status.tstamp = tu->tstamp;
        status.resolution = snd_timer_resolution(tu->timeri);
        status.lost = tu->timeri->lost;
        status.overrun = tu->overrun;
-       spin_lock_irqsave(&tu->qlock, flags);
+       spin_lock_irq(&tu->qlock);
        status.queue = tu->qused;
-       spin_unlock_irqrestore(&tu->qlock, flags);
+       spin_unlock_irq(&tu->qlock);
        if (copy_to_user(_status, &status, sizeof(status)))
                return -EFAULT;
        return 0;
@@ -1610,9 +1734,9 @@ static int snd_timer_user_status(struct file *file, snd_timer_status_t *_status)
 static int snd_timer_user_start(struct file *file)
 {
        int err;
-       snd_timer_user_t *tu;
-               
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       struct snd_timer_user *tu;
+
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        snd_timer_stop(tu->timeri);
        tu->timeri->lost = 0;
@@ -1623,9 +1747,9 @@ static int snd_timer_user_start(struct file *file)
 static int snd_timer_user_stop(struct file *file)
 {
        int err;
-       snd_timer_user_t *tu;
-               
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       struct snd_timer_user *tu;
+
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        return (err = snd_timer_stop(tu->timeri)) < 0 ? err : 0;
 }
@@ -1633,80 +1757,112 @@ static int snd_timer_user_stop(struct file *file)
 static int snd_timer_user_continue(struct file *file)
 {
        int err;
-       snd_timer_user_t *tu;
-               
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       struct snd_timer_user *tu;
+
+       tu = file->private_data;
        snd_assert(tu->timeri != NULL, return -ENXIO);
        tu->timeri->lost = 0;
        return (err = snd_timer_continue(tu->timeri)) < 0 ? err : 0;
 }
 
-static int snd_timer_user_ioctl(struct inode *inode, struct file *file,
-                               unsigned int cmd, unsigned long arg)
+static int snd_timer_user_pause(struct file *file)
 {
-       snd_timer_user_t *tu;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+       int err;
+       struct snd_timer_user *tu;
+
+       tu = file->private_data;
+       snd_assert(tu->timeri != NULL, return -ENXIO);
+       return (err = snd_timer_pause(tu->timeri)) < 0 ? err : 0;
+}
+
+enum {
+       SNDRV_TIMER_IOCTL_START_OLD = _IO('T', 0x20),
+       SNDRV_TIMER_IOCTL_STOP_OLD = _IO('T', 0x21),
+       SNDRV_TIMER_IOCTL_CONTINUE_OLD = _IO('T', 0x22),
+       SNDRV_TIMER_IOCTL_PAUSE_OLD = _IO('T', 0x23),
+};
+
+static long snd_timer_user_ioctl(struct file *file, unsigned int cmd,
+                                unsigned long arg)
+{
+       struct snd_timer_user *tu;
+       void __user *argp = (void __user *)arg;
+       int __user *p = argp;
+
+       tu = file->private_data;
        switch (cmd) {
        case SNDRV_TIMER_IOCTL_PVERSION:
-               return put_user(SNDRV_TIMER_VERSION, (int *)arg) ? -EFAULT : 0;
+               return put_user(SNDRV_TIMER_VERSION, p) ? -EFAULT : 0;
        case SNDRV_TIMER_IOCTL_NEXT_DEVICE:
-               return snd_timer_user_next_device((snd_timer_id_t *)arg);
+               return snd_timer_user_next_device(argp);
        case SNDRV_TIMER_IOCTL_TREAD:
        {
                int xarg;
-               
-               if (tu->timeri)         /* too late */
+
+               mutex_lock(&tu->tread_sem);
+               if (tu->timeri) {       /* too late */
+                       mutex_unlock(&tu->tread_sem);
                        return -EBUSY;
-               if (get_user(xarg, (int *) arg))
+               }
+               if (get_user(xarg, p)) {
+                       mutex_unlock(&tu->tread_sem);
                        return -EFAULT;
+               }
                tu->tread = xarg ? 1 : 0;
+               mutex_unlock(&tu->tread_sem);
                return 0;
        }
        case SNDRV_TIMER_IOCTL_GINFO:
-               return snd_timer_user_ginfo(file, (snd_timer_ginfo_t *)arg);
+               return snd_timer_user_ginfo(file, argp);
        case SNDRV_TIMER_IOCTL_GPARAMS:
-               return snd_timer_user_gparams(file, (snd_timer_gparams_t *)arg);
+               return snd_timer_user_gparams(file, argp);
        case SNDRV_TIMER_IOCTL_GSTATUS:
-               return snd_timer_user_gstatus(file, (snd_timer_gstatus_t *)arg);
+               return snd_timer_user_gstatus(file, argp);
        case SNDRV_TIMER_IOCTL_SELECT:
-               return snd_timer_user_tselect(file, (snd_timer_select_t *)arg);
+               return snd_timer_user_tselect(file, argp);
        case SNDRV_TIMER_IOCTL_INFO:
-               return snd_timer_user_info(file, (snd_timer_info_t *)arg);
+               return snd_timer_user_info(file, argp);
        case SNDRV_TIMER_IOCTL_PARAMS:
-               return snd_timer_user_params(file, (snd_timer_params_t *)arg);
+               return snd_timer_user_params(file, argp);
        case SNDRV_TIMER_IOCTL_STATUS:
-               return snd_timer_user_status(file, (snd_timer_status_t *)arg);
+               return snd_timer_user_status(file, argp);
        case SNDRV_TIMER_IOCTL_START:
+       case SNDRV_TIMER_IOCTL_START_OLD:
                return snd_timer_user_start(file);
        case SNDRV_TIMER_IOCTL_STOP:
+       case SNDRV_TIMER_IOCTL_STOP_OLD:
                return snd_timer_user_stop(file);
        case SNDRV_TIMER_IOCTL_CONTINUE:
+       case SNDRV_TIMER_IOCTL_CONTINUE_OLD:
                return snd_timer_user_continue(file);
+       case SNDRV_TIMER_IOCTL_PAUSE:
+       case SNDRV_TIMER_IOCTL_PAUSE_OLD:
+               return snd_timer_user_pause(file);
        }
        return -ENOTTY;
 }
 
 static int snd_timer_user_fasync(int fd, struct file * file, int on)
 {
-       snd_timer_user_t *tu;
+       struct snd_timer_user *tu;
        int err;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
+
+       tu = file->private_data;
        err = fasync_helper(fd, file, on, &tu->fasync);
         if (err < 0)
                return err;
        return 0;
 }
 
-static ssize_t snd_timer_user_read(struct file *file, char *buffer, size_t count, loff_t *offset)
+static ssize_t snd_timer_user_read(struct file *file, char __user *buffer,
+                                  size_t count, loff_t *offset)
 {
-       snd_timer_user_t *tu;
+       struct snd_timer_user *tu;
        long result = 0, unit;
        int err = 0;
-       
-       tu = snd_magic_cast(snd_timer_user_t, file->private_data, return -ENXIO);
-       unit = tu->tread ? sizeof(snd_timer_tread_t) : sizeof(snd_timer_read_t);
+
+       tu = file->private_data;
+       unit = tu->tread ? sizeof(struct snd_timer_tread) : sizeof(struct snd_timer_read);
        spin_lock_irq(&tu->qlock);
        while ((long)count - result >= unit) {
                while (!tu->qused) {
@@ -1738,12 +1894,14 @@ static ssize_t snd_timer_user_read(struct file *file, char *buffer, size_t count
                        goto _error;
 
                if (tu->tread) {
-                       if (copy_to_user(buffer, &tu->tqueue[tu->qhead++], sizeof(snd_timer_tread_t))) {
+                       if (copy_to_user(buffer, &tu->tqueue[tu->qhead++],
+                                        sizeof(struct snd_timer_tread))) {
                                err = -EFAULT;
                                goto _error;
                        }
                } else {
-                       if (copy_to_user(buffer, &tu->queue[tu->qhead++], sizeof(snd_timer_read_t))) {
+                       if (copy_to_user(buffer, &tu->queue[tu->qhead++],
+                                        sizeof(struct snd_timer_read))) {
                                err = -EFAULT;
                                goto _error;
                        }
@@ -1765,12 +1923,12 @@ static ssize_t snd_timer_user_read(struct file *file, char *buffer, size_t count
 static unsigned int snd_timer_user_poll(struct file *file, poll_table * wait)
 {
         unsigned int mask;
-        snd_timer_user_t *tu;
+        struct snd_timer_user *tu;
 
-        tu = snd_magic_cast(snd_timer_user_t, file->private_data, return 0);
+        tu = file->private_data;
 
         poll_wait(file, &tu->qchange_sleep, wait);
-       
+
        mask = 0;
        if (tu->qused)
                mask |= POLLIN | POLLRDNORM;
@@ -1778,6 +1936,12 @@ static unsigned int snd_timer_user_poll(struct file *file, poll_table * wait)
        return mask;
 }
 
+#ifdef CONFIG_COMPAT
+#include "timer_compat.c"
+#else
+#define snd_timer_user_ioctl_compat    NULL
+#endif
+
 static struct file_operations snd_timer_f_ops =
 {
        .owner =        THIS_MODULE,
@@ -1785,44 +1949,32 @@ static struct file_operations snd_timer_f_ops =
        .open =         snd_timer_user_open,
        .release =      snd_timer_user_release,
        .poll =         snd_timer_user_poll,
-       .ioctl =        snd_timer_user_ioctl,
+       .unlocked_ioctl =       snd_timer_user_ioctl,
+       .compat_ioctl = snd_timer_user_ioctl_compat,
        .fasync =       snd_timer_user_fasync,
 };
 
-static snd_minor_t snd_timer_reg =
-{
-       .comment =      "timer",
-       .f_ops =        &snd_timer_f_ops,
-};
-
 /*
  *  ENTRY functions
  */
 
-static snd_info_entry_t *snd_timer_proc_entry = NULL;
-
 static int __init alsa_timer_init(void)
 {
        int err;
-       snd_info_entry_t *entry;
 
 #ifdef SNDRV_OSS_INFO_DEV_TIMERS
-       snd_oss_info_register(SNDRV_OSS_INFO_DEV_TIMERS, SNDRV_CARDS - 1, "system timer");
+       snd_oss_info_register(SNDRV_OSS_INFO_DEV_TIMERS, SNDRV_CARDS - 1,
+                             "system timer");
 #endif
-       if ((entry = snd_info_create_module_entry(THIS_MODULE, "timers", NULL)) != NULL) {
-               entry->c.text.read_size = SNDRV_TIMER_DEVICES * 128;
-               entry->c.text.read = snd_timer_proc_read;
-               if (snd_info_register(entry) < 0) {
-                       snd_info_free_entry(entry);
-                       entry = NULL;
-               }
-       }
-       snd_timer_proc_entry = entry;
+
        if ((err = snd_timer_register_system()) < 0)
-               snd_printk(KERN_ERR "unable to register system timer (%i)\n", err);
-       if ((err = snd_register_device(SNDRV_DEVICE_TYPE_TIMER,
-                                       NULL, 0, &snd_timer_reg, "timer"))<0)
-               snd_printk(KERN_ERR "unable to register timer device (%i)\n", err);
+               snd_printk(KERN_ERR "unable to register system timer (%i)\n",
+                          err);
+       if ((err = snd_register_device(SNDRV_DEVICE_TYPE_TIMER, NULL, 0,
+                                      &snd_timer_f_ops, NULL, "timer")) < 0)
+               snd_printk(KERN_ERR "unable to register timer device (%i)\n",
+                          err);
+       snd_timer_proc_init();
        return 0;
 }
 
@@ -1833,13 +1985,10 @@ static void __exit alsa_timer_exit(void)
        snd_unregister_device(SNDRV_DEVICE_TYPE_TIMER, NULL, 0);
        /* unregister the system timer */
        list_for_each_safe(p, n, &snd_timer_list) {
-               snd_timer_t *timer = (snd_timer_t *)list_entry(p, snd_timer_t, device_list);
+               struct snd_timer *timer = list_entry(p, struct snd_timer, device_list);
                snd_timer_unregister(timer);
        }
-       if (snd_timer_proc_entry) {
-               snd_info_unregister(snd_timer_proc_entry);
-               snd_timer_proc_entry = NULL;
-       }
+       snd_timer_proc_done();
 #ifdef SNDRV_OSS_INFO_DEV_TIMERS
        snd_oss_info_unregister(SNDRV_OSS_INFO_DEV_TIMERS, SNDRV_CARDS - 1);
 #endif
@@ -1848,18 +1997,6 @@ static void __exit alsa_timer_exit(void)
 module_init(alsa_timer_init)
 module_exit(alsa_timer_exit)
 
-#ifndef MODULE
-/* format is: snd-timer=timer_limit */
-
-static int __init alsa_timer_setup(char *str)
-{
-       (void)(get_option(&str,&timer_limit) == 2);
-       return 1;
-}
-
-__setup("snd-timer=", alsa_timer_setup);
-#endif /* ifndef MODULE */
-
 EXPORT_SYMBOL(snd_timer_open);
 EXPORT_SYMBOL(snd_timer_close);
 EXPORT_SYMBOL(snd_timer_resolution);
@@ -1874,4 +2011,3 @@ EXPORT_SYMBOL(snd_timer_global_free);
 EXPORT_SYMBOL(snd_timer_global_register);
 EXPORT_SYMBOL(snd_timer_global_unregister);
 EXPORT_SYMBOL(snd_timer_interrupt);
-EXPORT_SYMBOL(snd_timer_system_resolution);