diff mbox

Improve media Kconfig menu

Message ID 201210191304.22294.hverkuil@xs4all.nl (mailing list archive)
State New, archived
Headers show

Commit Message

Hans Verkuil Oct. 19, 2012, 11:04 a.m. UTC
I've always found it very confusing that the "Media ancillary drivers (tuners,
sensors, i2c, frontends)" comment came after the "Autoselect" option. This patch
moves it up and changes the "Autoselect" text to correspond more closely to
the "Media ancillary drivers" comment.

It also fixes two typos.

Regards,

	Hans

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

Mauro Carvalho Chehab Oct. 28, 2012, 6:54 p.m. UTC | #1
Em Fri, 19 Oct 2012 13:04:22 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> I've always found it very confusing that the "Media ancillary drivers (tuners,
> sensors, i2c, frontends)" comment came after the "Autoselect" option. This patch
> moves it up and changes the "Autoselect" text to correspond more closely to
> the "Media ancillary drivers" comment.
> 
> It also fixes two typos.

Patch looks ok, except that there's no SOB ;)

Please re-send with your SOB.
> 
> Regards,
> 
> 	Hans
> 
> diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
> index 4ef0d80..9fbfb94 100644
> --- a/drivers/media/Kconfig
> +++ b/drivers/media/Kconfig
> @@ -158,17 +158,20 @@ source "drivers/media/firewire/Kconfig"
>  # Common driver options
>  source "drivers/media/common/Kconfig"
>  
> +comment "Media ancillary drivers (tuners, sensors, i2c, frontends)"
> +
>  #
>  # Ancillary drivers (tuners, i2c, frontends)
>  #
>  
>  config MEDIA_SUBDRV_AUTOSELECT
> -	bool "Autoselect tuners and i2c modules to build"
> +	bool "Autoselect ancillary drivers (tuners, sensors, i2c, frontends)"
>  	depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_CAMERA_SUPPORT
>  	default y
>  	help
> -	  By default, a media driver auto-selects all possible i2c
> -	  devices that are used by any of the supported devices.
> +	  By default, a media driver auto-selects all possible ancillary
> +	  devices such as tuners, sensors, video encoders/decoders and
> +	  frontends, that are used by any of the supported devices.
>  
>  	  This is generally the right thing to do, except when there
>  	  are strict constraints with regards to the kernel size,
> @@ -177,12 +180,10 @@ config MEDIA_SUBDRV_AUTOSELECT
>  	  Use this option with care, as deselecting ancillary drivers which
>  	  are, in fact, necessary will result in the lack of the needed
>  	  functionality for your device (it may not tune or may not have
> -	  the need demodulers).
> +	  the needed demodulators).
>  
>  	  If unsure say Y.
>  
> -comment "Media ancillary drivers (tuners, sensors, i2c, frontends)"
> -
>  source "drivers/media/i2c/Kconfig"
>  source "drivers/media/tuners/Kconfig"
>  source "drivers/media/dvb-frontends/Kconfig"
> --
> 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
diff mbox

Patch

diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
index 4ef0d80..9fbfb94 100644
--- a/drivers/media/Kconfig
+++ b/drivers/media/Kconfig
@@ -158,17 +158,20 @@  source "drivers/media/firewire/Kconfig"
 # Common driver options
 source "drivers/media/common/Kconfig"
 
+comment "Media ancillary drivers (tuners, sensors, i2c, frontends)"
+
 #
 # Ancillary drivers (tuners, i2c, frontends)
 #
 
 config MEDIA_SUBDRV_AUTOSELECT
-	bool "Autoselect tuners and i2c modules to build"
+	bool "Autoselect ancillary drivers (tuners, sensors, i2c, frontends)"
 	depends on MEDIA_ANALOG_TV_SUPPORT || MEDIA_DIGITAL_TV_SUPPORT || MEDIA_CAMERA_SUPPORT
 	default y
 	help
-	  By default, a media driver auto-selects all possible i2c
-	  devices that are used by any of the supported devices.
+	  By default, a media driver auto-selects all possible ancillary
+	  devices such as tuners, sensors, video encoders/decoders and
+	  frontends, that are used by any of the supported devices.
 
 	  This is generally the right thing to do, except when there
 	  are strict constraints with regards to the kernel size,
@@ -177,12 +180,10 @@  config MEDIA_SUBDRV_AUTOSELECT
 	  Use this option with care, as deselecting ancillary drivers which
 	  are, in fact, necessary will result in the lack of the needed
 	  functionality for your device (it may not tune or may not have
-	  the need demodulers).
+	  the needed demodulators).
 
 	  If unsure say Y.
 
-comment "Media ancillary drivers (tuners, sensors, i2c, frontends)"
-
 source "drivers/media/i2c/Kconfig"
 source "drivers/media/tuners/Kconfig"
 source "drivers/media/dvb-frontends/Kconfig"