Re: [PATCH] DVB: dvb-net, make the kconfig text helpful

[Randy Dunlap <randy.dunlap@oracle.com>]

On Thu, 16 Jun 2011 21:06:56 +0200 Jiri Slaby wrote:

> Telling the user they can disable an option if they want is not the
> much useful. Describe what it is good for instead.
> 
> The text was derived from Mauro's email.
> 
> Signed-off-by: Jiri Slaby <jslaby@suse.cz>
> Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
> Cc: Hans Petter Selasky <hselasky@c2i.net>

Yes, much better.  Thanks.

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

> ---
>  drivers/media/Kconfig |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
> index dc61895..279e2b9 100644
> --- a/drivers/media/Kconfig
> +++ b/drivers/media/Kconfig
> @@ -89,11 +89,13 @@ config DVB_NET
>  	default (NET && INET)
>  	depends on NET && INET
>  	help
> -	  The DVB network support in the DVB core can
> -	  optionally be disabled if this
> -	  option is set to N.
> +	  This option enables DVB Network Support which is a part of the DVB
> +	  standard. It is used, for example, by automatic firmware updates used
> +	  on Set-Top-Boxes. It can also be used to access the Internet via the
> +	  DVB card, if the network provider supports it.
>  
> -	  If unsure say Y.
> +	  You may want to disable the network support on embedded devices. If
> +	  unsure say Y.
>  
>  config VIDEO_MEDIA
>  	tristate
> -- 
> 1.7.5.4
> 
> 
> --
> 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/


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***