diff options
author | Rakib Mullick <rakib.mullick@gmail.com> | 2009-10-12 18:13:32 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-10-13 01:24:40 -0400 |
commit | dcb73eed70575c68f1389f7fac7cbd0feaeb50f3 (patch) | |
tree | 1efe226892b8c2a2471ddba335c06b9ee666b594 /drivers/platform | |
parent | 161291396e76e0832c08f617eb9bd364d1648148 (diff) |
eeepc-laptop: Properly annote eeepc_enable_camera().
Currently the annotation for function eeepc_enable_camera() is
__init, and refers to a
function eeepc_hotk_add() which is non-init. Use __devinit for both
functions which is
more appropriate and fixes a section mismatch warning.
We were warned by the following warning:
LD drivers/platform/x86/built-in.o
WARNING: drivers/platform/x86/built-in.o(.text+0x12e1): Section
mismatch in reference from the function eeepc_hotk_add() to the
function .init.text:eeepc_enable_camera()
The function eeepc_hotk_add() references
the function __init eeepc_enable_camera().
This is often because eeepc_hotk_add lacks a __init
annotation or the annotation of eeepc_enable_camera is wrong.
Signed-off-by: Rakib Mullick <rakib.mullick@gmail.com>
Signed-off-by: Corentin Chary <corentincj@iksaif.net>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/eeepc-laptop.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c index 749e2102b2be..789d6ae003fb 100644 --- a/drivers/platform/x86/eeepc-laptop.c +++ b/drivers/platform/x86/eeepc-laptop.c | |||
@@ -350,7 +350,7 @@ static const struct rfkill_ops eeepc_rfkill_ops = { | |||
350 | .set_block = eeepc_rfkill_set, | 350 | .set_block = eeepc_rfkill_set, |
351 | }; | 351 | }; |
352 | 352 | ||
353 | static void __init eeepc_enable_camera(void) | 353 | static void __devinit eeepc_enable_camera(void) |
354 | { | 354 | { |
355 | /* | 355 | /* |
356 | * If the following call to set_acpi() fails, it's because there's no | 356 | * If the following call to set_acpi() fails, it's because there's no |
@@ -1189,7 +1189,7 @@ static int eeepc_input_init(struct device *dev) | |||
1189 | return 0; | 1189 | return 0; |
1190 | } | 1190 | } |
1191 | 1191 | ||
1192 | static int eeepc_hotk_add(struct acpi_device *device) | 1192 | static int __devinit eeepc_hotk_add(struct acpi_device *device) |
1193 | { | 1193 | { |
1194 | struct device *dev; | 1194 | struct device *dev; |
1195 | int result; | 1195 | int result; |