linux-kernel - Re: 2.6.22-rc6-mm1: TUNER

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <46881CBB.4020000@linuxtv.org>
Date:	Sun, 01 Jul 2007 17:29:31 -0400
From:	Michael Krufky <mkrufky@...uxtv.org>
To:	Adrian Bunk <bunk@...sta.de>
CC:	Andrew Morton <akpm@...ux-foundation.org>,
	Mauro Carvalho Chehab <mchehab@...radead.org>,
	linux-kernel@...r.kernel.org
Subject: Re: 2.6.22-rc6-mm1: TUNER_TEA5761 kconfig fixes

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@...sta.de>
>   
Acked-by: Michael Krufky <mkrufky@...uxtv.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/