[Letux-kernel] [PATCHv2] i2c: omap: Use noirq system sleep pm ops to idle device for suspend

[Andreas Kemnade]

On Fri, 22 Feb 2019 06:54:06 -0800
Tony Lindgren <tony at atomide.com> wrote:

> Hi,
> 
> * Andreas Kemnade <andreas at kemnade.info> [190222 12:08]:
> > Hmm, so what do we know:
> > - git bisect at least led to something interesting, not some unrelated thing.
> >     which might just cause timing differences.  
> 
> Yes seems it's some twl related driver that gets only enabled for gta04
> with your .config.
> 
> > - rtcwake + actual wakeup via rtc on my config = problem
> > - rtcwake + actual wakeup via some other pin in the wkup domain = no problem
> > - rtcwake + actual wakeup via powerbutton (twl4030-pwrbutton) = problem
> > - omap2plus_defconfig seems to work here
> > - no problems at other systems with dm3730 + twl4030 with my config
> >   and with omap2plus defconfig
> > 
> > I deduce that there happens something strange at resume independant
> > from what happend at suspend. Probably some timing issue.  
> 
> It's probaby some twl related code is (wrongly) trying to read/write
> registers at noirq suspend level. Sounds like some driver tagged with
> noirq suspend ops while it should not.
> 
static irqreturn_t handle_twl4030_pih(int irq, void *devid)
in
drivers/mfd/twl4030-irq.c
is doing the failed i2c access.

Regards,
Andreas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.goldelico.com/pipermail/letux-kernel/attachments/20190222/40b05c6d/attachment.asc>