From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Srivatsa S. Bhat" Subject: Re: [PATCH V4 3/3] cpufreq: Make cpufreq_notify_transition & cpufreq_notify_post_transition static Date: Fri, 21 Mar 2014 13:21:30 +0530 Message-ID: <532BEF82.1080700@linux.vnet.ibm.com> References: <827ab1ca9caaa50ed672c5e01806314c06ad8db8.1395379422.git.viresh.kumar@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <827ab1ca9caaa50ed672c5e01806314c06ad8db8.1395379422.git.viresh.kumar@linaro.org> Sender: cpufreq-owner@vger.kernel.org To: Viresh Kumar Cc: rjw@rjwysocki.net, linaro-kernel@lists.linaro.org, cpufreq@vger.kernel.org, linux-pm@vger.kernel.org, "linux-kernel@vger.kernel.org" , "ego@linux.vnet.ibm.com" List-Id: linux-pm@vger.kernel.org On 03/21/2014 11:04 AM, Viresh Kumar wrote: > cpufreq_notify_transition() and cpufreq_notify_post_transition() shouldn't be > called directly by cpufreq drivers anymore and so these should be marked static. > > Signed-off-by: Viresh Kumar Reviewed-by: Srivatsa S. Bhat Regards, Srivatsa S. Bhat > --- > drivers/cpufreq/cpufreq.c | 6 ++---- > include/linux/cpufreq.h | 4 ---- > 2 files changed, 2 insertions(+), 8 deletions(-) > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index b63e7e4..7b1feff 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -331,16 +331,15 @@ static void __cpufreq_notify_transition(struct cpufreq_policy *policy, > * function. It is called twice on all CPU frequency changes that have > * external effects. > */ > -void cpufreq_notify_transition(struct cpufreq_policy *policy, > +static void cpufreq_notify_transition(struct cpufreq_policy *policy, > struct cpufreq_freqs *freqs, unsigned int state) > { > for_each_cpu(freqs->cpu, policy->cpus) > __cpufreq_notify_transition(policy, freqs, state); > } > -EXPORT_SYMBOL_GPL(cpufreq_notify_transition); > > /* Do post notifications when there are chances that transition has failed */ > -void cpufreq_notify_post_transition(struct cpufreq_policy *policy, > +static void cpufreq_notify_post_transition(struct cpufreq_policy *policy, > struct cpufreq_freqs *freqs, int transition_failed) > { > cpufreq_notify_transition(policy, freqs, CPUFREQ_POSTCHANGE); > @@ -351,7 +350,6 @@ void cpufreq_notify_post_transition(struct cpufreq_policy *policy, > cpufreq_notify_transition(policy, freqs, CPUFREQ_PRECHANGE); > cpufreq_notify_transition(policy, freqs, CPUFREQ_POSTCHANGE); > } > -EXPORT_SYMBOL_GPL(cpufreq_notify_post_transition); > > void cpufreq_freq_transition_begin(struct cpufreq_policy *policy, > struct cpufreq_freqs *freqs) > diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h > index 263173d..826830b 100644 > --- a/include/linux/cpufreq.h > +++ b/include/linux/cpufreq.h > @@ -338,10 +338,6 @@ static inline void cpufreq_resume(void) {} > int cpufreq_register_notifier(struct notifier_block *nb, unsigned int list); > int cpufreq_unregister_notifier(struct notifier_block *nb, unsigned int list); > > -void cpufreq_notify_transition(struct cpufreq_policy *policy, > - struct cpufreq_freqs *freqs, unsigned int state); > -void cpufreq_notify_post_transition(struct cpufreq_policy *policy, > - struct cpufreq_freqs *freqs, int transition_failed); > void cpufreq_freq_transition_begin(struct cpufreq_policy *policy, > struct cpufreq_freqs *freqs); > void cpufreq_freq_transition_end(struct cpufreq_policy *policy, >