Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752375AbaB1XLV (ORCPT ); Fri, 28 Feb 2014 18:11:21 -0500 Received: from v094114.home.net.pl ([79.96.170.134]:54599 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751734AbaB1XLT (ORCPT ); Fri, 28 Feb 2014 18:11:19 -0500 From: "Rafael J. Wysocki" To: Viresh Kumar Cc: Saravana Kannan , "Srivatsa S. Bhat" , "cpufreq@vger.kernel.org" , "linux-pm@vger.kernel.org" , Linux Kernel Mailing List , linux-arm-msm@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH v3 1/3] cpufreq: stats: Remove redundant cpufreq_cpu_get() call Date: Sat, 01 Mar 2014 00:26:23 +0100 Message-ID: <8945263.cDODqBSQxb@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/3.13.0+; KDE/4.11.5; x86_64; ; ) In-Reply-To: References: <1393552716-9754-1-git-send-email-skannan@codeaurora.org> 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, February 28, 2014 09:18:02 AM Viresh Kumar wrote: > On 28 February 2014 07:28, Saravana Kannan wrote: > > __cpufreq_stats_create_table always gets pass the valid and real policy > > struct. So, there's no need to call cpufreq_cpu_get() to get the policy > > again. > > > > Acked-by: Viresh Kumar > > Signed-off-by: Saravana Kannan > > --- > > drivers/cpufreq/cpufreq_stats.c | 12 +----------- > > 1 file changed, 1 insertion(+), 11 deletions(-) > > All patches look fine now. Rafael you can queue them up as per your > convenience. > > @Saravana: Though I am quite sure you still need my patches to get > your initial issue fixed. So, can you please get them tested as early > as possible, so that we can get them in for 3.14? That's a bit late ... > @Srivatsa: Your reviews will also help :) -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/