> -----Original Message----- > From: Mark Brown [mailto:broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx] > Sent: Wednesday, February 24, 2010 7:59 AM > To: Olaya, Margarita > Cc: alsa-devel@xxxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx; lrg@xxxxxxxxxxxxxxx > Subject: Re: [PATCHv4 7/7] ASoC: TWL6030: Detect power-up sequence completion > > On Tue, Feb 23, 2010 at 06:10:54PM -0600, Olaya, Margarita wrote: > > > + if (naudint) { > > + /* wait for ready interrupt with 48 ms timeout */ > > + time_left = wait_for_completion_timeout(&priv->ready, > > + msecs_to_jiffies(48)); > > + } else { > > + /* retry 3 times only */ > > + for (time_left = 3; time_left > 0; time_left--) { > > + mdelay(16); > > + twl_i2c_read_u8(TWL4030_MODULE_AUDIO_VOICE, &intid, > > + TWL6030_REG_INTID); > > + if (intid & TWL6030_READYINT) > > + break; > > + } > > + } > > It strikes me that you could combine these two cases - the > wait_for_completion_timeout() will function just as well as a delay. > I'd also expect to see an error reported if the device doesn't report as > ready one way or another. It is split to prevent the case of none valid irq line connected, in such case, wait_for_completion won't work. Regards, Margarita -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
