login
Header Space

 
 

[patch] video: build fix for drivers/media/video/pvrusb2/

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <linux-kernel@...>, Mauro Carvalho Chehab <mchehab@...>
Cc: Guennadi Liakhovetski <g.liakhovetski@...>, Michael Krufky <mkrufky@...>, Mike Isely <isely@...>
Date: Sunday, May 11, 2008 - 3:21 am

x86.git testing found the following build failure:

  drivers/built-in.o: In function `pvr2_dvb_feed_thread':
  pvrusb2-dvb.c:(.text+0x127e78): undefined reference to `dvb_dmx_swfilter'
  drivers/built-in.o: In function `pvr2_dvb_adapter_exit':
  pvrusb2-dvb.c:(.text+0x128357): undefined reference to `dvb_net_release'
  pvrusb2-dvb.c:(.text+0x12836f): undefined reference to `dvb_dmxdev_release'
  [...]

with this config:

  http://redhat.com/~mingo/misc/config-Sun_May_11_07_06_35_CEST_2008.bad

the reason for the missing symbols is this combination:

  CONFIG_VIDEO_PVRUSB2=y
  CONFIG_DVB_CORE=m

i.e. pvrusb2 is built-in, dvb-core is modular.

This patch solves the problem by adding a dependency on DVB_CORE - this 
is used by other drivers such as au0828 as well. This way the pvrusb2 
driver can still be built, but if dvb-core is a module then it will 
correctly be a module as well and cannot be built-in.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 drivers/media/video/pvrusb2/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux/drivers/media/video/pvrusb2/Kconfig
===================================================================
--- linux.orig/drivers/media/video/pvrusb2/Kconfig
+++ linux/drivers/media/video/pvrusb2/Kconfig
@@ -1,6 +1,6 @@
 config VIDEO_PVRUSB2
 	tristate "Hauppauge WinTV-PVR USB2 support"
-	depends on VIDEO_V4L2 && I2C
+	depends on VIDEO_V4L2 && I2C && DVB_CORE
 	select FW_LOADER
 	select MEDIA_TUNER
 	select VIDEO_TVEEPROM
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch] video: build fix for drivers/media/video/pvrusb2/, Ingo Molnar, (Sun May 11, 3:21 am)
Re: [patch] video: build fix for drivers/media/video/pvrusb2/, Mauro Carvalho Chehab, (Mon May 12, 10:54 pm)
Re: [patch] video: build fix for drivers/media/video/pvrusb2/, Mauro Carvalho Chehab, (Tue May 13, 11:46 am)
Re: [patch] video: build fix for drivers/media/video/pvrusb2/, Mauro Carvalho Chehab, (Wed May 14, 2:01 am)
speck-geostationary