diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-10-06 12:12:25 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-10-13 12:11:28 -0400 |
commit | b1b3f49ce4606452279b58b17f2bbe2ba00304b7 (patch) | |
tree | c9cfdc72dd580a9c3ace71846af45b4f65268f46 /arch/arm/mach-shmobile/Kconfig | |
parent | 4f1cd91497774488ed16119ec3f54b3daf1561de (diff) |
ARM: config: sort select statements alphanumerically
As suggested by Andrew Morton:
This is a pet peeve of mine. Any time there's a long list of items
(header file inclusions, kconfig entries, array initalisers, etc) and
someone wants to add a new item, they *always* go and stick it at the
end of the list.
Guys, don't do this. Either put the new item into a randomly-chosen
position or, probably better, alphanumerically sort the list.
lets sort all our select statements alphanumerically. This commit was
created by the following perl:
while (<>) {
while (/\\\s*$/) {
$_ .= <>;
}
undef %selects if /^\s*config\s+/;
if (/^\s+select\s+(\w+).*/) {
if (defined($selects{$1})) {
if ($selects{$1} eq $_) {
print STDERR "Warning: removing duplicated $1 entry\n";
} else {
print STDERR "Error: $1 differently selected\n".
"\tOld: $selects{$1}\n".
"\tNew: $_\n";
exit 1;
}
}
$selects{$1} = $_;
next;
}
if (%selects and (/^\s*$/ or /^\s+help/ or /^\s+---help---/ or
/^endif/ or /^endchoice/)) {
foreach $k (sort (keys %selects)) {
print "$selects{$k}";
}
undef %selects;
}
print;
}
if (%selects) {
foreach $k (sort (keys %selects)) {
print "$selects{$k}";
}
}
It found two duplicates:
Warning: removing duplicated S5P_SETUP_MIPIPHY entry
Warning: removing duplicated HARDIRQS_SW_RESEND entry
and they are identical duplicates, hence the shrinkage in the diffstat
of two lines.
We have four testers reporting success of this change (Tony, Stephen,
Linus and Sekhar.)
Acked-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Tony Lindgren <tony@atomide.com>
Acked-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/mach-shmobile/Kconfig')
-rw-r--r-- | arch/arm/mach-shmobile/Kconfig | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig index 4cacc2d22fbe..8ae100cc655c 100644 --- a/arch/arm/mach-shmobile/Kconfig +++ b/arch/arm/mach-shmobile/Kconfig | |||
@@ -4,49 +4,49 @@ comment "SH-Mobile System Type" | |||
4 | 4 | ||
5 | config ARCH_SH7367 | 5 | config ARCH_SH7367 |
6 | bool "SH-Mobile G3 (SH7367)" | 6 | bool "SH-Mobile G3 (SH7367)" |
7 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
7 | select CPU_V6 | 8 | select CPU_V6 |
8 | select SH_CLK_CPG | 9 | select SH_CLK_CPG |
9 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
10 | 10 | ||
11 | config ARCH_SH7377 | 11 | config ARCH_SH7377 |
12 | bool "SH-Mobile G4 (SH7377)" | 12 | bool "SH-Mobile G4 (SH7377)" |
13 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
13 | select CPU_V7 | 14 | select CPU_V7 |
14 | select SH_CLK_CPG | 15 | select SH_CLK_CPG |
15 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
16 | 16 | ||
17 | config ARCH_SH7372 | 17 | config ARCH_SH7372 |
18 | bool "SH-Mobile AP4 (SH7372)" | 18 | bool "SH-Mobile AP4 (SH7372)" |
19 | select CPU_V7 | ||
20 | select SH_CLK_CPG | ||
21 | select ARCH_WANT_OPTIONAL_GPIOLIB | 19 | select ARCH_WANT_OPTIONAL_GPIOLIB |
22 | select ARM_CPU_SUSPEND if PM || CPU_IDLE | 20 | select ARM_CPU_SUSPEND if PM || CPU_IDLE |
21 | select CPU_V7 | ||
22 | select SH_CLK_CPG | ||
23 | 23 | ||
24 | config ARCH_SH73A0 | 24 | config ARCH_SH73A0 |
25 | bool "SH-Mobile AG5 (R8A73A00)" | 25 | bool "SH-Mobile AG5 (R8A73A00)" |
26 | select CPU_V7 | ||
27 | select SH_CLK_CPG | ||
28 | select ARCH_WANT_OPTIONAL_GPIOLIB | 26 | select ARCH_WANT_OPTIONAL_GPIOLIB |
29 | select ARM_GIC | 27 | select ARM_GIC |
28 | select CPU_V7 | ||
30 | select I2C | 29 | select I2C |
30 | select SH_CLK_CPG | ||
31 | 31 | ||
32 | config ARCH_R8A7740 | 32 | config ARCH_R8A7740 |
33 | bool "R-Mobile A1 (R8A77400)" | 33 | bool "R-Mobile A1 (R8A77400)" |
34 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
34 | select CPU_V7 | 35 | select CPU_V7 |
35 | select SH_CLK_CPG | 36 | select SH_CLK_CPG |
36 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
37 | 37 | ||
38 | config ARCH_R8A7779 | 38 | config ARCH_R8A7779 |
39 | bool "R-Car H1 (R8A77790)" | 39 | bool "R-Car H1 (R8A77790)" |
40 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
41 | select ARM_GIC | ||
40 | select CPU_V7 | 42 | select CPU_V7 |
41 | select SH_CLK_CPG | 43 | select SH_CLK_CPG |
42 | select ARM_GIC | ||
43 | select ARCH_WANT_OPTIONAL_GPIOLIB | ||
44 | 44 | ||
45 | config ARCH_EMEV2 | 45 | config ARCH_EMEV2 |
46 | bool "Emma Mobile EV2" | 46 | bool "Emma Mobile EV2" |
47 | select CPU_V7 | ||
48 | select ARM_GIC | ||
49 | select ARCH_WANT_OPTIONAL_GPIOLIB | 47 | select ARCH_WANT_OPTIONAL_GPIOLIB |
48 | select ARM_GIC | ||
49 | select CPU_V7 | ||
50 | 50 | ||
51 | comment "SH-Mobile Board Type" | 51 | comment "SH-Mobile Board Type" |
52 | 52 | ||
@@ -65,9 +65,9 @@ config MACH_AP4EVB | |||
65 | bool "AP4EVB board" | 65 | bool "AP4EVB board" |
66 | depends on ARCH_SH7372 | 66 | depends on ARCH_SH7372 |
67 | select ARCH_REQUIRE_GPIOLIB | 67 | select ARCH_REQUIRE_GPIOLIB |
68 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | ||
68 | select SH_LCD_MIPI_DSI | 69 | select SH_LCD_MIPI_DSI |
69 | select SND_SOC_AK4642 if SND_SIMPLE_CARD | 70 | select SND_SOC_AK4642 if SND_SIMPLE_CARD |
70 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | ||
71 | 71 | ||
72 | choice | 72 | choice |
73 | prompt "AP4EVB LCD panel selection" | 73 | prompt "AP4EVB LCD panel selection" |
@@ -84,37 +84,37 @@ endchoice | |||
84 | 84 | ||
85 | config MACH_AG5EVM | 85 | config MACH_AG5EVM |
86 | bool "AG5EVM board" | 86 | bool "AG5EVM board" |
87 | depends on ARCH_SH73A0 | ||
87 | select ARCH_REQUIRE_GPIOLIB | 88 | select ARCH_REQUIRE_GPIOLIB |
88 | select SH_LCD_MIPI_DSI | ||
89 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 89 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
90 | depends on ARCH_SH73A0 | 90 | select SH_LCD_MIPI_DSI |
91 | 91 | ||
92 | config MACH_MACKEREL | 92 | config MACH_MACKEREL |
93 | bool "mackerel board" | 93 | bool "mackerel board" |
94 | depends on ARCH_SH7372 | 94 | depends on ARCH_SH7372 |
95 | select ARCH_REQUIRE_GPIOLIB | 95 | select ARCH_REQUIRE_GPIOLIB |
96 | select SND_SOC_AK4642 if SND_SIMPLE_CARD | ||
97 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 96 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
97 | select SND_SOC_AK4642 if SND_SIMPLE_CARD | ||
98 | 98 | ||
99 | config MACH_KOTA2 | 99 | config MACH_KOTA2 |
100 | bool "KOTA2 board" | 100 | bool "KOTA2 board" |
101 | depends on ARCH_SH73A0 | ||
101 | select ARCH_REQUIRE_GPIOLIB | 102 | select ARCH_REQUIRE_GPIOLIB |
102 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 103 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
103 | depends on ARCH_SH73A0 | ||
104 | 104 | ||
105 | config MACH_BONITO | 105 | config MACH_BONITO |
106 | bool "bonito board" | 106 | bool "bonito board" |
107 | depends on ARCH_R8A7740 | ||
107 | select ARCH_REQUIRE_GPIOLIB | 108 | select ARCH_REQUIRE_GPIOLIB |
108 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 109 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
109 | depends on ARCH_R8A7740 | ||
110 | 110 | ||
111 | config MACH_ARMADILLO800EVA | 111 | config MACH_ARMADILLO800EVA |
112 | bool "Armadillo-800 EVA board" | 112 | bool "Armadillo-800 EVA board" |
113 | depends on ARCH_R8A7740 | 113 | depends on ARCH_R8A7740 |
114 | select ARCH_REQUIRE_GPIOLIB | 114 | select ARCH_REQUIRE_GPIOLIB |
115 | select USE_OF | ||
116 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 115 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
117 | select SND_SOC_WM8978 if SND_SIMPLE_CARD | 116 | select SND_SOC_WM8978 if SND_SIMPLE_CARD |
117 | select USE_OF | ||
118 | 118 | ||
119 | config MACH_MARZEN | 119 | config MACH_MARZEN |
120 | bool "MARZEN board" | 120 | bool "MARZEN board" |
@@ -125,16 +125,16 @@ config MACH_MARZEN | |||
125 | config MACH_KZM9D | 125 | config MACH_KZM9D |
126 | bool "KZM9D board" | 126 | bool "KZM9D board" |
127 | depends on ARCH_EMEV2 | 127 | depends on ARCH_EMEV2 |
128 | select USE_OF | ||
129 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 128 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
129 | select USE_OF | ||
130 | 130 | ||
131 | config MACH_KZM9G | 131 | config MACH_KZM9G |
132 | bool "KZM-A9-GT board" | 132 | bool "KZM-A9-GT board" |
133 | depends on ARCH_SH73A0 | 133 | depends on ARCH_SH73A0 |
134 | select ARCH_REQUIRE_GPIOLIB | 134 | select ARCH_REQUIRE_GPIOLIB |
135 | select USE_OF | ||
136 | select SND_SOC_AK4642 if SND_SIMPLE_CARD | ||
137 | select REGULATOR_FIXED_VOLTAGE if REGULATOR | 135 | select REGULATOR_FIXED_VOLTAGE if REGULATOR |
136 | select SND_SOC_AK4642 if SND_SIMPLE_CARD | ||
137 | select USE_OF | ||
138 | 138 | ||
139 | comment "SH-Mobile System Configuration" | 139 | comment "SH-Mobile System Configuration" |
140 | 140 | ||