RE: [alsa-devel] [PATCHv4 7/7] ASoC: TWL6030: Detect power-up sequence completion

2010-03-01 Thread Olaya, Margarita
On Monday, March 01, 2010 6:14 AM Mark Brown wrote: > On Fri, Feb 26, 2010 at 06:22:34PM -0600, Olaya, Margarita wrote: > >> Do you mean something like this? >> time_left = wait_for_completion_timeout(&priv->ready, >> msecs_to_jiffies(48)); >> if(!ti

Re: [alsa-devel] [PATCHv4 7/7] ASoC: TWL6030: Detect power-up sequence completion

2010-03-01 Thread Mark Brown
On Fri, Feb 26, 2010 at 06:22:34PM -0600, Olaya, Margarita wrote: > Do you mean something like this? > time_left = wait_for_completion_timeout(&priv->ready, > msecs_to_jiffies(48)); > if(!time_left) { > twl_i2c_read_u8(TWL4030_MODULE_AUDIO_VOICE, &inti

RE: [alsa-devel] [PATCHv4 7/7] ASoC: TWL6030: Detect power-up sequence completion

2010-02-26 Thread Olaya, Margarita
On Friday, February 26, 2010 3:27 PM Mark Brown wrote: > On Fri, Feb 26, 2010 at 03:04:22PM -0600, Olaya, Margarita wrote: > >> In both cases if READYINIT is not set before the timeout runs out >> it means the codec is not powering on and the driver reports an >> error. > > That's kind of my

Re: [alsa-devel] [PATCHv4 7/7] ASoC: TWL6030: Detect power-up sequence completion

2010-02-26 Thread Mark Brown
On Fri, Feb 26, 2010 at 03:04:22PM -0600, Olaya, Margarita wrote: > In both cases if READYINIT is not set before the timeout runs out it means > the codec is not powering on and the driver reports an error. That's kind of my point - because you're checking for the same status there shouldn't be a