lkml.org 
[lkml]   [2018]   [Jan]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC] intel-lpss: remove .prepare() callback
From
Date
On Thu, 2018-01-25 at 16:12 +0800, Zhang Rui wrote:
> The .prepare() callback of intel-lpss driver does nothing but wakes up
> its
> children. I don't know if there is any reason to do so, but to me,
> this is
> not preferred because it should be the child device driver to do so
> when
> necessary, not the parent device driver.
> Plus, .prepare() does not support asynchronization.
>
> For example, on MS Surface Pro 4, there are 4 intel-lpss devices which
> are runtime suspended before system suspend, resuming each of them
> takes
> more than 100 milliseconds. Thus the .prepare() of intel-lpss driver
> takes
> 400ms+ on my surface pro 4, and I've seen platforms with 16 intel lpss
> devices.
>
> With this patch applied, the child devices are resumed in the
> .suspend()
> stage of the child device, and they are done in parallel, thus only
> 100ms
> is needed, no matter how many intel lpss devices there are.
>
> I have tested it on three different platforms and didn't find any
> obvious
> problem caused by this patch, and it indeed reduces the suspend time a
> lot.

> @@ -482,17 +482,6 @@ static int resume_lpss_device(struct device *dev,
> - device_for_each_child_reverse(dev, NULL, resume_lpss_device);

Besides introduced compiler warning, did you check the latest linux-pm
changes?

commit 8425ec7faff005500aad89b9fc00e5ba91ac57b9
Author: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Date: Wed Jan 3 01:34:53 2018 +0100

PM / mfd: intel-lpss: Use DPM_FLAG_SMART_SUSPEND


--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

\
 
 \ /
  Last update: 2018-01-25 13:29    [W:0.051 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site