Re: ARM: Section mismatch warnings

[Russell King]

On Mon, May 28, 2007 at 08:36:29PM +0200, Sam Ravnborg wrote:
> > 
> > > o-arm-neponset/log.out:WARNING: arch/arm/mach-sa1100/built-in.o(.text+0x1748):
> >  Section mismatch: reference to .init.text:sa1110_mb_disable
> >  (between 'neponset_init' and 'neponset_resume')
> > 
> > This one looks quite bogus.  sa1110_mb_disable() is marked __init, and
> > it's called from neponset_init() which is also marked __init.
> 
> Created a neponset.i file - output:
> 
> static int neponset_init(void)
> {
>  platform_driver_register(&neponset_device_driver);
> 
> So we loose the __init marker during preprocessing.
> Which is due the the buggy part of my path that adds __devinit.
> 
> There was a reason I noted the devinit markers should be reviewed
> carefully - this one seems wrong.

I don't understand why you added that __devinit in the first place.
What happens if you leave it as a mere __init function, as it should be?

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/