Revision 744c67ffeb06f2d2493f4049ba0bd19698ce0adf authored by Hui Wang on 19 March 2019, 01:28:43 UTC, committed by Takashi Iwai on 19 March 2019, 05:52:19 UTC
The commit 3baffc4a84d7 (ALSA: hda/intel: Refactoring PM code) changed
the behaviour of azx_resume(), it triggers the jackpoll_work after
applying this commit.
This change introduced a new issue, all codecs are runtime active
after S3, and will not call runtime_suspend() automatically.
The root cause is the jackpoll_work calls snd_hda_power_up/down_pm,
and it calls up_pm before snd_hdac_enter_pm is called, while calls
the down_pm in the middle of enter_pm and leave_pm is called. This
makes the dev->power.usage_count unbalanced after S3.
To fix it, let azx_resume() don't trigger jackpoll_work as before
it did.
Fixes: 3baffc4a84d7 ("ALSA: hda/intel: Refactoring PM code")
Signed-off-by: Hui Wang <hui.wang@canonical.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
1 parent b4748e7
| File | Mode | Size |
|---|---|---|
| Kconfig | -rw-r--r-- | 3.8 KB |
| Makefile | -rw-r--r-- | 779 bytes |
| badrange.c | -rw-r--r-- | 8.1 KB |
| blk.c | -rw-r--r-- | 8.7 KB |
| btt.c | -rw-r--r-- | 43.0 KB |
| btt.h | -rw-r--r-- | 7.6 KB |
| btt_devs.c | -rw-r--r-- | 9.0 KB |
| bus.c | -rw-r--r-- | 29.0 KB |
| claim.c | -rw-r--r-- | 8.0 KB |
| core.c | -rw-r--r-- | 10.7 KB |
| dax_devs.c | -rw-r--r-- | 3.3 KB |
| dimm.c | -rw-r--r-- | 3.5 KB |
| dimm_devs.c | -rw-r--r-- | 22.5 KB |
| e820.c | -rw-r--r-- | 2.3 KB |
| label.c | -rw-r--r-- | 35.1 KB |
| label.h | -rw-r--r-- | 4.5 KB |
| namespace_devs.c | -rw-r--r-- | 65.1 KB |
| nd-core.h | -rw-r--r-- | 6.8 KB |
| nd.h | -rw-r--r-- | 11.7 KB |
| of_pmem.c | -rw-r--r-- | 2.8 KB |
| pfn.h | -rw-r--r-- | 1.6 KB |
| pfn_devs.c | -rw-r--r-- | 20.8 KB |
| pmem.c | -rw-r--r-- | 15.5 KB |
| pmem.h | -rw-r--r-- | 1.1 KB |
| region.c | -rw-r--r-- | 4.5 KB |
| region_devs.c | -rw-r--r-- | 31.5 KB |
| security.c | -rw-r--r-- | 11.4 KB |
Computing file changes ...
