Re: [PATCH 1/2] mmc: enable mmc host device to suspend/resume asynchronously
On 1/12/2016 10:42 PM, Ulf Hansson wrote: > On 28 December 2015 at 16:39, Fu, Zhonghui > wrote: >> Now, PM core supports asynchronous suspend/resume mode for devices >> during system suspend/resume, and the power state transition of one >> device may be completed in separate kernel thread. PM core ensures >> all power state transition dependency between devices. This patch >> enables mmc hosts to suspend/resume asynchronously. This will take >> advantage of multicore and improve system suspend/resume speed. >> After applying this patch and enabling all mmc hosts' child devices >> to suspend/resume asynchronously on ASUS T100TA, the system >> suspend-to-idle time is reduced from 1645ms to 1107ms, and the >> system resume time is reduced from 940ms to 914ms. > Please update the change log as I don't think the above is really correct. > > I think you can simplify the change log quite a bit and just mention > what and why we want this change. I have updated the change log according to your comments and resent this patch - "[PATCH 1/2 v2] mmc: enable mmc host device to suspend/resume asynchronously". Thanks, Zhonghui > >> Signed-off-by: Zhonghui Fu >> --- >> drivers/mmc/core/host.c |1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c >> index da950c4..7222fd7 100644 >> --- a/drivers/mmc/core/host.c >> +++ b/drivers/mmc/core/host.c >> @@ -339,6 +339,7 @@ struct mmc_host *mmc_alloc_host(int extra, struct device >> *dev) >> host->class_dev.parent = dev; >> host->class_dev.class = &mmc_host_class; >> device_initialize(&host->class_dev); >> + device_enable_async_suspend(&host->class_dev); >> >> if (mmc_gpio_alloc(host)) { >> put_device(&host->class_dev); >> -- 1.7.1 >> > Otherwise I think this looks good! > > Kind regards > Uffe
RE: [PATCH 1/2] mmc: enable mmc host device to suspend/resume asynchronously
Looks good to me. Acked-by: Venu Byravarasu BR, Venu -Original Message- From: Fu, Zhonghui [mailto:zhonghui...@linux.intel.com] Sent: Monday, December 28, 2015 9:09 PM To: Ulf Hansson Cc: Adrian Hunter; chaotian.j...@mediatek.com; l...@metafoo.de; Venu Byravarasu; sergei.shtyl...@cogentembedded.com; linux-mmc; linux-kernel@vger.kernel.org Subject: [PATCH 1/2] mmc: enable mmc host device to suspend/resume asynchronously Now, PM core supports asynchronous suspend/resume mode for devices during system suspend/resume, and the power state transition of one device may be completed in separate kernel thread. PM core ensures all power state transition dependency between devices. This patch enables mmc hosts to suspend/resume asynchronously. This will take advantage of multicore and improve system suspend/resume speed. After applying this patch and enabling all mmc hosts' child devices to suspend/resume asynchronously on ASUS T100TA, the system suspend-to-idle time is reduced from 1645ms to 1107ms, and the system resume time is reduced from 940ms to 914ms. Signed-off-by: Zhonghui Fu --- drivers/mmc/core/host.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c index da950c4..7222fd7 100644 --- a/drivers/mmc/core/host.c +++ b/drivers/mmc/core/host.c @@ -339,6 +339,7 @@ struct mmc_host *mmc_alloc_host(int extra, struct device *dev) host->class_dev.parent = dev; host->class_dev.class = &mmc_host_class; device_initialize(&host->class_dev); + device_enable_async_suspend(&host->class_dev); if (mmc_gpio_alloc(host)) { put_device(&host->class_dev); -- 1.7.1
[PATCH 1/2] mmc: enable mmc host device to suspend/resume asynchronously
Now, PM core supports asynchronous suspend/resume mode for devices during system suspend/resume, and the power state transition of one device may be completed in separate kernel thread. PM core ensures all power state transition dependency between devices. This patch enables mmc hosts to suspend/resume asynchronously. This will take advantage of multicore and improve system suspend/resume speed. After applying this patch and enabling all mmc hosts' child devices to suspend/resume asynchronously on ASUS T100TA, the system suspend-to-idle time is reduced from 1645ms to 1107ms, and the system resume time is reduced from 940ms to 914ms. Signed-off-by: Zhonghui Fu --- drivers/mmc/core/host.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c index da950c4..7222fd7 100644 --- a/drivers/mmc/core/host.c +++ b/drivers/mmc/core/host.c @@ -339,6 +339,7 @@ struct mmc_host *mmc_alloc_host(int extra, struct device *dev) host->class_dev.parent = dev; host->class_dev.class = &mmc_host_class; device_initialize(&host->class_dev); + device_enable_async_suspend(&host->class_dev); if (mmc_gpio_alloc(host)) { put_device(&host->class_dev); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/