On 01-06-18, 14:05, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The function acpi_cpufreq_fast_switch is local to the source and does > not need to be in global scope, so make it static. > > Cleans up sparse warning: > drivers/cpufreq/acpi-cpufreq.c:468:14: warning: symbol > 'acpi_cpufreq_fast_switch' was not declared. Should it be static? > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > --- > drivers/cpufreq/acpi-cpufreq.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/cpufreq/acpi-cpufreq.c b/drivers/cpufreq/acpi-cpufreq.c > index 9449657d72f0..32ba4bc972e7 100644 > --- a/drivers/cpufreq/acpi-cpufreq.c > +++ b/drivers/cpufreq/acpi-cpufreq.c > @@ -465,8 +465,8 @@ static int acpi_cpufreq_target(struct cpufreq_policy *policy, > return result; > } > > -unsigned int acpi_cpufreq_fast_switch(struct cpufreq_policy *policy, > - unsigned int target_freq) > +static unsigned int acpi_cpufreq_fast_switch(struct cpufreq_policy *policy, > + unsigned int target_freq) > { > struct acpi_cpufreq_data *data = policy->driver_data; > struct acpi_processor_performance *perf; Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> -- viresh -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html