Commit 589a7c40 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

cpufreq: Drop unused cpufreq_get_policy() 



A recent change has introduced a bug into cpufreq_get_policy(), but this
function is not used, so it's better to drop it altogether.

Reported-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Acked-by: default avatarSudeep Holla <sudeep.holla@arm.com>
Link: https://patch.msgid.link/2802770.mvXUDI8C0e@rjwysocki.net
parent eaff6b62

drivers/cpufreq/cpufreq.c

+0 −25
Original line number Diff line number Diff line
@@ -2549,31 +2549,6 @@ EXPORT_SYMBOL_GPL(cpufreq_unregister_governor); 
 *                          POLICY INTERFACE                         *

 *********************************************************************/



/**

 * cpufreq_get_policy - get the current cpufreq_policy

 * @policy: struct cpufreq_policy into which the current cpufreq_policy

 *	is written

 * @cpu: CPU to find the policy for

 *

 * Reads the current cpufreq policy.

 */

int cpufreq_get_policy(struct cpufreq_policy *policy, unsigned int cpu)

{

	struct cpufreq_policy *cpu_policy __free(put_cpufreq_policy);



	if (!policy)

		return -EINVAL;



	cpu_policy = cpufreq_cpu_get(cpu);

	if (!cpu_policy)

		return -EINVAL;



	memcpy(policy, cpu_policy, sizeof(*policy));



	return 0;

}

EXPORT_SYMBOL(cpufreq_get_policy);



DEFINE_PER_CPU(unsigned long, cpufreq_pressure);



/**

include/linux/cpufreq.h

+0 −1
Original line number Diff line number Diff line
@@ -241,7 +241,6 @@ void disable_cpufreq(void); 


u64 get_cpu_idle_time(unsigned int cpu, u64 *wall, int io_busy);



int cpufreq_get_policy(struct cpufreq_policy *policy, unsigned int cpu);

void refresh_frequency_limits(struct cpufreq_policy *policy);

void cpufreq_update_policy(unsigned int cpu);

void cpufreq_update_limits(unsigned int cpu);