diff options
| author |   Lars-Peter Clausen <lars@metafoo.de> | 2015-11-21 12:01:22 +0100 |
|---|---|---|
| committer |   Takashi Iwai <tiwai@suse.de> | 2015-11-21 17:40:07 +0100 |
| commit | 7a7a2df434cec5614271666b84b2ea1f41048e91 (patch) | |
| tree | 3f8dcf95396557dd6183cd31e60df1442bd42dd6 /sound | |
| parent | ALSA: ua101: replace le16_to_cpu() with usb_endpoint_maxp() (diff) | |
| download | linux-dev-7a7a2df434cec5614271666b84b2ea1f41048e91.tar.xz linux-dev-7a7a2df434cec5614271666b84b2ea1f41048e91.zip | |
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>
Diffstat (limited to '')
| -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); |