On Mon, Jan 25, 2010 at 10:06 AM, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> wrote: > Hi Devin/Andy/Jean, > > The cx88-alsa and cx18-drivers are broken: the driver depend of request_modules that doesn't exist > when !CONFIG_MODULES, and has some wrong __init annotations. > > The sq905c has a warning. > > I'm compiling it with: > make ARCH=i386 allmodconfig drivers/media/|grep -v "^ CC" |grep -v "^ LD" > > Those are the errors found: > > drivers/media/video/cx18/cx18-driver.c:252: warning: ‘request_modules’ used but never defined > WARNING: drivers/media/video/cx18/cx18-alsa.o(.text+0x4de): Section mismatch in reference from the function cx18_alsa_load() to the function .init.text:snd_cx18_init() > The function cx18_alsa_load() references > the function __init snd_cx18_init(). > This is often because cx18_alsa_load lacks a __init > annotation or the annotation of snd_cx18_init is wrong. > > WARNING: drivers/media/video/cx18/built-in.o(.text+0x1c022): Section mismatch in reference from the function cx18_alsa_load() to the function .init.text:snd_cx18_init() > The function cx18_alsa_load() references > the function __init snd_cx18_init(). > This is often because cx18_alsa_load lacks a __init > annotation or the annotation of snd_cx18_init is wrong. > > drivers/media/video/gspca/sq905c.c: In function ‘sd_config’: > drivers/media/video/gspca/sq905c.c:207: warning: unused variable ‘i’ > WARNING: drivers/media/video/built-in.o(.text+0x28d24e): Section mismatch in reference from the function cx18_alsa_load() to the function .init.text:snd_cx18_init() > The function cx18_alsa_load() references > the function __init snd_cx18_init(). > This is often because cx18_alsa_load lacks a __init > annotation or the annotation of snd_cx18_init is wrong. > > WARNING: drivers/media/built-in.o(.text+0x2d2a2a): Section mismatch in reference from the function cx18_alsa_load() to the function .init.text:snd_cx18_init() > The function cx18_alsa_load() references > the function __init snd_cx18_init(). > This is often because cx18_alsa_load lacks a __init > annotation or the annotation of snd_cx18_init is wrong. This looks like breakage I probably introduced with the cx18 alsa support. I will dig into this tonight. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html