diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2013-08-06 13:23:03 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-08-07 17:34:09 -0400 |
commit | 5ff0a268037d344f86df690ccb994d8bc015d2d9 (patch) | |
tree | 63930dc6092b9654d3e5be9b3db52aa29438c8be | |
parent | 1133bfa6dcf034639486982629d29472737d5e96 (diff) |
cpufreq: Clean up header files included in the core
This patch addresses the following issues in the header files in the
cpufreq core:
- Include headers in ascending order, so that we don't add same
many times by mistake.
- <asm/> must be included after <linux/>, so that they override
whatever they need to.
- Remove unnecessary includes.
- Don't include files already included by cpufreq.h or
cpufreq_governor.h.
[rjw: Changelog]
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/cpufreq/cpufreq.c | 19 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_conservative.c | 12 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_governor.c | 6 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_governor.h | 5 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_ondemand.c | 12 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_performance.c | 3 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_powersave.c | 3 | ||||
-rw-r--r-- | drivers/cpufreq/cpufreq_stats.c | 9 | ||||
-rw-r--r-- | drivers/cpufreq/freq_table.c | 4 | ||||
-rw-r--r-- | include/linux/cpufreq.h | 11 |
10 files changed, 16 insertions, 68 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index e34bd94e12b4..845837174878 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c | |||
@@ -17,24 +17,17 @@ | |||
17 | 17 | ||
18 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 18 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
19 | 19 | ||
20 | #include <asm/cputime.h> | 20 | #include <linux/cpu.h> |
21 | #include <linux/kernel.h> | ||
22 | #include <linux/kernel_stat.h> | ||
23 | #include <linux/module.h> | ||
24 | #include <linux/init.h> | ||
25 | #include <linux/notifier.h> | ||
26 | #include <linux/cpufreq.h> | 21 | #include <linux/cpufreq.h> |
27 | #include <linux/delay.h> | 22 | #include <linux/delay.h> |
28 | #include <linux/interrupt.h> | ||
29 | #include <linux/spinlock.h> | ||
30 | #include <linux/tick.h> | ||
31 | #include <linux/device.h> | 23 | #include <linux/device.h> |
32 | #include <linux/slab.h> | 24 | #include <linux/init.h> |
33 | #include <linux/cpu.h> | 25 | #include <linux/kernel_stat.h> |
34 | #include <linux/completion.h> | 26 | #include <linux/module.h> |
35 | #include <linux/mutex.h> | 27 | #include <linux/mutex.h> |
28 | #include <linux/slab.h> | ||
36 | #include <linux/syscore_ops.h> | 29 | #include <linux/syscore_ops.h> |
37 | 30 | #include <linux/tick.h> | |
38 | #include <trace/events/power.h> | 31 | #include <trace/events/power.h> |
39 | 32 | ||
40 | /** | 33 | /** |
diff --git a/drivers/cpufreq/cpufreq_conservative.c b/drivers/cpufreq/cpufreq_conservative.c index 0ceb2eff5a7e..841e25699656 100644 --- a/drivers/cpufreq/cpufreq_conservative.c +++ b/drivers/cpufreq/cpufreq_conservative.c | |||
@@ -11,19 +11,7 @@ | |||
11 | * published by the Free Software Foundation. | 11 | * published by the Free Software Foundation. |
12 | */ | 12 | */ |
13 | 13 | ||
14 | #include <linux/cpufreq.h> | ||
15 | #include <linux/init.h> | ||
16 | #include <linux/kernel.h> | ||
17 | #include <linux/kernel_stat.h> | ||
18 | #include <linux/kobject.h> | ||
19 | #include <linux/module.h> | ||
20 | #include <linux/mutex.h> | ||
21 | #include <linux/notifier.h> | ||
22 | #include <linux/percpu-defs.h> | ||
23 | #include <linux/slab.h> | 14 | #include <linux/slab.h> |
24 | #include <linux/sysfs.h> | ||
25 | #include <linux/types.h> | ||
26 | |||
27 | #include "cpufreq_governor.h" | 15 | #include "cpufreq_governor.h" |
28 | 16 | ||
29 | /* Conservative governor macros */ | 17 | /* Conservative governor macros */ |
diff --git a/drivers/cpufreq/cpufreq_governor.c b/drivers/cpufreq/cpufreq_governor.c index 7409dbd1d897..556064e9cc01 100644 --- a/drivers/cpufreq/cpufreq_governor.c +++ b/drivers/cpufreq/cpufreq_governor.c | |||
@@ -16,15 +16,9 @@ | |||
16 | 16 | ||
17 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 17 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
18 | 18 | ||
19 | #include <asm/cputime.h> | ||
20 | #include <linux/cpufreq.h> | ||
21 | #include <linux/cpumask.h> | ||
22 | #include <linux/export.h> | 19 | #include <linux/export.h> |
23 | #include <linux/kernel_stat.h> | 20 | #include <linux/kernel_stat.h> |
24 | #include <linux/mutex.h> | ||
25 | #include <linux/slab.h> | 21 | #include <linux/slab.h> |
26 | #include <linux/types.h> | ||
27 | #include <linux/workqueue.h> | ||
28 | 22 | ||
29 | #include "cpufreq_governor.h" | 23 | #include "cpufreq_governor.h" |
30 | 24 | ||
diff --git a/drivers/cpufreq/cpufreq_governor.h b/drivers/cpufreq/cpufreq_governor.h index 0e0dd4c82020..cf0b7a4425b0 100644 --- a/drivers/cpufreq/cpufreq_governor.h +++ b/drivers/cpufreq/cpufreq_governor.h | |||
@@ -18,10 +18,9 @@ | |||
18 | #define _CPUFREQ_GOVERNOR_H | 18 | #define _CPUFREQ_GOVERNOR_H |
19 | 19 | ||
20 | #include <linux/cpufreq.h> | 20 | #include <linux/cpufreq.h> |
21 | #include <linux/kobject.h> | 21 | #include <linux/kernel_stat.h> |
22 | #include <linux/module.h> | ||
22 | #include <linux/mutex.h> | 23 | #include <linux/mutex.h> |
23 | #include <linux/workqueue.h> | ||
24 | #include <linux/sysfs.h> | ||
25 | 24 | ||
26 | /* | 25 | /* |
27 | * The polling frequency depends on the capability of the processor. Default | 26 | * The polling frequency depends on the capability of the processor. Default |
diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c index a3c5574f9b3a..27c732eee431 100644 --- a/drivers/cpufreq/cpufreq_ondemand.c +++ b/drivers/cpufreq/cpufreq_ondemand.c | |||
@@ -12,20 +12,10 @@ | |||
12 | 12 | ||
13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
14 | 14 | ||
15 | #include <linux/cpufreq.h> | 15 | #include <linux/cpu.h> |
16 | #include <linux/init.h> | ||
17 | #include <linux/kernel.h> | ||
18 | #include <linux/kernel_stat.h> | ||
19 | #include <linux/kobject.h> | ||
20 | #include <linux/module.h> | ||
21 | #include <linux/mutex.h> | ||
22 | #include <linux/percpu-defs.h> | 16 | #include <linux/percpu-defs.h> |
23 | #include <linux/slab.h> | 17 | #include <linux/slab.h> |
24 | #include <linux/sysfs.h> | ||
25 | #include <linux/tick.h> | 18 | #include <linux/tick.h> |
26 | #include <linux/types.h> | ||
27 | #include <linux/cpu.h> | ||
28 | |||
29 | #include "cpufreq_governor.h" | 19 | #include "cpufreq_governor.h" |
30 | 20 | ||
31 | /* On-demand governor macros */ | 21 | /* On-demand governor macros */ |
diff --git a/drivers/cpufreq/cpufreq_performance.c b/drivers/cpufreq/cpufreq_performance.c index 9fef7d6e4e6a..cf117deb39b1 100644 --- a/drivers/cpufreq/cpufreq_performance.c +++ b/drivers/cpufreq/cpufreq_performance.c | |||
@@ -12,10 +12,9 @@ | |||
12 | 12 | ||
13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
14 | 14 | ||
15 | #include <linux/kernel.h> | ||
16 | #include <linux/module.h> | ||
17 | #include <linux/cpufreq.h> | 15 | #include <linux/cpufreq.h> |
18 | #include <linux/init.h> | 16 | #include <linux/init.h> |
17 | #include <linux/module.h> | ||
19 | 18 | ||
20 | static int cpufreq_governor_performance(struct cpufreq_policy *policy, | 19 | static int cpufreq_governor_performance(struct cpufreq_policy *policy, |
21 | unsigned int event) | 20 | unsigned int event) |
diff --git a/drivers/cpufreq/cpufreq_powersave.c b/drivers/cpufreq/cpufreq_powersave.c index 32109a14f5dc..e3b874c235ea 100644 --- a/drivers/cpufreq/cpufreq_powersave.c +++ b/drivers/cpufreq/cpufreq_powersave.c | |||
@@ -12,10 +12,9 @@ | |||
12 | 12 | ||
13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 13 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
14 | 14 | ||
15 | #include <linux/kernel.h> | ||
16 | #include <linux/module.h> | ||
17 | #include <linux/cpufreq.h> | 15 | #include <linux/cpufreq.h> |
18 | #include <linux/init.h> | 16 | #include <linux/init.h> |
17 | #include <linux/module.h> | ||
19 | 18 | ||
20 | static int cpufreq_governor_powersave(struct cpufreq_policy *policy, | 19 | static int cpufreq_governor_powersave(struct cpufreq_policy *policy, |
21 | unsigned int event) | 20 | unsigned int event) |
diff --git a/drivers/cpufreq/cpufreq_stats.c b/drivers/cpufreq/cpufreq_stats.c index cb3841355454..a7143b0597af 100644 --- a/drivers/cpufreq/cpufreq_stats.c +++ b/drivers/cpufreq/cpufreq_stats.c | |||
@@ -9,17 +9,10 @@ | |||
9 | * published by the Free Software Foundation. | 9 | * published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/kernel.h> | ||
13 | #include <linux/slab.h> | ||
14 | #include <linux/cpu.h> | 12 | #include <linux/cpu.h> |
15 | #include <linux/sysfs.h> | ||
16 | #include <linux/cpufreq.h> | 13 | #include <linux/cpufreq.h> |
17 | #include <linux/module.h> | 14 | #include <linux/module.h> |
18 | #include <linux/jiffies.h> | 15 | #include <linux/slab.h> |
19 | #include <linux/percpu.h> | ||
20 | #include <linux/kobject.h> | ||
21 | #include <linux/spinlock.h> | ||
22 | #include <linux/notifier.h> | ||
23 | #include <asm/cputime.h> | 16 | #include <asm/cputime.h> |
24 | 17 | ||
25 | static spinlock_t cpufreq_stats_lock; | 18 | static spinlock_t cpufreq_stats_lock; |
diff --git a/drivers/cpufreq/freq_table.c b/drivers/cpufreq/freq_table.c index f0d87412cc91..f111454a7aea 100644 --- a/drivers/cpufreq/freq_table.c +++ b/drivers/cpufreq/freq_table.c | |||
@@ -11,10 +11,8 @@ | |||
11 | 11 | ||
12 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | 12 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
13 | 13 | ||
14 | #include <linux/kernel.h> | ||
15 | #include <linux/module.h> | ||
16 | #include <linux/init.h> | ||
17 | #include <linux/cpufreq.h> | 14 | #include <linux/cpufreq.h> |
15 | #include <linux/module.h> | ||
18 | 16 | ||
19 | /********************************************************************* | 17 | /********************************************************************* |
20 | * FREQUENCY TABLE HELPERS * | 18 | * FREQUENCY TABLE HELPERS * |
diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h index e1fd215e16c9..97627bbbca82 100644 --- a/include/linux/cpufreq.h +++ b/include/linux/cpufreq.h | |||
@@ -11,16 +11,11 @@ | |||
11 | #ifndef _LINUX_CPUFREQ_H | 11 | #ifndef _LINUX_CPUFREQ_H |
12 | #define _LINUX_CPUFREQ_H | 12 | #define _LINUX_CPUFREQ_H |
13 | 13 | ||
14 | #include <asm/cputime.h> | 14 | #include <linux/cpumask.h> |
15 | #include <linux/mutex.h> | 15 | #include <linux/completion.h> |
16 | #include <linux/notifier.h> | ||
17 | #include <linux/threads.h> | ||
18 | #include <linux/kobject.h> | 16 | #include <linux/kobject.h> |
17 | #include <linux/notifier.h> | ||
19 | #include <linux/sysfs.h> | 18 | #include <linux/sysfs.h> |
20 | #include <linux/completion.h> | ||
21 | #include <linux/workqueue.h> | ||
22 | #include <linux/cpumask.h> | ||
23 | #include <asm/div64.h> | ||
24 | 19 | ||
25 | #define CPUFREQ_NAME_LEN 16 | 20 | #define CPUFREQ_NAME_LEN 16 |
26 | /* Print length for names. Extra 1 space for accomodating '\n' in prints */ | 21 | /* Print length for names. Extra 1 space for accomodating '\n' in prints */ |