diff options
author | Sameer Pujar <spujar@nvidia.com> | 2018-12-26 16:04:49 +0530 |
---|---|---|
committer | Ben Hutchings <ben@decadent.org.uk> | 2019-04-04 16:14:10 +0100 |
commit | 43cbb7c1e73072f70be348263d33a266ce839b00 (patch) | |
tree | ff72c9eb5805405e7d01c85d03076050245f74a0 | |
parent | 61b77ece53982b821849f49fcd64968a77787388 (diff) | |
download | linux-stable-43cbb7c1e73072f70be348263d33a266ce839b00.tar.gz linux-stable-43cbb7c1e73072f70be348263d33a266ce839b00.tar.bz2 linux-stable-43cbb7c1e73072f70be348263d33a266ce839b00.zip |
ALSA: hda/tegra: clear pending irq handlers
commit 63d2a9ec310d8bcc955574220d4631aa55c1a80c upstream.
Even after disabling interrupts on the module, it could be possible
that irq handlers are still running. System hang is seen during
suspend path. It was found that, there were pending writes on the
HDA bus and clock was disabled by that time.
Above mentioned issue is fixed by clearing any pending irq handlers
before disabling clocks and returning from hda suspend.
Suggested-by: Mohan Kumar <mkumard@nvidia.com>
Suggested-by: Dara Ramesh <dramesh@nvidia.com>
Signed-off-by: Sameer Pujar <spujar@nvidia.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
[bwh: Backported to 3.16:
- Use azx::irq instead of hdac_bus::irq
- Adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
-rw-r--r-- | sound/pci/hda/hda_tegra.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/sound/pci/hda/hda_tegra.c b/sound/pci/hda/hda_tegra.c index 358414da6418..44953a273246 100644 --- a/sound/pci/hda/hda_tegra.c +++ b/sound/pci/hda/hda_tegra.c @@ -261,6 +261,7 @@ static int hda_tegra_suspend(struct device *dev) snd_hda_suspend(chip->bus); azx_stop_chip(chip); + synchronize_irq(chip->irq); azx_enter_link_reset(chip); hda_tegra_disable_clocks(hda); |