From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] OMAP3: decrease cpufreq transition latency Date: Thu, 29 Oct 2009 14:14:27 -0700 Message-ID: <87k4ydzyvg.fsf@deeprootsystems.com> References: <1256850129-31618-1-git-send-email-mturquette@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-gx0-f216.google.com ([209.85.217.216]:59977 "EHLO mail-gx0-f216.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753159AbZJ2VO0 (ORCPT ); Thu, 29 Oct 2009 17:14:26 -0400 Received: by gxk8 with SMTP id 8so47347gxk.1 for ; Thu, 29 Oct 2009 14:14:31 -0700 (PDT) In-Reply-To: <1256850129-31618-1-git-send-email-mturquette@ti.com> (Mike Turquette's message of "Thu\, 29 Oct 2009 16\:02\:09 -0500") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Mike Turquette Cc: linux-omap@vger.kernel.org, Mike Turquette , Vishwa Sripathy Mike Turquette writes: > Adjust OMAP3 frequency transition latency from 10,000,000uS to a more > reasonable 300,000uS. This causes ondemand and conservative governors to > sample CPU load more often resulting in more responsive behavior. > > Tested on Android 2.6.29; using this value and conservative governor, CORE > power consumption on Zoom2 was comparable to the old and unresponsive > 10,000,000uS value while UI responsiveness was greatly improved. > > This patch applies against Tony's omap-fixes branch. > > Signed-off-by: Mike Turquette > --- > arch/arm/plat-omap/cpu-omap.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/plat-omap/cpu-omap.c b/arch/arm/plat-omap/cpu-omap.c > index 1868c0d..341235c 100644 > --- a/arch/arm/plat-omap/cpu-omap.c > +++ b/arch/arm/plat-omap/cpu-omap.c > @@ -127,7 +127,7 @@ static int __init omap_cpu_init(struct cpufreq_policy *policy) > } > > /* FIXME: what's the actual transition time? */ Can probably drop this 'FIXME' comment now. > - policy->cpuinfo.transition_latency = 10 * 1000 * 1000; > + policy->cpuinfo.transition_latency = 300 * 1000; otherwise, Signed-off-by: Kevin Hilman Tony, can this still be queued for .32-rc? Kevin