Commit cb04fc3b authored by Takashi Iwai's avatar Takashi Iwai

ALSA: hda/ca0132 - Avoid endless loop

Introduce a timeout to dspio_clear_response_queue() so that it won't
be caught in an endless loop even if the hardware doesn't respond
properly.

Fixes: a73d511c ("ALSA: hda/ca0132: Add unsol handler for DSP and jack detection")
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20191213085111.22855-3-tiwai@suse.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 377bc0cf
...@@ -1809,13 +1809,14 @@ struct scp_msg { ...@@ -1809,13 +1809,14 @@ struct scp_msg {
static void dspio_clear_response_queue(struct hda_codec *codec) static void dspio_clear_response_queue(struct hda_codec *codec)
{ {
unsigned long timeout = jiffies + msecs_to_jiffies(1000);
unsigned int dummy = 0; unsigned int dummy = 0;
int status = -1; int status;
/* clear all from the response queue */ /* clear all from the response queue */
do { do {
status = dspio_read(codec, &dummy); status = dspio_read(codec, &dummy);
} while (status == 0); } while (status == 0 && time_before(jiffies, timeout));
} }
static int dspio_get_response_data(struct hda_codec *codec) static int dspio_get_response_data(struct hda_codec *codec)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment