From patchwork Thu Jun 30 19:37:23 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rafael Wysocki X-Patchwork-Id: 933392 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.4) with ESMTP id p5UJZwP2023123 for ; Thu, 30 Jun 2011 19:36:46 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753892Ab1F3Tgp (ORCPT ); Thu, 30 Jun 2011 15:36:45 -0400 Received: from ogre.sisk.pl ([217.79.144.158]:44356 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753837Ab1F3Tgo (ORCPT ); Thu, 30 Jun 2011 15:36:44 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by ogre.sisk.pl (Postfix) with ESMTP id 2702E1AF867; Thu, 30 Jun 2011 21:13:12 +0200 (CEST) Received: from ogre.sisk.pl ([127.0.0.1]) by localhost (ogre.sisk.pl [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 11853-04; Thu, 30 Jun 2011 21:12:52 +0200 (CEST) Received: from ferrari.rjw.lan (220-bem-13.acn.waw.pl [82.210.184.220]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ogre.sisk.pl (Postfix) with ESMTP id AA4991B3933; Thu, 30 Jun 2011 21:12:52 +0200 (CEST) From: "Rafael J. Wysocki" To: Kevin Hilman Subject: Re: [PATCH 7/10 v6] PM / Domains: Don't stop wakeup devices during system sleep transitions Date: Thu, 30 Jun 2011 21:37:23 +0200 User-Agent: KMail/1.13.6 (Linux/3.0.0-rc5+; KDE/4.6.0; x86_64; ; ) Cc: Linux PM mailing list , "Greg Kroah-Hartman" , Magnus Damm , Paul Walmsley , Alan Stern , LKML , linux-sh@vger.kernel.org, Paul Mundt References: <201106112223.04972.rjw@sisk.pl> <201106252329.24342.rjw@sisk.pl> <87fwmsnqno.fsf@ti.com> In-Reply-To: <87fwmsnqno.fsf@ti.com> MIME-Version: 1.0 Message-Id: <201106302137.23801.rjw@sisk.pl> X-Virus-Scanned: amavisd-new at ogre.sisk.pl using MkS_Vir for Linux Sender: linux-sh-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-sh@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Thu, 30 Jun 2011 19:36:46 +0000 (UTC) On Thursday, June 30, 2011, Kevin Hilman wrote: > "Rafael J. Wysocki" writes: > > > From: Rafael J. Wysocki > > > > Devices that are set up to wake up the system from sleep states > > should not be stopped and power should not be removed from them > > when the system goes into a sleep state. > > I don't think this belongs in the generic layer since the two > assumptions above are not generally true on embedded systems, and would > result in rather significant power consumption unnecessarily. As to whether or not this belongs to the generic layer, I don't quite agree (see below), but the changelog seems to be a bit inaccurate. > First, whether the device should be stopped on device_may_wakeup(): > > Some IP blocks (at least on OMAP) have "asynchronous" wakeups. Meaning > that they can generate wakeups even when they're not clocked (a.k.a > stopped). So in this case, even after a ->stop_device (which clock > gates the IP), it can still generate wakeups. > > Second, whether the device should be powered off if device_may_wakeup(): > > Embedded SoCs have other ways to wakeup than device-level wakeups. > > For example, on OMAP, every pad on the SoC can be configured as a wakeup > source So, for example, you could completely power down the UART IP > blocks (and the enclosing power domain), configure the UART RX pad as a > wakeup source, and still wakeup the system on UART activity. The OMAP > docs call these IO pad wakeups. > > On OMAP in fact, this is the common, default behavior when we enable > "off-mode" in idle and/or suspend, since most of the IPs are powered off > but can still wake up the system. > > So in summary, even if device_may_wakeup() is true, many devices (with > additional SoC magic) can still generate wakeups even when stopped and > powered off. Well, on the other hand, on some SoCs there are devices that can't be powered off (or "declocked") if they are supposed to generate wakeups. Also, I'm sure there are cases in which wakeups can be generated for devices with their clocks off, but only if power is present. So there are multiple cases, but not so many overall. So, IMO, it makes sense to handle that at the generic level, although not necessarily in such a simplistic way. Now, at this point, I want to do something very simple, which I think is done by this patch. Is this optimal power comsumption-wise for every potential user of the framework? No, but certainly for some it's sufficient. Is it going to work in general? I think it is. Of course, there's the question how to handle that more accurately and I have some ideas. If you have any, please let me know. In the meantime, I'm going to modify the changelog so that it's clear that it's a "first approximation" thing, like in the patch below. Thanks, Rafael --- From: Rafael J. Wysocki Subject: PM / Domains: Don't stop wakeup devices during system sleep transitions There is the problem how to handle devices set up to wake up the system from sleep states during system-wide power transitions. In some cases, those devices can be turned off entirely, because the wakeup signals will be generated on their behalf anyway. In some other cases, they will generate wakeup signals if their clocks are stopped, but only if power is not removed from them. Finally, in some cases, they can only generate wakeup signals if power is not removed from them and their clocks are enabled. In the future, it will be necessary to take all of the above situations into account, but for starters it is possible to use the observation that if all wakeup devices are treated like the last group (i.e. their clocks are enabled and power in not removed from them during system suspend transitions), they all will be able to generate wakeups, although power consumption in the resulting system sleep state may not be optimal in some cases. Signed-off-by: Rafael J. Wysocki --- drivers/base/power/domain.c | 6 ++++++ 1 file changed, 6 insertions(+) -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Index: linux-2.6/drivers/base/power/domain.c =================================================================== --- linux-2.6.orig/drivers/base/power/domain.c +++ linux-2.6/drivers/base/power/domain.c @@ -450,6 +450,9 @@ static int pm_genpd_suspend_noirq(struct if (ret) return ret; + if (device_may_wakeup(dev)) + return 0; + if (genpd->stop_device) genpd->stop_device(dev); @@ -670,6 +673,9 @@ static int pm_genpd_dev_poweroff_noirq(s if (ret) return ret; + if (device_may_wakeup(dev)) + return 0; + if (genpd->stop_device) genpd->stop_device(dev);