diff options
| author | Daniel Lezcano <daniel.lezcano@linaro.org> | 2012-09-17 23:01:56 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rjw@sisk.pl> | 2012-09-17 23:01:56 +0200 |
| commit | 3d339dcbb56d8d70c1b959aff87d74adc3a84eea (patch) | |
| tree | ffa3dc8414d49c4cb638c221fd27bc65cb9b1aa1 /include/acpi | |
| parent | ed1511b80c92b1b1a8dde567adc090e470a4344a (diff) | |
| download | op-kernel-dev-3d339dcbb56d8d70c1b959aff87d74adc3a84eea.zip op-kernel-dev-3d339dcbb56d8d70c1b959aff87d74adc3a84eea.tar.gz | |
cpuidle / ACPI : move cpuidle_device field out of the acpi_processor_power structure
Currently we have the cpuidle_device field in the acpi_processor_power structure.
This adds a dependency between processor.h and cpuidle.h
Although it is not a real problem, removing this dependency has the benefit of
separating a bit more the cpuidle code from the rest of the acpi code.
Also, the compilation should be a bit improved because we do no longer
include cpuidle.h in processor.h. The preprocessor was generating 30418 loc
and with this patch it generates 30256 loc for processor_thermal.c, a file
which is not concerned at all by cpuidle, like processor_perflib.c and
processor_throttling.c.
That may sound ridiculous, but "small streams make big rivers" :P
This patch moves this field into a static global per cpu variable like what is
done in the intel_idle driver.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Diffstat (limited to 'include/acpi')
| -rw-r--r-- | include/acpi/processor.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/acpi/processor.h b/include/acpi/processor.h index 1d3c1a6..555d033 100644 --- a/include/acpi/processor.h +++ b/include/acpi/processor.h @@ -3,7 +3,6 @@ #include <linux/kernel.h> #include <linux/cpu.h> -#include <linux/cpuidle.h> #include <linux/thermal.h> #include <asm/acpi.h> @@ -64,7 +63,6 @@ struct acpi_processor_cx { }; struct acpi_processor_power { - struct cpuidle_device dev; struct acpi_processor_cx *state; unsigned long bm_check_timestamp; u32 default_state; |
