diff options
author | Ezequiel Garcia <ezequiel.garcia@free-electrons.com> | 2014-03-03 11:42:38 -0500 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2014-03-04 07:49:43 -0500 |
commit | ca2b722d1ab5bc3ffc34b5995248968cd8a7cb6f (patch) | |
tree | 4f2ae5d0e8963e7fb82d31774a6b399e6383c9bf /drivers/mtd/ubi/block.c | |
parent | 4d283ee2517303afa54ad6cbd9342a2f748cf509 (diff) |
UBI: block: Mark init-only symbol as __initdata
ubiblock_param_ops should be marked as __init as it's only used to set
a driver parameter on insertion time. This commit fixes the following:
WARNING: drivers/mtd/built-in.o(.text+0x653ac): Section mismatch in
reference from the variable ubiblock_param_ops to the function
.init.text:ubiblock_set_param()
The function ubiblock_param_ops() references the function __init
ubiblock_set_param(). This is often because ubiblock_param_ops lacks a
__init annotation or the annotation of ubiblock_set_param is wrong.
Given gcc errors if the struct is marked const __initdata, this commit
drops the const mark from it.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Diffstat (limited to 'drivers/mtd/ubi/block.c')
-rw-r--r-- | drivers/mtd/ubi/block.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/ubi/block.c b/drivers/mtd/ubi/block.c index 6402e41e40b4..cd6be981be30 100644 --- a/drivers/mtd/ubi/block.c +++ b/drivers/mtd/ubi/block.c | |||
@@ -156,7 +156,7 @@ static int __init ubiblock_set_param(const char *val, | |||
156 | return 0; | 156 | return 0; |
157 | } | 157 | } |
158 | 158 | ||
159 | static const struct kernel_param_ops ubiblock_param_ops = { | 159 | static struct kernel_param_ops ubiblock_param_ops __initdata = { |
160 | .set = ubiblock_set_param, | 160 | .set = ubiblock_set_param, |
161 | }; | 161 | }; |
162 | module_param_cb(block, &ubiblock_param_ops, NULL, 0); | 162 | module_param_cb(block, &ubiblock_param_ops, NULL, 0); |