On Thu, Oct 30, 2014 at 01:46:43PM -0700, Kevin Hilman wrote: > Mark Brown <broonie@xxxxxxxxxx> writes: > > On Fri, Oct 24, 2014 at 09:12:39AM -0700, Kevin Hilman wrote: > >> I'm confused. Why arent' pm_runtime_get*() and pm_runtime_put*() working? > >> What's not explained here (or what I'm not understanding) is why a PM > >> domain is powering off if it has active devices. > > 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.
Attachment:
signature.asc
Description: Digital signature
