diff options
author | Denis Efremov <yefremov.denis@gmail.com> | 2013-02-11 10:49:48 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2013-02-11 10:52:46 -0500 |
commit | dacae5a19b4cbe1b5e3a86de23ea74cbe9ec9652 (patch) | |
tree | 64b108a9fc93221f56e3a9fec6b9b774ce98bf95 /sound/pci | |
parent | f49a59c4471d81a233e09dda45187cc44fda009d (diff) |
ALSA: ali5451: remove irq enabling in pointer callback
snd_ali_pointer function is called with local
interrupts disabled. However it seems very strange to
reenable them in such way.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Denis Efremov <yefremov.denis@gmail.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci')
-rw-r--r-- | sound/pci/ali5451/ali5451.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/ali5451/ali5451.c b/sound/pci/ali5451/ali5451.c index 136a393b70ab..e760af9d1fb6 100644 --- a/sound/pci/ali5451/ali5451.c +++ b/sound/pci/ali5451/ali5451.c | |||
@@ -1435,7 +1435,7 @@ static snd_pcm_uframes_t snd_ali_pointer(struct snd_pcm_substream *substream) | |||
1435 | 1435 | ||
1436 | spin_lock(&codec->reg_lock); | 1436 | spin_lock(&codec->reg_lock); |
1437 | if (!pvoice->running) { | 1437 | if (!pvoice->running) { |
1438 | spin_unlock_irq(&codec->reg_lock); | 1438 | spin_unlock(&codec->reg_lock); |
1439 | return 0; | 1439 | return 0; |
1440 | } | 1440 | } |
1441 | outb(pvoice->number, ALI_REG(codec, ALI_GC_CIR)); | 1441 | outb(pvoice->number, ALI_REG(codec, ALI_GC_CIR)); |