Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0d105394 authored by Viresh Kumar's avatar Viresh Kumar Committed by Rafael J. Wysocki
Browse files

cpufreq: e_powersaver: Don't validate the frequency table twice 



The cpufreq core is already validating the CPU frequency table after
calling the ->init() callback of the cpufreq drivers and the drivers
don't need to do the same anymore. Though they need to set the
policy->freq_table field directly from the ->init() callback now.

Stop validating the frequency table from e_powersaver driver.

Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 29aa18a7

drivers/cpufreq/e_powersaver.c

+1 −7
Original line number Diff line number Diff line
@@ -184,7 +184,6 @@ static int eps_cpu_init(struct cpufreq_policy *policy) 
	struct cpuinfo_x86 *c = &cpu_data(0);

	struct cpufreq_frequency_table *f_table;

	int k, step, voltage;

	int ret;

	int states;

#if IS_ENABLED(CONFIG_ACPI_PROCESSOR)

	unsigned int limit;
@@ -359,12 +358,7 @@ static int eps_cpu_init(struct cpufreq_policy *policy) 
	}



	policy->cpuinfo.transition_latency = 140000; /* 844mV -> 700mV in ns */



	ret = cpufreq_table_validate_and_show(policy, &centaur->freq_table[0]);

	if (ret) {

		kfree(centaur);

		return ret;

	}

	policy->freq_table = &centaur->freq_table[0];



	return 0;

}