diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-29 19:00:30 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-10-29 19:00:30 -0400 |
commit | 34411a69a4925dd6de8e1069956f59d758da3ce9 (patch) | |
tree | b861bf6803e8468fb3cb3f663ac76d6d868877d5 /drivers/acpi/processor_perflib.c | |
parent | 764d022133979f9d9a179152cd89717e9752c61a (diff) |
ACPI / processor: Do not request ACPI cpufreq module directly
Function acpi_processor_load_module() used by the ACPI processor
driver can only really work if the acpi-cpufreq module is available
when acpi_processor_start() is executed which usually is not the case
for systems loading the processor driver module from an initramfs.
Moreover, that used to be a hackish workaround for module autoloading
issues, but udev loads acpi-cpufreq just fine nowadays, so that
function isn't really necessary any more. For this reason, drop
acpi_processor_load_module() entirely.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/processor_perflib.c')
-rw-r--r-- | drivers/acpi/processor_perflib.c | 22 |
1 files changed, 0 insertions, 22 deletions
diff --git a/drivers/acpi/processor_perflib.c b/drivers/acpi/processor_perflib.c index 51d7948611da..60a7c28fc167 100644 --- a/drivers/acpi/processor_perflib.c +++ b/drivers/acpi/processor_perflib.c | |||
@@ -235,28 +235,6 @@ void acpi_processor_ppc_exit(void) | |||
235 | acpi_processor_ppc_status &= ~PPC_REGISTERED; | 235 | acpi_processor_ppc_status &= ~PPC_REGISTERED; |
236 | } | 236 | } |
237 | 237 | ||
238 | /* | ||
239 | * Do a quick check if the systems looks like it should use ACPI | ||
240 | * cpufreq. We look at a _PCT method being available, but don't | ||
241 | * do a whole lot of sanity checks. | ||
242 | */ | ||
243 | void acpi_processor_load_module(struct acpi_processor *pr) | ||
244 | { | ||
245 | static int requested; | ||
246 | acpi_status status = 0; | ||
247 | struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL }; | ||
248 | |||
249 | if (!arch_has_acpi_pdc() || requested) | ||
250 | return; | ||
251 | status = acpi_evaluate_object(pr->handle, "_PCT", NULL, &buffer); | ||
252 | if (!ACPI_FAILURE(status)) { | ||
253 | printk(KERN_INFO PREFIX "Requesting acpi_cpufreq\n"); | ||
254 | request_module_nowait("acpi_cpufreq"); | ||
255 | requested = 1; | ||
256 | } | ||
257 | kfree(buffer.pointer); | ||
258 | } | ||
259 | |||
260 | static int acpi_processor_get_performance_control(struct acpi_processor *pr) | 238 | static int acpi_processor_get_performance_control(struct acpi_processor *pr) |
261 | { | 239 | { |
262 | int result = 0; | 240 | int result = 0; |