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 [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
