media: fix radio-sf16fmr2 build when SND is not enabled

Message ID 20110630103104.0a5b8ce6.randy.dunlap@oracle.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Randy Dunlap June 30, 2011, 5:31 p.m. UTC
  From: Randy Dunlap <randy.dunlap@oracle.com>

When CONFIG_SND is not enabled, radio-sf16fmr2 build fails with:

ERROR: "snd_tea575x_init" [drivers/media/radio/radio-sf16fmr2.ko] undefined!
ERROR: "snd_tea575x_exit" [drivers/media/radio/radio-sf16fmr2.ko] undefined!

so make this driver depend on SND.

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: linux-media@vger.kernel.org
---
 drivers/media/radio/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  

Comments

Ondrej Zary June 30, 2011, 6:36 p.m. UTC | #1
On Thursday 30 June 2011 19:31:04 Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@oracle.com>
> 
> When CONFIG_SND is not enabled, radio-sf16fmr2 build fails with:
> 
> ERROR: "snd_tea575x_init" [drivers/media/radio/radio-sf16fmr2.ko] undefined!
> ERROR: "snd_tea575x_exit" [drivers/media/radio/radio-sf16fmr2.ko] undefined!
> 
> so make this driver depend on SND.

I broke this when converting the driver to use common TEA575x code.
Thanks for finding and fixing it.

> Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
> Cc: Hans Verkuil <hverkuil@xs4all.nl>
> Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
> Cc: linux-media@vger.kernel.org
> ---
>  drivers/media/radio/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-next-20110630.orig/drivers/media/radio/Kconfig
> +++ linux-next-20110630/drivers/media/radio/Kconfig
> @@ -201,7 +201,7 @@ config RADIO_SF16FMI
>  
>  config RADIO_SF16FMR2
>  	tristate "SF16FMR2 Radio"
> -	depends on ISA && VIDEO_V4L2
> +	depends on ISA && VIDEO_V4L2 && SND
>  	---help---
>  	  Choose Y here if you have one of these FM radio cards.
  

Patch

--- linux-next-20110630.orig/drivers/media/radio/Kconfig
+++ linux-next-20110630/drivers/media/radio/Kconfig
@@ -201,7 +201,7 @@  config RADIO_SF16FMI
 
 config RADIO_SF16FMR2
 	tristate "SF16FMR2 Radio"
-	depends on ISA && VIDEO_V4L2
+	depends on ISA && VIDEO_V4L2 && SND
 	---help---
 	  Choose Y here if you have one of these FM radio cards.