diff options
| author | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-01-08 14:46:37 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-01-08 14:46:37 -0500 |
| commit | d238998fbfa49f30b02f0a5de5294ca53c58348c (patch) | |
| tree | 6be58353f26552342b2fb8bec3e5cb35dedcb0ce | |
| parent | b59f8197c5ddd0d5d74b663650be5449dacd34aa (diff) | |
Revert "hda_intel suspend latency: shorten codec read"
This reverts commit 57a04513cb35086d54bcb2cb92e6627fc8fa0fae.
Harald Dunkel reports that it broke sound for him:
"Alsa stopped working for me. I still can access /dev/dsp, change the
volume and so on, but the speakers are quiet."
Reverting it fixed things for him.
Reported-and-tested-by: Harald Dunkel <harald.dunkel@t-online.de>
Acked-by: Takashi Iwai <tiwai@suse.de>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | sound/pci/hda/hda_intel.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 62b9fb386903..3fa0f9704909 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c | |||
| @@ -555,8 +555,7 @@ static unsigned int azx_rirb_get_response(struct hda_codec *codec) | |||
| 555 | } | 555 | } |
| 556 | if (!chip->rirb.cmds) | 556 | if (!chip->rirb.cmds) |
| 557 | return chip->rirb.res; /* the last value */ | 557 | return chip->rirb.res; /* the last value */ |
| 558 | udelay(10); | 558 | schedule_timeout_uninterruptible(1); |
| 559 | cond_resched(); | ||
| 560 | } while (time_after_eq(timeout, jiffies)); | 559 | } while (time_after_eq(timeout, jiffies)); |
| 561 | 560 | ||
| 562 | if (chip->msi) { | 561 | if (chip->msi) { |