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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 22 Feb 2019 06:54:06 -0800
Tony Lindgren <tony@xxxxxxxxxxx> wrote:

> Hi,
> 
> * Andreas Kemnade <andreas@xxxxxxxxxxxx> [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

Attachment: pgpY75R4TTClV.pgp
Description: OpenPGP digital signature


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux