Re: [sound:for-next 37/38] WARNING: Section mismatches on snd_opti9xx_resume()/snd_opti9xx_configure

At Tue, 17 Jul 2012 18:12:38 +0800,
Fengguang Wu wrote:

Hi Ondrej,

There are new compile warnings show up in

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
head:   0c7293b6ea0b363c6715c6f0b2894fa63b8e9053
commit: 5dd250728aa4cad49cfa18eb8ed11ba470ce382a [37/38] ALSA: snd-opti9xx: Implement suspend/resume
config: i386-allmodconfig (attached as .config)

All warnings:


WARNING: sound/isa/opti9xx/snd-opti92x-ad1848.o(.text+0x1b2): Section mismatch in reference from the function snd_opti9xx_resume() to the function .devinit.text:snd_opti9xx_configure.isra.2()
The function snd_opti9xx_resume() references
the function __devinit snd_opti9xx_configure.isra.2().
This is often because snd_opti9xx_resume lacks a __devinit 
annotation or the annotation of snd_opti9xx_configure.isra.2 is wrong.

WARNING: sound/isa/opti9xx/snd-opti92x-cs4231.o(.text+0xb6): Section mismatch in reference from the function snd_opti9xx_resume() to the function .devinit.text:snd_opti9xx_configure()
The function snd_opti9xx_resume() references
the function __devinit snd_opti9xx_configure().
This is often because snd_opti9xx_resume lacks a __devinit 
annotation or the annotation of snd_opti9xx_configure is wrong.

WARNING: sound/isa/opti9xx/snd-opti93x.o(.text+0x1ea): Section mismatch in reference from the function snd_opti9xx_resume() to the function .devinit.text:snd_opti9xx_configure()
The function snd_opti9xx_resume() references
the function __devinit snd_opti9xx_configure().
This is often because snd_opti9xx_resume lacks a __devinit 
annotation or the annotation of snd_opti9xx_configure is wrong.

Fixed now with the patch below.  Thanks.


Takashi

---
From: Takashi Iwai <redacted>
Subject: [PATCH] ALSA: opti9xx: Fix section mismatch by PM support

In the previous commit, snd_opti9xx_configure() is called from the
resume handler but it's still marked as __devinit.  Fix it.

Reported-by: Fengguang Wu <redacted>
Signed-off-by: Takashi Iwai <redacted>
---
 sound/isa/opti9xx/opti92x-ad1848.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/isa/opti9xx/opti92x-ad1848.c b/sound/isa/opti9xx/opti92x-ad1848.c
index d7ce012..f8fbe22 100644
--- a/sound/isa/opti9xx/opti92x-ad1848.c
+++ b/sound/isa/opti9xx/opti92x-ad1848.c

@@ -348,7 +348,7 @@ static void snd_opti9xx_write(struct snd_opti9xx *chip, unsigned char reg,
 		(snd_opti9xx_read(chip, reg) & ~(mask)) | ((value) & (mask)))
 
 
-static int __devinit snd_opti9xx_configure(struct snd_opti9xx *chip,
+static int snd_opti9xx_configure(struct snd_opti9xx *chip,
 					   long port,
 					   int irq, int dma1, int dma2,
 					   long mpu_port, int mpu_irq)

-- 
1.7.10.4