* [PATCH/RFC] video gfx: merge kconfig menus
@ 2007-09-25 6:25 Randy Dunlap
2007-09-25 8:42 ` Dave Airlie
2007-09-29 0:12 ` Antonino A. Daplas
0 siblings, 2 replies; 3+ messages in thread
From: Randy Dunlap @ 2007-09-25 6:25 UTC (permalink / raw)
To: lkml; +Cc: airlied, adaplas, akpm
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 <randy.dunlap@oracle.com>
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 <randy.dunlap@oracle.com>
---
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
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH/RFC] video gfx: merge kconfig menus
2007-09-25 6:25 [PATCH/RFC] video gfx: merge kconfig menus Randy Dunlap
@ 2007-09-25 8:42 ` Dave Airlie
2007-09-29 0:12 ` Antonino A. Daplas
1 sibling, 0 replies; 3+ messages in thread
From: Dave Airlie @ 2007-09-25 8:42 UTC (permalink / raw)
To: Randy Dunlap; +Cc: lkml, airlied, adaplas, akpm
On 9/25/07, Randy Dunlap <randy.dunlap@oracle.com> 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 <randy.dunlap@oracle.com>
>
> 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 <randy.dunlap@oracle.com>
Fine by me.. some day we might move the directories but this is a good start..
Acked-by: Dave Airlie <airlied@linux.ie>
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 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/
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH/RFC] video gfx: merge kconfig menus
2007-09-25 6:25 [PATCH/RFC] video gfx: merge kconfig menus Randy Dunlap
2007-09-25 8:42 ` Dave Airlie
@ 2007-09-29 0:12 ` Antonino A. Daplas
1 sibling, 0 replies; 3+ messages in thread
From: Antonino A. Daplas @ 2007-09-29 0:12 UTC (permalink / raw)
To: Randy Dunlap; +Cc: lkml, airlied, akpm
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-09-29 0:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-09-25 6:25 [PATCH/RFC] video gfx: merge kconfig menus Randy Dunlap
2007-09-25 8:42 ` Dave Airlie
2007-09-29 0:12 ` Antonino A. Daplas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox