From patchwork Tue Jun 7 20:57:52 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Todd Poynor X-Patchwork-Id: 859672 X-Patchwork-Delegate: khilman@deeprootsystems.com Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.3) with ESMTP id p57Kw4QG031633 for ; Tue, 7 Jun 2011 20:58:04 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757267Ab1FGU6D (ORCPT ); Tue, 7 Jun 2011 16:58:03 -0400 Received: from smtp-out.google.com ([74.125.121.67]:2025 "EHLO smtp-out.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757262Ab1FGU6B (ORCPT ); Tue, 7 Jun 2011 16:58:01 -0400 Received: from kpbe17.cbf.corp.google.com (kpbe17.cbf.corp.google.com [172.25.105.81]) by smtp-out.google.com with ESMTP id p57KvsGY010242; Tue, 7 Jun 2011 13:57:55 -0700 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=google.com; s=beta; t=1307480275; bh=oTJAQINsTJB1A8uco7D8ypesjEI=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type; b=oAGkZ9TFAIwJb+rZNTDAOxIKESPO+rVHrB32AyMlj5es+/9cTVIuOZqr1pu1QZP6a CkcaeF8ILnOBKpEXAoZGg== DomainKey-Signature: a=rsa-sha1; s=beta; d=google.com; c=nofws; q=dns; h=date:from:to:cc:subject:message-id:mime-version: content-type:content-disposition:user-agent:x-system-of-record; b=v+px5OqNGiIA+lZ6W+EhdVQdBbULuEXO4PrMkib+B9BbdpPMMoWl5YtqgVnRllZSE 1zorZ4nDttffQ+s1YYqCg== Received: from conslugarocko.mtv.corp.google.com (conslugarocko.mtv.corp.google.com [172.18.102.26]) by kpbe17.cbf.corp.google.com with ESMTP id p57Kvrr1014649; Tue, 7 Jun 2011 13:57:53 -0700 Received: by conslugarocko.mtv.corp.google.com (Postfix, from userid 115684) id F30D9234324; Tue, 7 Jun 2011 13:57:52 -0700 (PDT) Date: Tue, 7 Jun 2011 13:57:52 -0700 From: Todd Poynor To: Kevin Hilman Cc: Santosh Shilimkar , Nishanth Menon , Mike Turquette , linux-omap@vger.kernel.org Subject: [PATCH pm_wip/cpufreq] OMAP2+: cpufreq: Enable all CPUs in shared policy mask Message-ID: <20110607205752.GA11299@google.com> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.20 (2009-06-14) X-System-Of-Record: true Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@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]); Tue, 07 Jun 2011 20:58:04 +0000 (UTC) Enable all CPUs in the shared policy in the CPU init callback. Otherwise, the governor CPUFREQ_GOV_START event is invoked with a policy that only includes the first CPU, leaving other CPUs uninitialized by the governor. Signed-off-by: Todd Poynor Acked-by: Santosh Shilimkar --- arch/arm/mach-omap2/omap2plus-cpufreq.c | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-omap2/omap2plus-cpufreq.c b/arch/arm/mach-omap2/omap2plus-cpufreq.c index 77efcb0..0fe4edb 100644 --- a/arch/arm/mach-omap2/omap2plus-cpufreq.c +++ b/arch/arm/mach-omap2/omap2plus-cpufreq.c @@ -166,7 +166,6 @@ static inline void freq_table_free(void) static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy) { int result = 0; - static cpumask_var_t cpumask; mpu_clk = clk_get(NULL, mpu_clk_name); if (IS_ERR(mpu_clk)) @@ -207,8 +206,7 @@ static int __cpuinit omap_cpu_init(struct cpufreq_policy *policy) */ if (is_smp()) { policy->shared_type = CPUFREQ_SHARED_TYPE_ANY; - cpumask_or(cpumask, cpumask_of(policy->cpu), cpumask); - cpumask_copy(policy->cpus, cpumask); + cpumask_setall(policy->cpus); } /* FIXME: what's the actual transition time? */