ARM: msm: proc_comm_boot_wait should not be __init
authorArnd Bergmann <arnd@arndb.de>
Fri, 25 Jan 2013 14:14:20 +0000 (14:14 +0000)
committerOlof Johansson <olof@lixom.net>
Tue, 12 Feb 2013 01:42:28 +0000 (17:42 -0800)
msm_smd_probe is a driver probe function and may get
called after the __init time, so it must not call
any __init function, as the link-time warning reports.
Take away the __init annotation on proc_comm_boot_wait
to fix this.

Without this patch, building msm_defconfig results in:

WARNING: vmlinux.o(.text+0xb048): Section mismatch in reference from the function msm_smd_probe() to the function .init.text:proc_comm_boot_wait()
The function msm_smd_probe() references
the function __init proc_comm_boot_wait().
This is often because msm_smd_probe lacks a __init
annotation or the annotation of proc_comm_boot_wait is wrong.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: Bryan Huntsman <bryanh@codeaurora.org>
Cc: Daniel Walker <c_dwalke@quicinc.com>
Cc: linux-arm-msm@vger.kernel.org
Acked-by: David Brown <davidb@codeaurora.org>
Signed-off-by: Olof Johansson <olof@lixom.net>
arch/arm/mach-msm/proc_comm.h

index 12da4cacd4a8141b7119fe9335cf3eafaee40ce1..e8d043a0e9906e3969a9011c25b573f75e86a8d3 100644 (file)
@@ -253,6 +253,6 @@ enum {
                (((drvstr) & 0xF) << 17))
 
 int msm_proc_comm(unsigned cmd, unsigned *data1, unsigned *data2);
-void __init proc_comm_boot_wait(void);
+void proc_comm_boot_wait(void);
 
 #endif