Re: [alsa-devel] [PATCH] mfd: arizona: Call the runtime PM function if the state is runtime resumed

From: Charles Keepax
Date: Tue Sep 22 2015 - 04:10:50 EST


On Mon, Sep 21, 2015 at 11:16:42AM +0900, Inha Song wrote:
> Hi, Charles,
>
> I've already tried to change.
> If I change to that, we can't enter the suspen during the playback.
>
> -
> [ 72.538263] arizona spi1.0: Suspend, disabling IRQ
> ...
> [ 72.585823] arizona spi1.0: Late suspend, reengabling IRQ
> [ 72.585904] arizona spi1.0: Early resume, disabling IRQ
> ...
> [ 72.646770] PM: noirq suspend of devices failed
> <- because of "spi1.0" pm_wakeup_pending() in suspend_noirq()

Ok this looks more reasonable, suspend should fail because we do
have an IRQ waiting to be handled, so we shouldn't suspend. But
you are saying this happens every time the CODEC goes to suspend,
thus blocking the suspend from suspending?

First step then would be to identify what IRQ is occurring, could
you set LOG_DEVICE at the top of drivers/base/regmap/regmap.c to
"spi1.0" and provide a log, so we can see what IRQ is being
asserted from the CODEC.

Thanks,
Charles

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/