diff options
| author |   Harsha Priya <harshapriya.n@intel.com> | 2020-09-30 14:41:39 +0300 |
|---|---|---|
| committer |   Takashi Iwai <tiwai@suse.de> | 2020-09-30 16:52:48 +0200 |
| commit | a0645daf16101bb9a6d87598c17e9a8b7bd60ea7 (patch) | |
| tree | 59a61c0248b82ca63c4cca82ad45f1cb498c06d2 /sound/pci/hda/hda_codec.c | |
| parent | ALSA: hda - Don't register a cb func if it is registered already (diff) | |
| download | linux-dev-a0645daf16101bb9a6d87598c17e9a8b7bd60ea7.tar.xz linux-dev-a0645daf16101bb9a6d87598c17e9a8b7bd60ea7.zip | |
ALSA: HDA: Early Forbid of runtime PM
For certain codecs (like Realtek), pm_runtime_forbid() is invoked
in the probe function after build_controls(). In a stress test,
its observed occasionally that runtime PM calls are invoked
before controls are built. This causes the codec to be
runtime suspended before probe completes. Because of this, not all
controls are enumerated correctly, and audio does not work until
system is rebooted.
This issue being common across all codecs, pm_runtime_forbid() is
called when the codec object is created to fix this issue.
A codec enables or disables runtime pm in its own probe function.
Multiple stress tests of 2000+ cycles has been done to test the fix.
Signed-off-by: Harsha Priya <harshapriya.n@intel.com>
Signed-off-by: Emmanuel Jillela <emmanuel.jillela@intel.com>
Reviewed-by: Kailang Yang <kailang@realtek.com>
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Link: https://lore.kernel.org/r/20200930114140.3839617-2-kai.vehmanen@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/hda/hda_codec.c')
| -rw-r--r-- | sound/pci/hda/hda_codec.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c index e96a87f1b611..a356c21edb90 100644 --- a/sound/pci/hda/hda_codec.c +++ b/sound/pci/hda/hda_codec.c @@ -1000,6 +1000,9 @@ int snd_hda_codec_device_new(struct hda_bus *bus, struct snd_card *card, if (err < 0) goto error; + /* PM runtime needs to be enabled later after binding codec */ + pm_runtime_forbid(&codec->core.dev); + return 0; error: |