diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2015-11-21 14:01:22 +0300 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-11-21 19:40:07 +0300 |
commit | 7a7a2df434cec5614271666b84b2ea1f41048e91 (patch) | |
tree | 3f8dcf95396557dd6183cd31e60df1442bd42dd6 | |
parent | 3c7a09358729e64119669f454fb1ac3c5cd20b63 (diff) | |
download | linux-7a7a2df434cec5614271666b84b2ea1f41048e91.tar.xz |
ALSA: azt3328: Remove unnecessary synchronize_irq() before free_irq()
Calling synchronize_irq() right before free_irq() is quite useless. On one
hand the IRQ can easily fire again before free_irq() is entered, on the
other hand free_irq() itself calls synchronize_irq() internally (in a race
condition free way) before any state associated with the IRQ is freed.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/pci/azt3328.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/pci/azt3328.c b/sound/pci/azt3328.c index 07a4acc99541..5e2ef0bb7057 100644 --- a/sound/pci/azt3328.c +++ b/sound/pci/azt3328.c @@ -2294,8 +2294,6 @@ snd_azf3328_free(struct snd_azf3328 *chip) snd_azf3328_timer_stop(chip->timer); snd_azf3328_gameport_free(chip); - if (chip->irq >= 0) - synchronize_irq(chip->irq); __end_hw: if (chip->irq >= 0) free_irq(chip->irq, chip); |