From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Gortmaker Subject: Re: [PATCH 25/32] cpufreq: delete __cpuinit usage from all cpufreq files Date: Tue, 25 Jun 2013 10:30:42 -0400 Message-ID: <51C9A992.7010209@windriver.com> References: <1372102237-8757-1-git-send-email-paul.gortmaker@windriver.com> <1372102237-8757-26-git-send-email-paul.gortmaker@windriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: cpufreq-owner@vger.kernel.org To: Viresh Kumar Cc: linux-kernel@vger.kernel.org, "Rafael J. Wysocki" , cpufreq@vger.kernel.org, linux-pm@vger.kernel.org List-Id: linux-pm@vger.kernel.org On 13-06-24 11:31 PM, Viresh Kumar wrote: > On 25 June 2013 01:00, Paul Gortmaker wrote: > >> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c >> index f8c2860..5687d28 100644 >> --- a/drivers/cpufreq/cpufreq.c >> +++ b/drivers/cpufreq/cpufreq.c >> @@ -1900,8 +1900,8 @@ no_policy: >> } >> EXPORT_SYMBOL(cpufreq_update_policy); >> >> -static int __cpuinit cpufreq_cpu_callback(struct notifier_block *nfb, >> - unsigned long action, void *hcpu) >> +static int cpufreq_cpu_callback(struct notifier_block *nfb, >> + unsigned long action, void *hcpu) > > You were not required to change second line here and also don't > change its indentation level. Check this everywhere. > [...] Since you've explicitly requested it, I've left all second lines of args untouched in v2, now pushed to the patch queue: http://git.kernel.org/cgit/linux/kernel/git/paulg/cpuinit-delete.git > > Fix these and add my > Acked-by: Viresh Kumar > > Probably this can go through Rafael's pm tree. So, you can take it > out of this series if you want. I'd encourage people to not take the patches into their specific trees unless they really feel a specific need to do so. It will keep the majority of them clumped together in git history that way. Thanks, Paul. -- >