From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756605Ab3LVAsN (ORCPT ); Sat, 21 Dec 2013 19:48:13 -0500 Received: from v094114.home.net.pl ([79.96.170.134]:60883 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1756269Ab3LVAsL (ORCPT ); Sat, 21 Dec 2013 19:48:11 -0500 From: "Rafael J. Wysocki" To: Ulf Hansson Cc: Len Brown , Pavel Machek , "linux-pm@vger.kernel.org" , Greg Kroah-Hartman , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Alan Stern , Kevin Hilman Subject: Re: [PATCH V2 0/3] PM: Enable option of re-use runtime PM callbacks at system suspend Date: Sun, 22 Dec 2013 02:01:35 +0100 Message-ID: <1670603.cXT40Mn7Pp@vostro.rjw.lan> User-Agent: KMail/4.10.5 (Linux/3.12.0-rc6+; KDE/4.10.5; x86_64; ; ) In-Reply-To: <1605004.ViTQsllvdk@vostro.rjw.lan> References: <1386682662-17633-1-git-send-email-ulf.hansson@linaro.org> <1605004.ViTQsllvdk@vostro.rjw.lan> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday, December 20, 2013 02:18:22 PM Rafael J. Wysocki wrote: > On Friday, December 20, 2013 09:43:16 AM Ulf Hansson wrote: > > On 10 December 2013 14:37, Ulf Hansson wrote: > > > To put devices into low power state during system suspend, it is convenient > > > for some runtime PM supported power domains, subsystems and drivers to have > > > the option of re-using and invoking the runtime PM callbacks from their system > > > PM callbacks. > > > > > > The benefit is that we don't need to implement wrapper functions which > > > handles runtime PM resourses, typically called from both runtime PM and system > > > PM callbacks. > > > > > > Some new helper macros for defining the PM callbacks are added. > > > > > > SET_PM_RUNTIME_PM_OPS; to define runtime PM callbacks for CONFIG_PM instead of > > > CONFIG_PM_RUNTIME. > > > > > > SET_LATE_SYSTEM_SLEEP_PM_OPS; to define common late/early system PM callbacks, > > > same approach is applicable to the existing SET_SYSTEM_SLEEP_PM_OPS which > > > defines common suspend/resume system PM callbacks. > > > > > > A minor fix was needed for the platform bus, which runtime PM callbacks are set > > > to the pm_generic_runtime_suspend|resume functions. These were implemented only > > > for CONFIG_PM_RUNTIME and thus the platform bus prevented driver's runtime PM > > > callbacks to be invoked when only CONFIG_PM_SLEEP was used. We move them into > > > CONFIG_PM to resolve the problem. > > > > > > Changes in v2: > > > Updated commit messages to try to better reflect the changes. > > > Removed the below patches from this patch set. Let's leave those to be > > > addressed separately, if needed. > > > - PM / Sleep: Add pm_generic functions to re-use runtime PM callbacks > > > - drm/exynos: Convert to suspend_late/resume_early callbacks for fimc > > > > > > > > > Ulf Hansson (3): > > > PM / Runtime: Add second macro for definition of runtime PM callbacks > > > PM / Runtime: Implement the pm_generic_runtime functions for > > > CONFIG_PM > > > PM / Sleep: Add macro to define common late/early system PM callbacks > > > > > > drivers/base/power/generic_ops.c | 4 ++-- > > > include/linux/pm.h | 21 +++++++++++++++++++++ > > > include/linux/pm_runtime.h | 12 ++++++++---- > > > 3 files changed, 31 insertions(+), 6 deletions(-) > > > > > > -- > > > 1.7.9.5 > > > > > > > Hi, > > > > Just wanted to send a kind ping on this patchset. > > > > Would be nice to try to conclude on the way forward after the quite > > long and good discussions we had around this topic. > > Sorry, I was traveling last week. I'm processing my backlog now, hopefully > I'll get to your patches shortly. So I've just queued them up for 3.14. Thanks! -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.