On 29.11.2012 03:12, Fengguang Wu wrote: > On Wed, Nov 28, 2012 at 02:16:49PM -0800, Greg KH wrote: >> On Thu, Nov 29, 2012 at 05:38:31AM +0800, kbuild test robot wrote: >>> tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git driver-core-next >>> head: da095fd3d5063f2dd03468d71f7df39a0430d86f >>> commit: f791be492f76dea7b0641ed227a60eeb2fa7e255 [65/93] mfd: remove use of __devinit >>> config: x86_64-randconfig-s363 (attached as .config) >>> >>> All warnings: >>> >>> WARNING: drivers/built-in.o(.text+0xe2399): Section mismatch in reference from the function stmpe_i2c_probe() to the function .devinit.text:stmpe_probe() >>> The function stmpe_i2c_probe() references >>> the function __devinit stmpe_probe(). >>> This is often because stmpe_i2c_probe lacks a __devinit >>> annotation or the annotation of stmpe_probe is wrong. >> >> Not an issue anymore as __devinit is always defined to nothing, so this >> check doesn't mean anything. Really? If __devinit was defined to do nothing, then the function would not end up in .devinit.text. > So modpost.c or something in the kernel should be updated? modpost.c checks the sections in the binary. So if a macro is defined to nothing, modpost will stop complaining (and of course the check can be cleaned up later). Michal _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
