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
History
File Mode Size
governors
Kconfig -rw-r--r-- 1.0 KB
Kconfig.arm -rw-r--r-- 2.4 KB
Kconfig.mips -rw-r--r-- 618 bytes
Kconfig.powerpc -rw-r--r-- 480 bytes
Makefile -rw-r--r-- 1.3 KB
coupled.c -rw-r--r-- 25.6 KB
cpuidle-arm.c -rw-r--r-- 3.9 KB
cpuidle-at91.c -rw-r--r-- 1.6 KB
cpuidle-big_little.c -rw-r--r-- 6.7 KB
cpuidle-calxeda.c -rw-r--r-- 2.2 KB
cpuidle-clps711x.c -rw-r--r-- 1.5 KB
cpuidle-cps.c -rw-r--r-- 4.4 KB
cpuidle-exynos.c -rw-r--r-- 3.4 KB
cpuidle-kirkwood.c -rw-r--r-- 2.3 KB
cpuidle-mvebu-v7.c -rw-r--r-- 3.3 KB
cpuidle-powernv.c -rw-r--r-- 10.6 KB
cpuidle-pseries.c -rw-r--r-- 6.7 KB
cpuidle-ux500.c -rw-r--r-- 3.2 KB
cpuidle-zynq.c -rw-r--r-- 2.0 KB
cpuidle.c -rw-r--r-- 17.0 KB
cpuidle.h -rw-r--r-- 2.1 KB
driver.c -rw-r--r-- 8.7 KB
dt_idle_states.c -rw-r--r-- 6.5 KB
dt_idle_states.h -rw-r--r-- 216 bytes
governor.c -rw-r--r-- 2.6 KB
poll_state.c -rw-r--r-- 1.4 KB
sysfs.c -rw-r--r-- 18.3 KB

back to top