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-realview | |
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-realview')
-rw-r--r-- | arch/arm/mach-realview/Kconfig | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/arm/mach-realview/Kconfig b/arch/arm/mach-realview/Kconfig index c593be428b8f..14c1d47e1abf 100644 --- a/arch/arm/mach-realview/Kconfig +++ b/arch/arm/mach-realview/Kconfig | |||
@@ -21,8 +21,8 @@ config REALVIEW_EB_A9MP | |||
21 | config REALVIEW_EB_ARM11MP | 21 | config REALVIEW_EB_ARM11MP |
22 | bool "Support ARM11MPCore Tile" | 22 | bool "Support ARM11MPCore Tile" |
23 | depends on MACH_REALVIEW_EB | 23 | depends on MACH_REALVIEW_EB |
24 | select CPU_V6K | ||
25 | select ARCH_HAS_BARRIERS if SMP | 24 | select ARCH_HAS_BARRIERS if SMP |
25 | select CPU_V6K | ||
26 | select HAVE_SMP | 26 | select HAVE_SMP |
27 | select MIGHT_HAVE_CACHE_L2X0 | 27 | select MIGHT_HAVE_CACHE_L2X0 |
28 | help | 28 | help |
@@ -40,12 +40,12 @@ config REALVIEW_EB_ARM11MP_REVB | |||
40 | 40 | ||
41 | config MACH_REALVIEW_PB11MP | 41 | config MACH_REALVIEW_PB11MP |
42 | bool "Support RealView(R) Platform Baseboard for ARM11MPCore" | 42 | bool "Support RealView(R) Platform Baseboard for ARM11MPCore" |
43 | select CPU_V6K | 43 | select ARCH_HAS_BARRIERS if SMP |
44 | select ARM_GIC | 44 | select ARM_GIC |
45 | select CPU_V6K | ||
45 | select HAVE_PATA_PLATFORM | 46 | select HAVE_PATA_PLATFORM |
46 | select HAVE_SMP | 47 | select HAVE_SMP |
47 | select MIGHT_HAVE_CACHE_L2X0 | 48 | select MIGHT_HAVE_CACHE_L2X0 |
48 | select ARCH_HAS_BARRIERS if SMP | ||
49 | help | 49 | help |
50 | Include support for the ARM(R) RealView(R) Platform Baseboard for | 50 | Include support for the ARM(R) RealView(R) Platform Baseboard for |
51 | the ARM11MPCore. This platform has an on-board ARM11MPCore and has | 51 | the ARM11MPCore. This platform has an on-board ARM11MPCore and has |
@@ -54,8 +54,8 @@ config MACH_REALVIEW_PB11MP | |||
54 | # ARMv6 CPU without K extensions, but does have the new exclusive ops | 54 | # ARMv6 CPU without K extensions, but does have the new exclusive ops |
55 | config MACH_REALVIEW_PB1176 | 55 | config MACH_REALVIEW_PB1176 |
56 | bool "Support RealView(R) Platform Baseboard for ARM1176JZF-S" | 56 | bool "Support RealView(R) Platform Baseboard for ARM1176JZF-S" |
57 | select CPU_V6 | ||
58 | select ARM_GIC | 57 | select ARM_GIC |
58 | select CPU_V6 | ||
59 | select HAVE_TCM | 59 | select HAVE_TCM |
60 | select MIGHT_HAVE_CACHE_L2X0 | 60 | select MIGHT_HAVE_CACHE_L2X0 |
61 | help | 61 | help |
@@ -73,8 +73,8 @@ config REALVIEW_PB1176_SECURE_FLASH | |||
73 | 73 | ||
74 | config MACH_REALVIEW_PBA8 | 74 | config MACH_REALVIEW_PBA8 |
75 | bool "Support RealView(R) Platform Baseboard for Cortex(tm)-A8 platform" | 75 | bool "Support RealView(R) Platform Baseboard for Cortex(tm)-A8 platform" |
76 | select CPU_V7 | ||
77 | select ARM_GIC | 76 | select ARM_GIC |
77 | select CPU_V7 | ||
78 | select HAVE_PATA_PLATFORM | 78 | select HAVE_PATA_PLATFORM |
79 | help | 79 | help |
80 | Include support for the ARM(R) RealView Platform Baseboard for | 80 | Include support for the ARM(R) RealView Platform Baseboard for |
@@ -83,11 +83,11 @@ config MACH_REALVIEW_PBA8 | |||
83 | 83 | ||
84 | config MACH_REALVIEW_PBX | 84 | config MACH_REALVIEW_PBX |
85 | bool "Support RealView(R) Platform Baseboard Explore" | 85 | bool "Support RealView(R) Platform Baseboard Explore" |
86 | select ARCH_SPARSEMEM_ENABLE if CPU_V7 && !REALVIEW_HIGH_PHYS_OFFSET | ||
86 | select ARM_GIC | 87 | select ARM_GIC |
87 | select HAVE_PATA_PLATFORM | 88 | select HAVE_PATA_PLATFORM |
88 | select HAVE_SMP | 89 | select HAVE_SMP |
89 | select MIGHT_HAVE_CACHE_L2X0 | 90 | select MIGHT_HAVE_CACHE_L2X0 |
90 | select ARCH_SPARSEMEM_ENABLE if CPU_V7 && !REALVIEW_HIGH_PHYS_OFFSET | ||
91 | select ZONE_DMA if SPARSEMEM | 91 | select ZONE_DMA if SPARSEMEM |
92 | help | 92 | help |
93 | Include support for the ARM(R) RealView(R) Platform Baseboard | 93 | Include support for the ARM(R) RealView(R) Platform Baseboard |