On 13-06-24 11:31 PM, Viresh Kumar wrote: > On 25 June 2013 01:00, Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> 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 <viresh.kumar@xxxxxxxxxx> > > 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. -- > -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html