[PATCH] ASoC: imx-audmux: Fix section mismatch

[PATCH] ASoC: imx-audmux: Fix section mismatch

[Lars-Peter Clausen]

audmux_debugfs_init() is marked as __init, but is called from imx_audmux_probe()
which is not marked as __init. This creates a section mismatch and a potential
runtime crash (if imx_audmux_probe() is called after the .init section was
dropped). This patch removes the __init annotation from audmux_debugfs_init(),
which fixes the following warning:
	WARNING: sound/soc/built-in.o(.text+0x86960): Section mismatch in reference
	from the function imx_audmux_probe() to the function
	.init.text:audmux_debugfs_init()

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 sound/soc/fsl/imx-audmux.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx-audmux.c
index ac86993..267717a 100644
--- a/sound/soc/fsl/imx-audmux.c
+++ b/sound/soc/fsl/imx-audmux.c
@@ -145,7 +145,7 @@ static const struct file_operations audmux_debugfs_fops = {
 	.llseek = default_llseek,
 };
 
-static void __init audmux_debugfs_init(void)
+static void audmux_debugfs_init(void)
 {
 	int i;
 	char buf[20];
-- 
1.8.0

Re: [PATCH] ASoC: imx-audmux: Fix section mismatch

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 445 bytes --]

On Thu, Apr 24, 2014 at 08:25:56AM +0200, Lars-Peter Clausen wrote:
> audmux_debugfs_init() is marked as __init, but is called from imx_audmux_probe()
> which is not marked as __init. This creates a section mismatch and a potential
> runtime crash (if imx_audmux_probe() is called after the .init section was
> dropped). This patch removes the __init annotation from audmux_debugfs_init(),
> which fixes the following warning:

Applied, thanks.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]