Hi, On Thu, Oct 11, 2012 at 02:08:25PM -0700, Kevin Hilman wrote: > Hi Kalle, > > Kalle Jokiniemi <kalle.jokiniemi@xxxxxxxxxxxxxxx> writes: > > > The resume_noirq enables interrupts one-by-one starting from > > first one. Now if the wake up event for suspend came from i2c > > device, the i2c bus irq gets enabled before the threaded > > i2c device irq, causing a flood of i2c bus interrupts as the > > threaded irq that should clear the event is not enabled yet. > > > > Fixed the issue by adding suspend_noirq and resume_early > > functions that keep i2c bus interrupts disabled until > > resume_noirq has run completely. > > > > Issue was detected doing a wake up from autosleep with > > twl4030 power key on N9. Patch tested on N9. > > > > Signed-off-by: Kalle Jokiniemi <kalle.jokiniemi@xxxxxxxxxxxxxxx> > > This version looks good, thanks for the extra comments. > > Reviewed-by: Kevin Hilman <khilman@xxxxxx> > Tested-by: Kevin Hilman <khilman@xxxxxx> > > Wolfram, This should also probably be Cc'd to stable since it affects > earlier kernels as well. Thanks, just to make sure we're not fixing the wrong problem... does [1] help in any way ? [1] http://marc.info/?l=linux-omap&m=135048839915719&w=2 -- balbi
Attachment:
signature.asc
Description: Digital signature
