Re: [PATCH] V4L/DVB: mantis: Fix IR_CORE dependency

[Ingo Molnar]

* Randy Dunlap <randy.dunlap@xxxxxxxxxx> wrote:

> On 08/24/10 01:45, Ingo Molnar wrote:
> > 
> > * Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> wrote:
> > 
> >> Linus,
> >>
> >> Please pull from:
> >>   ssh://master.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-2.6.git v4l_for_linus
> >>
> >> For 3 build fixes.
> >>
> >> Cheers,
> >> Mauro.
> >>
> >> The following changes since commit ad41a1e0cab07c5125456e8d38e5b1ab148d04aa:
> >>
> >>   Merge branch 'io_remap_pfn_range' of git://www.jni.nu/cris (2010-08-12 10:17:19 -0700)
> >>
> >> are available in the git repository at:
> >>
> >>   ssh://master.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-2.6.git v4l_for_linus
> >>
> >> Mauro Carvalho Chehab (2):
> >>       V4L/DVB: Fix IR_CORE dependencies
> >>       V4L/DVB: fix Kconfig to depends on VIDEO_IR
> >>
> >> Randy Dunlap (1):
> >>       V4L/DVB: v4l2-ctrls.c: needs to include slab.h
> > 
> > FYI, there's one more IR_CORE related build bug which triggers 
> > frequently in randconfig tests - see the fix below.
> > 
> > Thanks,
> > 
> > 	Ingo
> > 
> > ------------------->
> > From c56aef270d7ec01564c632c1f7ebab6b8f9f032c Mon Sep 17 00:00:00 2001
> > From: Ingo Molnar <mingo@xxxxxxx>
> > Date: Tue, 24 Aug 2010 10:41:33 +0200
> > Subject: [PATCH] V4L/DVB: mantis: Fix IR_CORE dependency
> > 
> > This build bug triggers:
> > 
> >  drivers/built-in.o: In function `mantis_exit':
> >  (.text+0x377413): undefined reference to `ir_input_unregister'
> >  drivers/built-in.o: In function `mantis_input_init':
> >  (.text+0x3774ff): undefined reference to `__ir_input_register'
> > 
> > If MANTIS_CORE is enabled but IR_CORE is not. Add the correct
> > dependency.
> > 
> > Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
> > ---
> >  drivers/media/dvb/mantis/Kconfig |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/media/dvb/mantis/Kconfig b/drivers/media/dvb/mantis/Kconfig
> > index decdeda..fd0830e 100644
> > --- a/drivers/media/dvb/mantis/Kconfig
> > +++ b/drivers/media/dvb/mantis/Kconfig
> > @@ -1,6 +1,6 @@
> >  config MANTIS_CORE
> >  	tristate "Mantis/Hopper PCI bridge based devices"
> > -	depends on PCI && I2C && INPUT
> > +	depends on PCI && I2C && INPUT && IR_CORE
> >  
> >  	help
> >  	  Support for PCI cards based on the Mantis and Hopper PCi bridge.
> 
> 
> Acked-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> http://lkml.org/lkml/2010/8/17/341

Your patch came first :-)

Btw., the reason i missed your patch is that i grepped lkml for the 
static build failure - while your changelog contained the modular one. 
Oh well :)

Thanks,

	Ingo
--
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/