diff mbox

[RFC,5/6] V4L menu: move all platform drivers to the bottom of the menu.

Message ID 99c353d49539e4a2a8f165db612ed6a7e82a57b9.1314797675.git.hans.verkuil@cisco.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hans Verkuil Aug. 31, 2011, 1:38 p.m. UTC 
From: Hans Verkuil <hans.verkuil@cisco.com>

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
---
 drivers/media/video/Kconfig |  106 ++++++++++++++++++++++---------------------
 1 files changed, 55 insertions(+), 51 deletions(-)

Comments

Mauro Carvalho Chehab Aug. 31, 2011, 1:50 p.m. UTC | #1 
Em 31-08-2011 10:38, Hans Verkuil escreveu:
> From: Hans Verkuil <hans.verkuil@cisco.com>

IMO, a submenu for those drivers makes sense.

> 
> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> ---
>  drivers/media/video/Kconfig |  106 ++++++++++++++++++++++---------------------
>  1 files changed, 55 insertions(+), 51 deletions(-)
> 
> diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig
> index 5beff36..d14da37 100644
> --- a/drivers/media/video/Kconfig
> +++ b/drivers/media/video/Kconfig
> @@ -630,25 +630,6 @@ config USB_S2255
>  
>  endif # V4L_USB_DRIVERS
>  
> -config VIDEO_SH_VOU
> -	tristate "SuperH VOU video output driver"
> -	depends on VIDEO_DEV && ARCH_SHMOBILE
> -	select VIDEOBUF_DMA_CONTIG
> -	help
> -	  Support for the Video Output Unit (VOU) on SuperH SoCs.
> -
> -config VIDEO_VIU
> -	tristate "Freescale VIU Video Driver"
> -	depends on VIDEO_V4L2 && PPC_MPC512x
> -	select VIDEOBUF_DMA_CONTIG
> -	default y
> -	---help---
> -	  Support for Freescale VIU video driver. This device captures
> -	  video data, or overlays video on DIU frame buffer.
> -
> -	  Say Y here if you want to enable VIU device on MPC5121e Rev2+.
> -	  In doubt, say N.
> -
>  config VIDEO_VIVI
>  	tristate "Virtual Video Driver"
>  	depends on VIDEO_DEV && VIDEO_V4L2 && !SPARC32 && !SPARC64
> @@ -663,20 +644,8 @@ config VIDEO_VIVI
>  	  Say Y here if you want to test video apps or debug V4L devices.
>  	  In doubt, say N.
>  
> -source "drivers/media/video/davinci/Kconfig"
> -
> -source "drivers/media/video/omap/Kconfig"
> -
>  source "drivers/media/video/bt8xx/Kconfig"
>  
> -config VIDEO_VINO
> -	tristate "SGI Vino Video For Linux"
> -	depends on I2C && SGI_IP22 && VIDEO_V4L2
> -	select VIDEO_SAA7191 if VIDEO_HELPER_CHIPS_AUTO
> -	help
> -	  Say Y here to build in support for the Vino video input system found
> -	  on SGI Indy machines.
> -
>  source "drivers/media/video/zoran/Kconfig"
>  
>  config VIDEO_MEYE
> @@ -695,16 +664,6 @@ config VIDEO_MEYE
>  
>  source "drivers/media/video/saa7134/Kconfig"
>  
> -config VIDEO_TIMBERDALE
> -	tristate "Support for timberdale Video In/LogiWIN"
> -	depends on VIDEO_V4L2 && I2C && DMADEVICES
> -	select DMA_ENGINE
> -	select TIMB_DMA
> -	select VIDEO_ADV7180
> -	select VIDEOBUF_DMA_CONTIG
> -	---help---
> -	  Add support for the Video In peripherial of the timberdale FPGA.
> -
>  source "drivers/media/video/cx88/Kconfig"
>  
>  source "drivers/media/video/cx23885/Kconfig"
> @@ -719,6 +678,61 @@ source "drivers/media/video/saa7164/Kconfig"
>  
>  source "drivers/media/video/marvell-ccic/Kconfig"
>  
> +config VIDEO_VIA_CAMERA
> +	tristate "VIAFB camera controller support"
> +	depends on FB_VIA
> +	select VIDEOBUF_DMA_SG
> +	select VIDEO_OV7670
> +	help
> +	   Driver support for the integrated camera controller in VIA
> +	   Chrome9 chipsets.  Currently only tested on OLPC xo-1.5 systems
> +	   with ov7670 sensors.
> +
> +#
> +# Platform multimedia device configuration
> +#
> +
> +source "drivers/media/video/davinci/Kconfig"
> +
> +source "drivers/media/video/omap/Kconfig"
> +
> +config VIDEO_SH_VOU
> +	tristate "SuperH VOU video output driver"
> +	depends on VIDEO_DEV && ARCH_SHMOBILE
> +	select VIDEOBUF_DMA_CONTIG
> +	help
> +	  Support for the Video Output Unit (VOU) on SuperH SoCs.
> +
> +config VIDEO_VIU
> +	tristate "Freescale VIU Video Driver"
> +	depends on VIDEO_V4L2 && PPC_MPC512x
> +	select VIDEOBUF_DMA_CONTIG
> +	default y
> +	---help---
> +	  Support for Freescale VIU video driver. This device captures
> +	  video data, or overlays video on DIU frame buffer.
> +
> +	  Say Y here if you want to enable VIU device on MPC5121e Rev2+.
> +	  In doubt, say N.
> +
> +config VIDEO_TIMBERDALE
> +	tristate "Support for timberdale Video In/LogiWIN"
> +	depends on VIDEO_V4L2 && I2C && DMADEVICES
> +	select DMA_ENGINE
> +	select TIMB_DMA
> +	select VIDEO_ADV7180
> +	select VIDEOBUF_DMA_CONTIG
> +	---help---
> +	  Add support for the Video In peripherial of the timberdale FPGA.
> +
> +config VIDEO_VINO
> +	tristate "SGI Vino Video For Linux"
> +	depends on I2C && SGI_IP22 && VIDEO_V4L2
> +	select VIDEO_SAA7191 if VIDEO_HELPER_CHIPS_AUTO
> +	help
> +	  Say Y here to build in support for the Vino video input system found
> +	  on SGI Indy machines.
> +
>  config VIDEO_M32R_AR
>  	tristate "AR devices"
>  	depends on M32R && VIDEO_V4L2
> @@ -738,16 +752,6 @@ config VIDEO_M32R_AR_M64278
>  	  To compile this driver as a module, choose M here: the
>  	  module will be called arv.
>  
> -config VIDEO_VIA_CAMERA
> -	tristate "VIAFB camera controller support"
> -	depends on FB_VIA
> -	select VIDEOBUF_DMA_SG
> -	select VIDEO_OV7670
> -	help
> -	   Driver support for the integrated camera controller in VIA
> -	   Chrome9 chipsets.  Currently only tested on OLPC xo-1.5 systems
> -	   with ov7670 sensors.
> -
>  config VIDEO_OMAP3
>  	tristate "OMAP 3 Camera support (EXPERIMENTAL)"
>  	select OMAP_IOMMU

--
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
Hans Verkuil Aug. 31, 2011, 2:13 p.m. UTC | #2 
On Wednesday, August 31, 2011 15:50:35 Mauro Carvalho Chehab wrote:
> Em 31-08-2011 10:38, Hans Verkuil escreveu:
> > From: Hans Verkuil <hans.verkuil@cisco.com>
> 
> IMO, a submenu for those drivers makes sense.

I thought about that, but actually you never see any of these drivers
unless you have the right architecture options enabled.

So if you would put them in a submenu, then you'll end up with just a
few entries. It's not worth it in my opinion.

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

Patch

diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig
index 5beff36..d14da37 100644
--- a/drivers/media/video/Kconfig
+++ b/drivers/media/video/Kconfig
@@ -630,25 +630,6 @@  config USB_S2255
 
 endif # V4L_USB_DRIVERS
 
-config VIDEO_SH_VOU
-	tristate "SuperH VOU video output driver"
-	depends on VIDEO_DEV && ARCH_SHMOBILE
-	select VIDEOBUF_DMA_CONTIG
-	help
-	  Support for the Video Output Unit (VOU) on SuperH SoCs.
-
-config VIDEO_VIU
-	tristate "Freescale VIU Video Driver"
-	depends on VIDEO_V4L2 && PPC_MPC512x
-	select VIDEOBUF_DMA_CONTIG
-	default y
-	---help---
-	  Support for Freescale VIU video driver. This device captures
-	  video data, or overlays video on DIU frame buffer.
-
-	  Say Y here if you want to enable VIU device on MPC5121e Rev2+.
-	  In doubt, say N.
-
 config VIDEO_VIVI
 	tristate "Virtual Video Driver"
 	depends on VIDEO_DEV && VIDEO_V4L2 && !SPARC32 && !SPARC64
@@ -663,20 +644,8 @@  config VIDEO_VIVI
 	  Say Y here if you want to test video apps or debug V4L devices.
 	  In doubt, say N.
 
-source "drivers/media/video/davinci/Kconfig"
-
-source "drivers/media/video/omap/Kconfig"
-
 source "drivers/media/video/bt8xx/Kconfig"
 
-config VIDEO_VINO
-	tristate "SGI Vino Video For Linux"
-	depends on I2C && SGI_IP22 && VIDEO_V4L2
-	select VIDEO_SAA7191 if VIDEO_HELPER_CHIPS_AUTO
-	help
-	  Say Y here to build in support for the Vino video input system found
-	  on SGI Indy machines.
-
 source "drivers/media/video/zoran/Kconfig"
 
 config VIDEO_MEYE
@@ -695,16 +664,6 @@  config VIDEO_MEYE
 
 source "drivers/media/video/saa7134/Kconfig"
 
-config VIDEO_TIMBERDALE
-	tristate "Support for timberdale Video In/LogiWIN"
-	depends on VIDEO_V4L2 && I2C && DMADEVICES
-	select DMA_ENGINE
-	select TIMB_DMA
-	select VIDEO_ADV7180
-	select VIDEOBUF_DMA_CONTIG
-	---help---
-	  Add support for the Video In peripherial of the timberdale FPGA.
-
 source "drivers/media/video/cx88/Kconfig"
 
 source "drivers/media/video/cx23885/Kconfig"
@@ -719,6 +678,61 @@  source "drivers/media/video/saa7164/Kconfig"
 
 source "drivers/media/video/marvell-ccic/Kconfig"
 
+config VIDEO_VIA_CAMERA
+	tristate "VIAFB camera controller support"
+	depends on FB_VIA
+	select VIDEOBUF_DMA_SG
+	select VIDEO_OV7670
+	help
+	   Driver support for the integrated camera controller in VIA
+	   Chrome9 chipsets.  Currently only tested on OLPC xo-1.5 systems
+	   with ov7670 sensors.
+
+#
+# Platform multimedia device configuration
+#
+
+source "drivers/media/video/davinci/Kconfig"
+
+source "drivers/media/video/omap/Kconfig"
+
+config VIDEO_SH_VOU
+	tristate "SuperH VOU video output driver"
+	depends on VIDEO_DEV && ARCH_SHMOBILE
+	select VIDEOBUF_DMA_CONTIG
+	help
+	  Support for the Video Output Unit (VOU) on SuperH SoCs.
+
+config VIDEO_VIU
+	tristate "Freescale VIU Video Driver"
+	depends on VIDEO_V4L2 && PPC_MPC512x
+	select VIDEOBUF_DMA_CONTIG
+	default y
+	---help---
+	  Support for Freescale VIU video driver. This device captures
+	  video data, or overlays video on DIU frame buffer.
+
+	  Say Y here if you want to enable VIU device on MPC5121e Rev2+.
+	  In doubt, say N.
+
+config VIDEO_TIMBERDALE
+	tristate "Support for timberdale Video In/LogiWIN"
+	depends on VIDEO_V4L2 && I2C && DMADEVICES
+	select DMA_ENGINE
+	select TIMB_DMA
+	select VIDEO_ADV7180
+	select VIDEOBUF_DMA_CONTIG
+	---help---
+	  Add support for the Video In peripherial of the timberdale FPGA.
+
+config VIDEO_VINO
+	tristate "SGI Vino Video For Linux"
+	depends on I2C && SGI_IP22 && VIDEO_V4L2
+	select VIDEO_SAA7191 if VIDEO_HELPER_CHIPS_AUTO
+	help
+	  Say Y here to build in support for the Vino video input system found
+	  on SGI Indy machines.
+
 config VIDEO_M32R_AR
 	tristate "AR devices"
 	depends on M32R && VIDEO_V4L2
@@ -738,16 +752,6 @@  config VIDEO_M32R_AR_M64278
 	  To compile this driver as a module, choose M here: the
 	  module will be called arv.
 
-config VIDEO_VIA_CAMERA
-	tristate "VIAFB camera controller support"
-	depends on FB_VIA
-	select VIDEOBUF_DMA_SG
-	select VIDEO_OV7670
-	help
-	   Driver support for the integrated camera controller in VIA
-	   Chrome9 chipsets.  Currently only tested on OLPC xo-1.5 systems
-	   with ov7670 sensors.
-
 config VIDEO_OMAP3
 	tristate "OMAP 3 Camera support (EXPERIMENTAL)"
 	select OMAP_IOMMU