ALSA: opl3: Use setup_timer() and mod_timer()

No functional change, refactoring with the standard helpers.

Signed-off-by: Takashi Iwai <tiwai@suse.de>

2 files changed, 5 insertions, 10 deletions

diff --git a/sound/drivers/opl3/opl3_midi.c b/sound/drivers/opl3/opl3_midi.c
index 6c6d09a51f42..f62780ed64ad 100644
--- a/sound/drivers/opl3/opl3_midi.c
+++ b/sound/drivers/opl3/opl3_midi.c
@@ -258,12 +258,10 @@ void snd_opl3_timer_func(unsigned long data)
         spin_unlock_irqrestore(&opl3->voice_lock, flags);
 
         spin_lock_irqsave(&opl3->sys_timer_lock, flags);
-        if (again) {
-                opl3->tlist.expires = jiffies + 1;      /* invoke again */
-                add_timer(&opl3->tlist);
-        } else {
+        if (again)
+                mod_timer(&opl3->tlist, jiffies + 1);   /* invoke again */
+        else
                 opl3->sys_timer_status = 0;
-        }
         spin_unlock_irqrestore(&opl3->sys_timer_lock, flags);
 }
 
@@ -275,8 +273,7 @@ static void snd_opl3_start_timer(struct snd_opl3 *opl3)
         unsigned long flags;
         spin_lock_irqsave(&opl3->sys_timer_lock, flags);
         if (! opl3->sys_timer_status) {
-                opl3->tlist.expires = jiffies + 1;
-                add_timer(&opl3->tlist);
+                mod_timer(&opl3->tlist, jiffies + 1);
                 opl3->sys_timer_status = 1;
         }
         spin_unlock_irqrestore(&opl3->sys_timer_lock, flags);
diff --git a/sound/drivers/opl3/opl3_seq.c b/sound/drivers/opl3/opl3_seq.c
index 68399538e435..a9f618e06a22 100644
--- a/sound/drivers/opl3/opl3_seq.c
+++ b/sound/drivers/opl3/opl3_seq.c
@@ -247,9 +247,7 @@ static int snd_opl3_seq_new_device(struct snd_seq_device *dev)
         }
 
         /* setup system timer */
-        init_timer(&opl3->tlist);
-        opl3->tlist.function = snd_opl3_timer_func;
-        opl3->tlist.data = (unsigned long) opl3;
+        setup_timer(&opl3->tlist, snd_opl3_timer_func, (unsigned long) opl3);
         spin_lock_init(&opl3->sys_timer_lock);
         opl3->sys_timer_status = 0;