In azx_[rirb_]get_response(), the timeout is checked at the end of the loop.
It's better to be checked just after the check of the RIRB index to avoid
a bogus error with a too long msleep().
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
again:
timeout = jiffies + msecs_to_jiffies(1000);
- do {
+ for (;;) {
if (chip->polling_mode) {
spin_lock_irq(&chip->reg_lock);
azx_update_rirb(chip);
}
if (!chip->rirb.cmds)
return chip->rirb.res; /* the last value */
+ if (time_after(jiffies, timeout))
+ break;
if (codec->bus->needs_damn_long_delay)
msleep(2); /* temporary workaround */
else {
udelay(10);
cond_resched();
}
- } while (time_after_eq(timeout, jiffies));
+ }
if (chip->msi) {
snd_printk(KERN_WARNING "hda_intel: No response from codec, "