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

[Takashi Iwai]

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 <tiwai at suse.de>
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 <fengguang.wu at intel.com>
Signed-off-by: Takashi Iwai <tiwai at suse.de>
---
 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