summaryrefslogtreecommitdiffstats
path: root/include/sound
diff options
context:
space:
mode:
authorHui Wang <hui.wang@canonical.com>2021-11-30 10:06:06 +0100
committerMark Brown <broonie@kernel.org>2021-11-30 14:08:03 +0100
commit046aede2f847676f93a2ea4f48b77909c51dba40 (patch)
tree77226f8077ef6583af1a0e8c0830f7311baf8d7f /include/sound
parentASoC: amd: fix uninitialized variable in snd_acp6x_probe() (diff)
downloadlinux-046aede2f847676f93a2ea4f48b77909c51dba40.tar.xz
linux-046aede2f847676f93a2ea4f48b77909c51dba40.zip
ASoC: SOF: Intel: Retry codec probing if it fails
On the latest Lenovo Thinkstation laptops, we often experience the speaker failure after rebooting, check the dmesg, we could see: sof-audio-pci-intel-tgl 0000:00:1f.3: codec #0 probe error, ret: -5 The analogue codec on the machine is ALC287, then we designed a testcase to reboot and check the codec probing result repeatedly, we found the analogue codec probing always failed at least once within several minutes to several hours (roughly 1 reboot per min). This issue happens on all laptops of this Thinkstation model, but with legacy HDA driver, we couldn't reproduce this issue on those laptops. And so far, this issue is not reproduced on machines which don't belong to this model. We tried to make the hda_dsp_ctrl_init_chip() same as hda_intel_init_chip() which is the controller init routine in the legacy HDA driver, but it didn't help. We found when issue happens, the resp is -1, and if we let driver re-run send_cmd() and get_response(), it will get the correct response 10ec0287, then driver continues the rest work, finally boot to the desktop and all audio function work well. Here adding codec probing retries to 3 times, it could fix the issue on this Thinkstation model, and it doesn't bring impact to other machines. Reviewed-by: Bard Liao <bard.liao@intel.com> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Hui Wang <hui.wang@canonical.com> Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com> Link: https://lore.kernel.org/r/20211130090606.529348-1-kai.vehmanen@linux.intel.com Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include/sound')
0 files changed, 0 insertions, 0 deletions