diff options
author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2015-02-18 15:01:45 -0500 |
---|---|---|
committer | Arnd Bergmann <arnd@arndb.de> | 2015-02-19 03:44:17 -0500 |
commit | 543c5040f564c80fe59ae82a60fc061055d04a41 (patch) | |
tree | 40f47351a06a1d8e54f0956803e58cdcfe09bb8a /arch/arm/mach-prima2 | |
parent | d88d6cfc912e9e54f9ec0337a84691436c654077 (diff) |
ARM: make arrays containing machine compatible strings const
The definition
static const char *axxia_dt_match[] __initconst = {
...
defines a changable array of constant strings. That is you must not do:
*axxia_dt_match[0] = 'k';
but
axxia_dt_match[0] = "different string";
is fine. So the annotation __initconst is wrong and yields a compiler
error when other really const variables are added with __initconst.
As the struct machine_desc member dt_compat is declared as
const char *const *dt_compat;
making the arrays const is the better alternative over changing all
annotations to __initdata.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Diffstat (limited to 'arch/arm/mach-prima2')
-rw-r--r-- | arch/arm/mach-prima2/common.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/arch/arm/mach-prima2/common.c b/arch/arm/mach-prima2/common.c index 0c819bb88418..8cadb302a7d2 100644 --- a/arch/arm/mach-prima2/common.c +++ b/arch/arm/mach-prima2/common.c | |||
@@ -21,7 +21,7 @@ static void __init sirfsoc_init_late(void) | |||
21 | } | 21 | } |
22 | 22 | ||
23 | #ifdef CONFIG_ARCH_ATLAS6 | 23 | #ifdef CONFIG_ARCH_ATLAS6 |
24 | static const char *atlas6_dt_match[] __initconst = { | 24 | static const char *const atlas6_dt_match[] __initconst = { |
25 | "sirf,atlas6", | 25 | "sirf,atlas6", |
26 | NULL | 26 | NULL |
27 | }; | 27 | }; |
@@ -36,7 +36,7 @@ MACHINE_END | |||
36 | #endif | 36 | #endif |
37 | 37 | ||
38 | #ifdef CONFIG_ARCH_PRIMA2 | 38 | #ifdef CONFIG_ARCH_PRIMA2 |
39 | static const char *prima2_dt_match[] __initconst = { | 39 | static const char *const prima2_dt_match[] __initconst = { |
40 | "sirf,prima2", | 40 | "sirf,prima2", |
41 | NULL | 41 | NULL |
42 | }; | 42 | }; |
@@ -52,7 +52,7 @@ MACHINE_END | |||
52 | #endif | 52 | #endif |
53 | 53 | ||
54 | #ifdef CONFIG_ARCH_ATLAS7 | 54 | #ifdef CONFIG_ARCH_ATLAS7 |
55 | static const char *atlas7_dt_match[] __initdata = { | 55 | static const char *const atlas7_dt_match[] __initconst = { |
56 | "sirf,atlas7", | 56 | "sirf,atlas7", |
57 | NULL | 57 | NULL |
58 | }; | 58 | }; |