[2.6 patch] add missing select's to DVB_BUDGET_AV

[Adrian Bunk]

On Tue, Aug 30, 2005 at 03:47:14PM -0400, Michael Krufky wrote:

> I wish I had seen this before 2.6.13 was released... I guess this only 
> goes to show that there haven't been any testers using saa7134-hybrid 
> dvb/v4l boards that depend on the tda1004x module, during the 2.6.13-rc 
> series :-(
> 
> Please apply this to 2.6.14, and also to 2.6.13.1 -stable.  Without this 
> patch, users will have to EXPLICITLY select tda1004x in Kconfig.  This 
> SHOULD be done automatically when saa7134-dvb is selected.  This patch 
> corrects this problem.
>...

What about the patch below fixing the following similar problems in 
drivers/media/dvb/ttpci/budget-ci.c ?

cu
Adrian


<--  snip  -->



Add missing select's to DVB_BUDGET_AV fixing the following compile 
error:

<--  snip  -->

...
  LD      .tmp_vmlinux1
drivers/built-in.o: In function `frontend_init':
budget-av.c:(.text+0xb9448): undefined reference to `tda10046_attach'
budget-av.c:(.text+0xb9518): undefined reference to `tda10021_attach'
drivers/built-in.o: In function `philips_tu1216_request_firmware':
budget-av.c:(.text+0xb937b): undefined reference to `request_firmware'
make: *** [.tmp_vmlinux1] Error 1

<--  snip  -->


Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>

--- linux-2.6.13/drivers/media/dvb/ttpci/Kconfig.old	2005-08-31 17:36:33.000000000 +0200
+++ linux-2.6.13/drivers/media/dvb/ttpci/Kconfig	2005-08-31 17:39:57.000000000 +0200
@@ -99,12 +99,15 @@
 config DVB_BUDGET_AV
 	tristate "Budget cards with analog video inputs"
 	depends on DVB_CORE && PCI
 	select VIDEO_DEV
 	select VIDEO_SAA7146_VV
 	select DVB_STV0299
+	select DVB_TDA1004X
+	select DVB_TDA10021
+	select FW_LOADER
 	help
 	  Support for simple SAA7146 based DVB cards
 	  (so called Budget- or Nova-PCI cards) without onboard
 	  MPEG2 decoder, but with one or more analog video inputs.
 
 	  Say Y if you own such a card and want to use it.

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