Re: 2.6.22-rc6-mm1: TUNER_TEA5761 kconfig fixes

[Michael Krufky <mkrufky@linuxtv.org>]

Adrian Bunk wrote:
> On Thu, Jun 28, 2007 at 03:43:21AM -0700, Andrew Morton wrote:
>   
>> ...
>> Changes since 2.6.22-rc4-mm2:
>> ...
>>  git-dvb.patch
>> ...
>>  git trees
>> ...
>>     
>
>
> The following doesn't make much sense:
>
> drivers/media/video/Kconfig:
> ...
> config TUNER_TEA5761
>         tristate "TEA 5761 radio tuner (EXPERIMENTAL)"
> ...
>
> drivers/media/video/Makefile:
> ...
> ifneq ($(CONFIG_TUNER_TEA5761),)
>   tuner-objs += tea5761.o
> endif
> ...
>
> With this setup, TUNER_TEA5761=m is equivalent to TUNER_TEA5761=y.
>
> This patch therefore changes TUNER_TEA5761 to a bool.
>
> The missing dependency on EXPERIMENTAL the prompt text indicates also 
> gets added by this patch.
>
> Additionally, the Makefile entry can now be written in a more compact way.
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
>   
Acked-by: Michael Krufky <mkrufky@linuxtv.org>

Thanks, Adrian

Mauro,  I'll push this into hg and send you a pull request.
> ---
>
> BTW: tea5761.c is currently completely unused.
>
>  drivers/media/video/Kconfig  |    3 ++-
>  drivers/media/video/Makefile |    4 +---
>  2 files changed, 3 insertions(+), 4 deletions(-)
>
> --- linux-2.6.22-rc6-mm1/drivers/media/video/Kconfig.old	2007-06-30 03:51:13.000000000 +0200
> +++ linux-2.6.22-rc6-mm1/drivers/media/video/Kconfig	2007-06-30 03:51:40.000000000 +0200
> @@ -490,7 +490,8 @@
>  	  If in doubt, say N.
>  
>  config TUNER_TEA5761
> -	tristate "TEA 5761 radio tuner (EXPERIMENTAL)"
> +	bool "TEA 5761 radio tuner (EXPERIMENTAL)"
> +	depends on EXPERIMENTAL
>  	depends on I2C
>  	select VIDEO_TUNER
>  	help
> --- linux-2.6.22-rc6-mm1/drivers/media/video/Makefile.old	2007-06-30 03:52:00.000000000 +0200
> +++ linux-2.6.22-rc6-mm1/drivers/media/video/Makefile	2007-06-30 03:53:25.000000000 +0200
> @@ -7,9 +7,7 @@
>  tuner-objs	:=	tuner-core.o tuner-types.o tuner-simple.o \
>  			mt20xx.o tda8290.o tea5767.o tda9887.o
>  
> -ifneq ($(CONFIG_TUNER_TEA5761),)
> -  tuner-objs += tea5761.o
> -endif
> +tuner-$(CONFIG_TUNER_TEA5761)	+= tea5761.o
>  
>  msp3400-objs	:=	msp3400-driver.o msp3400-kthreads.o
>  
>   

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