2007-09-25 06:26:18

by Randy Dunlap

[permalink] [raw]
Subject: [PATCH/RFC] video gfx: merge kconfig menus

Is there some reason that we don't put all video gfx config in one
place? Is the split just historical, based on subdirectory locations,
or is there a bigger reason for it?

[to apply cleanly, this patch depends on a patch that was sent
to linux-fbdev-devel, which is subscribers-only, so I didn't cc: lkml
on it: http://marc.info/?l=linux-fbdev-devel&m=119070044232621&w=2]

---

From: Randy Dunlap <[email protected]>

Move AGP and DRM menus into the video graphics support menu.
They use 'menuconfig' so that they can all be disabled with
one selection.
Make the console menu use 'menuconfig' so that it can all be
disabled with one selection.
Make the frame buffer menu use 'menuconfig' so that it can all be
disabled with one selection.

Signed-off-by: Randy Dunlap <[email protected]>
---
drivers/char/Kconfig | 4 ----
drivers/char/agp/Kconfig | 2 +-
drivers/char/drm/Kconfig | 2 +-
drivers/video/Kconfig | 11 +++++++----
4 files changed, 9 insertions(+), 10 deletions(-)

--- linux-2.6.23-rc8.orig/drivers/char/Kconfig
+++ linux-2.6.23-rc8/drivers/char/Kconfig
@@ -896,10 +896,6 @@ config GPIO_TB0219
depends on TANBAC_TB022X
select GPIO_VR41XX

-source "drivers/char/agp/Kconfig"
-
-source "drivers/char/drm/Kconfig"
-
source "drivers/char/pcmcia/Kconfig"

config MWAVE
--- linux-2.6.23-rc8.orig/drivers/video/Kconfig
+++ linux-2.6.23-rc8/drivers/video/Kconfig
@@ -5,8 +5,9 @@
menu "Graphics support"
depends on HAS_IOMEM

-source "drivers/video/backlight/Kconfig"
-source "drivers/video/display/Kconfig"
+source "drivers/char/agp/Kconfig"
+
+source "drivers/char/drm/Kconfig"

config VGASTATE
tristate
@@ -19,7 +20,7 @@ config VIDEO_OUTPUT_CONTROL
This framework adds support for low-level control of the video
output switch.

-config FB
+menuconfig FB
tristate "Support for frame buffer devices"
---help---
The frame buffer device provides an abstraction for the graphics
@@ -1860,6 +1861,9 @@ if ARCH_OMAP
source "drivers/video/omap/Kconfig"
endif

+source "drivers/video/backlight/Kconfig"
+source "drivers/video/display/Kconfig"
+
if VT
source "drivers/video/console/Kconfig"
endif
@@ -1869,4 +1873,3 @@ if FB || SGI_NEWPORT_CONSOLE
endif

endmenu
-
--- linux-2.6.23-rc8.orig/drivers/char/agp/Kconfig
+++ linux-2.6.23-rc8/drivers/char/agp/Kconfig
@@ -1,4 +1,4 @@
-config AGP
+menuconfig AGP
tristate "/dev/agpgart (AGP Support)"
depends on ALPHA || IA64 || PARISC || PPC || X86
depends on PCI
--- linux-2.6.23-rc8.orig/drivers/char/drm/Kconfig
+++ linux-2.6.23-rc8/drivers/char/drm/Kconfig
@@ -4,7 +4,7 @@
# This driver provides support for the
# Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher.
#
-config DRM
+menuconfig DRM
tristate "Direct Rendering Manager (XFree86 4.1.0 and higher DRI support)"
depends on (AGP || AGP=n) && PCI && !EMULATED_CMPXCHG
help


2007-09-25 08:42:25

by Dave Airlie

[permalink] [raw]
Subject: Re: [PATCH/RFC] video gfx: merge kconfig menus

On 9/25/07, Randy Dunlap <[email protected]> wrote:
> Is there some reason that we don't put all video gfx config in one
> place? Is the split just historical, based on subdirectory locations,
> or is there a bigger reason for it?
>
> [to apply cleanly, this patch depends on a patch that was sent
> to linux-fbdev-devel, which is subscribers-only, so I didn't cc: lkml
> on it: http://marc.info/?l=linux-fbdev-devel&m=119070044232621&w=2]
>
> ---
>
> From: Randy Dunlap <[email protected]>
>
> Move AGP and DRM menus into the video graphics support menu.
> They use 'menuconfig' so that they can all be disabled with
> one selection.
> Make the console menu use 'menuconfig' so that it can all be
> disabled with one selection.
> Make the frame buffer menu use 'menuconfig' so that it can all be
> disabled with one selection.
>
> Signed-off-by: Randy Dunlap <[email protected]>

Fine by me.. some day we might move the directories but this is a good start..

Acked-by: Dave Airlie <[email protected]>

Dave.
> ---
> drivers/char/Kconfig | 4 ----
> drivers/char/agp/Kconfig | 2 +-
> drivers/char/drm/Kconfig | 2 +-
> drivers/video/Kconfig | 11 +++++++----
> 4 files changed, 9 insertions(+), 10 deletions(-)
>
> --- linux-2.6.23-rc8.orig/drivers/char/Kconfig
> +++ linux-2.6.23-rc8/drivers/char/Kconfig
> @@ -896,10 +896,6 @@ config GPIO_TB0219
> depends on TANBAC_TB022X
> select GPIO_VR41XX
>
> -source "drivers/char/agp/Kconfig"
> -
> -source "drivers/char/drm/Kconfig"
> -
> source "drivers/char/pcmcia/Kconfig"
>
> config MWAVE
> --- linux-2.6.23-rc8.orig/drivers/video/Kconfig
> +++ linux-2.6.23-rc8/drivers/video/Kconfig
> @@ -5,8 +5,9 @@
> menu "Graphics support"
> depends on HAS_IOMEM
>
> -source "drivers/video/backlight/Kconfig"
> -source "drivers/video/display/Kconfig"
> +source "drivers/char/agp/Kconfig"
> +
> +source "drivers/char/drm/Kconfig"
>
> config VGASTATE
> tristate
> @@ -19,7 +20,7 @@ config VIDEO_OUTPUT_CONTROL
> This framework adds support for low-level control of the video
> output switch.
>
> -config FB
> +menuconfig FB
> tristate "Support for frame buffer devices"
> ---help---
> The frame buffer device provides an abstraction for the graphics
> @@ -1860,6 +1861,9 @@ if ARCH_OMAP
> source "drivers/video/omap/Kconfig"
> endif
>
> +source "drivers/video/backlight/Kconfig"
> +source "drivers/video/display/Kconfig"
> +
> if VT
> source "drivers/video/console/Kconfig"
> endif
> @@ -1869,4 +1873,3 @@ if FB || SGI_NEWPORT_CONSOLE
> endif
>
> endmenu
> -
> --- linux-2.6.23-rc8.orig/drivers/char/agp/Kconfig
> +++ linux-2.6.23-rc8/drivers/char/agp/Kconfig
> @@ -1,4 +1,4 @@
> -config AGP
> +menuconfig AGP
> tristate "/dev/agpgart (AGP Support)"
> depends on ALPHA || IA64 || PARISC || PPC || X86
> depends on PCI
> --- linux-2.6.23-rc8.orig/drivers/char/drm/Kconfig
> +++ linux-2.6.23-rc8/drivers/char/drm/Kconfig
> @@ -4,7 +4,7 @@
> # This driver provides support for the
> # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher.
> #
> -config DRM
> +menuconfig DRM
> tristate "Direct Rendering Manager (XFree86 4.1.0 and higher DRI support)"
> depends on (AGP || AGP=n) && PCI && !EMULATED_CMPXCHG
> help
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

2007-09-29 00:12:37

by Antonino A. Daplas

[permalink] [raw]
Subject: Re: [PATCH/RFC] video gfx: merge kconfig menus

On Mon, 2007-09-24 at 23:25 -0700, Randy Dunlap wrote:
> Is there some reason that we don't put all video gfx config in one
> place? Is the split just historical, based on subdirectory locations,
> or is there a bigger reason for it?

Just historical, based on subdirectory locations. Someone did attempt to
move the drm subdirectory under the video subdirectory during the 2.5
era but it was rejected.

Anyway, I'm fine with this too.

Tony