24 Jul
2020
24 Jul
'20
1:16 p.m.
On Fri, Jul 24, 2020 at 03:07:31PM +0800, Tzung-Bi Shih wrote:
Commit 4e872a46823c ("ASoC: dapm: Don't force card bias level to be updated") stops to force update bias_level on card. If card doesn't provide set_bias_level callback, the snd_soc_dapm_set_bias_level() is equivalent to NOP for card device.
As a result, dapm_pre_sequence_async() doesn't change the bias_level of card device correctly. Thus, pm_runtime_get_sync() would be called in dapm_pre_sequence_async() without symmetric pm_runtime_put() in dapm_post_sequence_async().
Don't call pm_runtime_* on card device.
Why is this a good fix, as opposed to only skipping the set_bias_level() bit?