diff options
| author | Takashi Iwai <tiwai@suse.de> | 2013-06-06 16:20:19 +0400 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2013-06-06 16:20:19 +0400 |
| commit | 63e51fd708f511a5989da04c669647993bc1a512 (patch) | |
| tree | eaa604d5faae416c0a51b5cdba78d625ca62cf2b /sound/pci/hda/hda_intel.c | |
| parent | e7ecc27e520a9c9891362c6dabd18c4da9885946 (diff) | |
| download | linux-63e51fd708f511a5989da04c669647993bc1a512.tar.xz | |
ALSA: hda - Don't take unresponsive D3 transition too serious
When a codec is powered off, some systems don't respond properly after
D3 FG transition, while the driver still expects the response and
tries to fall back to different modes (polling and single-cmd). When
the fallback happens, the driver stays in that mode, and falling back
to the single-cmd mode means it'll loose the unsol event handling,
too.
The unresponsiveness at D3 isn't too serious, thus this fallback is
mostly superfluous. We can gracefully ignore the error there so that
the driver keeps the normal operation mode.
This patch adds a new bit flag for codec read/write, set in the power
transition stage, which is notified to the controller driver via a new
bus->no_response_fallback flag.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/hda/hda_intel.c')
| -rw-r--r-- | sound/pci/hda/hda_intel.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 49dfad4a099e..f089fa0aa03d 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -942,6 +942,9 @@ static unsigned int azx_rirb_get_response(struct hda_bus *bus, } } + if (!bus->no_response_fallback) + return -1; + if (!chip->polling_mode && chip->poll_count < 2) { snd_printdd(SFX "%s: azx_get_response timeout, " "polling the codec once: last cmd=0x%08x\n", |