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-hisi | |
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-hisi')
-rw-r--r-- | arch/arm/mach-hisi/hisilicon.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/arm/mach-hisi/hisilicon.c b/arch/arm/mach-hisi/hisilicon.c index 76b907078b58..c6bd7c7bd4aa 100644 --- a/arch/arm/mach-hisi/hisilicon.c +++ b/arch/arm/mach-hisi/hisilicon.c | |||
@@ -45,7 +45,7 @@ static void __init hi3620_map_io(void) | |||
45 | iotable_init(hi3620_io_desc, ARRAY_SIZE(hi3620_io_desc)); | 45 | iotable_init(hi3620_io_desc, ARRAY_SIZE(hi3620_io_desc)); |
46 | } | 46 | } |
47 | 47 | ||
48 | static const char *hi3xxx_compat[] __initconst = { | 48 | static const char *const hi3xxx_compat[] __initconst = { |
49 | "hisilicon,hi3620-hi4511", | 49 | "hisilicon,hi3620-hi4511", |
50 | NULL, | 50 | NULL, |
51 | }; | 51 | }; |
@@ -55,7 +55,7 @@ DT_MACHINE_START(HI3620, "Hisilicon Hi3620 (Flattened Device Tree)") | |||
55 | .dt_compat = hi3xxx_compat, | 55 | .dt_compat = hi3xxx_compat, |
56 | MACHINE_END | 56 | MACHINE_END |
57 | 57 | ||
58 | static const char *hix5hd2_compat[] __initconst = { | 58 | static const char *const hix5hd2_compat[] __initconst = { |
59 | "hisilicon,hix5hd2", | 59 | "hisilicon,hix5hd2", |
60 | NULL, | 60 | NULL, |
61 | }; | 61 | }; |
@@ -64,7 +64,7 @@ DT_MACHINE_START(HIX5HD2_DT, "Hisilicon HIX5HD2 (Flattened Device Tree)") | |||
64 | .dt_compat = hix5hd2_compat, | 64 | .dt_compat = hix5hd2_compat, |
65 | MACHINE_END | 65 | MACHINE_END |
66 | 66 | ||
67 | static const char *hip04_compat[] __initconst = { | 67 | static const char *const hip04_compat[] __initconst = { |
68 | "hisilicon,hip04-d01", | 68 | "hisilicon,hip04-d01", |
69 | NULL, | 69 | NULL, |
70 | }; | 70 | }; |
@@ -73,7 +73,7 @@ DT_MACHINE_START(HIP04, "Hisilicon HiP04 (Flattened Device Tree)") | |||
73 | .dt_compat = hip04_compat, | 73 | .dt_compat = hip04_compat, |
74 | MACHINE_END | 74 | MACHINE_END |
75 | 75 | ||
76 | static const char *hip01_compat[] __initconst = { | 76 | static const char *const hip01_compat[] __initconst = { |
77 | "hisilicon,hip01", | 77 | "hisilicon,hip01", |
78 | "hisilicon,hip01-ca9x2", | 78 | "hisilicon,hip01-ca9x2", |
79 | NULL, | 79 | NULL, |