On Thu, Oct 30, 2014 at 01:46:43PM -0700, Kevin Hilman wrote:

Mark Brown [off-list ref] writes:

quoted

On Fri, Oct 24, 2014 at 09:12:39AM -0700, Kevin Hilman wrote:

quoted

quoted

I'm confused. Why arent' pm_runtime_get*() and pm_runtime_put*() working?

quoted

quoted

What's not explained here (or what I'm not understanding) is why a PM
domain is powering off if it has active devices.

quoted

The issue AIUI is what happens during system boot - if one device in a
domain probes and marks itself runtime idle then that will trigger
domain powerdown even if there is another device in the domain that
hasn't yet been probed.  This can cause undesirable glitches (or worse)
during boot depending on what's getting powered down.

I'm not quite seeing how this series fixes that problem.

Looking at platform devices in PATCH 4/9, the new _get() and _put() are
still happening around ->probe(), so if a platform device runtime suspends
after probe, don't we still have a PM domain that can turn off?

Yeah, I haven't actually reviewed the series so I don't know if it does
the above but that was my understanding of the intention.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20141030/54f67faa/attachment-0001.sig>