Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1492311

Re: [PATCH 2/2] mfd: intel-lpss: Avoid resuming runtime-suspended lpss unnecessarily

Show all headers | View raw

Hi Oliver,
On Wed, Sep 28, 2016 at 09:42:54AM +0200, Oliver Neukum wrote:
> On Wed, 2016-09-28 at 11:28 +0800, Chen Yu wrote:
> > So first try is to use pm_request_resume() instead, to make the
> > runtime
> > resume process asynchronously. Unfortunately the asynchronous runtime
> > resume relies on pm_wq, which is freezed at early stage. So we choose
> > another method, that is to avoid resuming runtime-suspended devices,
> > if they are already runtime suspended. This is safe because for LPSS
> > driver, the runtime suspend and system suspend are of the same
> > hook - i.e., intel_lpss_suspend(). And moreover, this device is
> > neither runtime wakeup source nor system wakeup source.
> 
> I agree with the reasoning but I don't see the specificity to LPSS.
> Shouldn't this go into the core?
>
Thanks for your reply. Do you mean we should add the logic to pm core?
There is already one if the driver's
.prepare() returns a positive number(aka, RPM_SUSPENDED), then pm core will keep
this device in runtime-suspended during sleep.
Documentation/power/runtime_pm.txt:
"the PM core provides a mechanism allowing some coordination between
different levels of device hierarchy.  Namely, if a system suspend .prepare()
callback returns RPM_SUSPENDED for a device, that indicates to the PM core
that the device appears to be runtime-suspended and its state is fine, so it
may be left in runtime suspend provided that all of its descendants are also
left in runtime suspend.  If that happens, the PM core will not execute any
system suspend and resume callbacks for all of those devices, except for the
complete callback, which is then entirely responsible for handling the device
as appropriate. "

Thanks,
Yu

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] Define positive return value to RPM_SUSPEND for runtime-suspended devices Chen Yu <yu.c.chen@intel.com> - 2016-09-28 05:20 +0200
  [PATCH 1/2] PM / sleep: Return RPM_SUSPENDED to keep devices in runtime-suspended Chen Yu <yu.c.chen@intel.com> - 2016-09-28 05:20 +0200
    Re: [PATCH 1/2] PM / sleep: Return RPM_SUSPENDED to keep devices in runtime-suspended "Rafael J. Wysocki" <rafael@kernel.org> - 2016-09-28 13:50 +0200
      Re: [PATCH 1/2] PM / sleep: Return RPM_SUSPENDED to keep devices in  runtime-suspended Chen Yu <yu.c.chen@intel.com> - 2016-09-28 17:00 +0200
  [PATCH 2/2] mfd: intel-lpss: Avoid resuming runtime-suspended lpss unnecessarily Chen Yu <yu.c.chen@intel.com> - 2016-09-28 05:30 +0200
    Re: [PATCH 2/2] mfd: intel-lpss: Avoid resuming runtime-suspended  lpss unnecessarily Oliver Neukum <oneukum@suse.com> - 2016-09-28 09:50 +0200
      Re: [PATCH 2/2] mfd: intel-lpss: Avoid resuming runtime-suspended  lpss unnecessarily Chen Yu <yu.c.chen@intel.com> - 2016-09-28 10:10 +0200
        Re: [PATCH 2/2] mfd: intel-lpss: Avoid resuming runtime-suspended  lpss unnecessarily Oliver Neukum <oneukum@suse.com> - 2016-09-28 10:20 +0200

csiph-web