All of lore.kernel.org
 help / color / mirror / Atom feed
* [Xenomai-core] [PATCH] Reorder Kconfig menus
@ 2007-01-03 17:51 Jan Kiszka
  2007-01-03 18:29 ` Philippe Gerum
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Kiszka @ 2007-01-03 17:51 UTC (permalink / raw)
  To: xenomai-core


[-- Attachment #1.1: Type: text/plain, Size: 211 bytes --]

The menu ordering bugged me, specifically under the skins menu with all
the new period options. There was some bug /wrt RTDM and some redundancy
around the drivers menu. Any concerns about this patch?

Jan

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: Kconfig-cleanup.patch --]
[-- Type: text/x-patch; name="Kconfig-cleanup.patch", Size: 10081 bytes --]

Index: ksrc/arch/arm/Kconfig
===================================================================
--- ksrc/arch/arm/Kconfig	(Revision 2044)
+++ ksrc/arch/arm/Kconfig	(Arbeitskopie)
@@ -19,9 +19,4 @@ config XENO_HW_FPU
 endmenu
 
 source "kernel/xenomai/skins/Kconfig"
-
-menu "Drivers"
-
 source "drivers/xenomai/Kconfig"
-
-endmenu
Index: ksrc/arch/i386/Kconfig
===================================================================
--- ksrc/arch/i386/Kconfig	(Revision 2044)
+++ ksrc/arch/i386/Kconfig	(Arbeitskopie)
@@ -186,11 +186,4 @@ endmenu
 endmenu
 
 source "kernel/xenomai/skins/Kconfig"
-
-menu "Drivers"
-
-depends on XENO_OPT_NUCLEUS
-
 source "drivers/xenomai/Kconfig"
-
-endmenu
Index: ksrc/arch/ia64/Kconfig
===================================================================
--- ksrc/arch/ia64/Kconfig	(Revision 2044)
+++ ksrc/arch/ia64/Kconfig	(Arbeitskopie)
@@ -20,9 +20,4 @@ config XENO_HW_IA64_STACK_POOL
 endmenu
 
 source "kernel/xenomai/skins/Kconfig"
-
-menu "Drivers"
-
 source "drivers/xenomai/Kconfig"
-
-endmenu
Index: ksrc/arch/blackfin/Kconfig
===================================================================
--- ksrc/arch/blackfin/Kconfig	(Revision 2044)
+++ ksrc/arch/blackfin/Kconfig	(Arbeitskopie)
@@ -36,9 +36,4 @@ endmenu
 endmenu
 
 source "kernel/xenomai/skins/Kconfig"
-
-menu "Drivers"
-
 source "drivers/xenomai/Kconfig"
-
-endmenu
Index: ksrc/arch/powerpc/Kconfig
===================================================================
--- ksrc/arch/powerpc/Kconfig	(Revision 2044)
+++ ksrc/arch/powerpc/Kconfig	(Arbeitskopie)
@@ -20,9 +20,4 @@ config XENO_HW_FPU
 endmenu
 
 source "kernel/xenomai/skins/Kconfig"
-
-menu "Drivers"
-
 source "drivers/xenomai/Kconfig"
-
-endmenu
Index: ksrc/skins/rtai/Kconfig
===================================================================
--- ksrc/skins/rtai/Kconfig	(Revision 2044)
+++ ksrc/skins/rtai/Kconfig	(Arbeitskopie)
@@ -1,14 +1,14 @@
-config XENO_SKIN_RTAI
+menuconfig XENO_SKIN_RTAI
 	depends on XENO_OPT_NUCLEUS
 	tristate "RTAI emulator"
-	default n
 	help
 
 	An emulator of the RTAI system.
 
+if XENO_SKIN_RTAI
+
 config XENO_OPT_RTAI_PERIOD
-	depends on XENO_OPT_TIMING_PERIODIC && XENO_SKIN_RTAI
-	int "Base period (us)"
+	int "Base period (us)" if XENO_OPT_TIMING_PERIODIC
 	default 0
 	help
 	Duration of the base periodic tick (in microseconds).
@@ -24,37 +24,29 @@ config XENO_OPT_RTAI_PERIOD
 
 	If in doubt, leave zero in this field.
 
-if XENO_OPT_TIMING_PERIODIC != y
-
-config XENO_OPT_RTAI_PERIOD
-	int
-	default 0
-
-endif
-
 config XENO_OPT_RTAI_FIFO
 	bool "Fifos"
-	depends on XENO_OPT_PIPE && XENO_SKIN_RTAI
+	select XENO_OPT_PIPE
 	default y
 	help
-	
+
 	Fifos are uni-directional communication channels allowing data
 	exchange between real-time tasks in kernel-space and regular
 	user-space processes.
 
 config XENO_OPT_RTAI_SEM
 	bool "Semaphores"
-	depends on XENO_SKIN_RTAI
 	default y
 	help
-	
+
 	Semaphores are synchronization objects granting RTAI
 	tasks a concurrent access to a given number of resources.
 
 config XENO_OPT_RTAI_SHM
 	bool "Shared Memory"
-	depends on XENO_SKIN_RTAI
 	default y
 	help
-	
+
 	SHM are Shared Memory objects across RTAI tasks.
+
+endif
Index: ksrc/skins/rtdm/Kconfig
===================================================================
--- ksrc/skins/rtdm/Kconfig	(Revision 2044)
+++ ksrc/skins/rtdm/Kconfig	(Arbeitskopie)
@@ -1,4 +1,4 @@
-config XENO_SKIN_RTDM
+menuconfig XENO_SKIN_RTDM
 	depends on XENO_OPT_NUCLEUS
 	tristate "Real-Time Driver Model"
 	default y
@@ -8,9 +8,10 @@ config XENO_SKIN_RTDM
 	light weight interface in kernel mode, but use them across all other
 	skins in both kernel and user mode.
 
+if XENO_SKIN_RTDM
+
 config XENO_OPT_RTDM_PERIOD
-	depends on XENO_OPT_TIMING_PERIODIC
-	int "Base period (us)"
+	int "Base period (us)" if XENO_OPT_TIMING_PERIODIC
 	default 0
 	help
 	Duration of the base periodic tick (in microseconds).
@@ -26,17 +27,8 @@ config XENO_OPT_RTDM_PERIOD
 
 	If in doubt, leave zero in this field.
 
-if XENO_OPT_TIMING_PERIODIC != y
-
-config XENO_OPT_RTDM_PERIOD
-	int
-	default 0
-
-endif
-
 config XENO_OPT_RTDM_FILDES
 	int "Maximum number of RTDM file descriptors"
-	depends on XENO_SKIN_RTDM
 	default 128
 	help
 
@@ -47,10 +39,12 @@ config XENO_OPT_RTDM_FILDES
 
 config XENO_OPT_DEBUG_RTDM
 	bool "RTDM debugging support"
-	depends on XENO_OPT_DEBUG && XENO_SKIN_RTDM
+	depends on XENO_OPT_DEBUG
 	default y
 	help
 
 	This option activates debugging checks for the RTDM subsystem.
 	It is a recommended option for analysing potential issues in RTDM
 	drivers. A minor runtime overhead is added.
+
+endif
Index: ksrc/skins/vrtx/Kconfig
===================================================================
--- ksrc/skins/vrtx/Kconfig	(Revision 2044)
+++ ksrc/skins/vrtx/Kconfig	(Arbeitskopie)
@@ -1,4 +1,4 @@
-config XENO_SKIN_VRTX
+menuconfig XENO_SKIN_VRTX
 	depends on XENO_OPT_NUCLEUS
 	select XENO_OPT_TIMING_PERIODIC
 	tristate "VRTX emulator"
Index: ksrc/skins/posix/Kconfig
===================================================================
--- ksrc/skins/posix/Kconfig	(Revision 2044)
+++ ksrc/skins/posix/Kconfig	(Arbeitskopie)
@@ -1,4 +1,4 @@
-config XENO_SKIN_POSIX
+menuconfig XENO_SKIN_POSIX
 	depends on XENO_OPT_NUCLEUS
 	tristate "POSIX API"
 	default y
@@ -6,11 +6,10 @@ config XENO_SKIN_POSIX
 
 	A POSIX-compliant skin.
 
-if XENO_SKIN_POSIX != n
+if XENO_SKIN_POSIX
 
 config XENO_OPT_POSIX_PERIOD
-	depends on XENO_OPT_TIMING_PERIODIC
-	int "Base period (us)"
+	int "Base period (us)" if XENO_OPT_TIMING_PERIODIC
 	default 0
 	help
 	Duration of the base periodic tick (in microseconds).
@@ -26,14 +25,6 @@ config XENO_OPT_POSIX_PERIOD
 
 	If in doubt, leave zero in this field.
 
-if XENO_OPT_TIMING_PERIODIC != y
-
-config XENO_OPT_POSIX_PERIOD
-	int
-	default 0
-
-endif
-
 config XENO_OPT_POSIX_SHM
 	bool "Shared memory"
 	help
Index: ksrc/skins/psos+/Kconfig
===================================================================
--- ksrc/skins/psos+/Kconfig	(Revision 2044)
+++ ksrc/skins/psos+/Kconfig	(Arbeitskopie)
@@ -1,8 +1,7 @@
-config XENO_SKIN_PSOS
+menuconfig XENO_SKIN_PSOS
 	depends on XENO_OPT_NUCLEUS
 	select XENO_OPT_TIMING_PERIODIC
 	tristate "pSOS+ emulator"
-	default n
 	help
 
 	This API skin emulates WindRiver's pSOS+ operating system.
Index: ksrc/skins/Kconfig
===================================================================
--- ksrc/skins/Kconfig	(Revision 2044)
+++ ksrc/skins/Kconfig	(Arbeitskopie)
@@ -3,19 +3,12 @@ menu "Interfaces"
 depends on XENO_OPT_NUCLEUS
 
 source "kernel/xenomai/skins/native/Kconfig"
-
 source "kernel/xenomai/skins/posix/Kconfig"
-
 source "kernel/xenomai/skins/psos+/Kconfig"
-
 source "kernel/xenomai/skins/uitron/Kconfig"
-
 source "kernel/xenomai/skins/vrtx/Kconfig"
-
 source "kernel/xenomai/skins/vxworks/Kconfig"
-
 source "kernel/xenomai/skins/rtai/Kconfig"
-
 source "kernel/xenomai/skins/rtdm/Kconfig"
 
 endmenu
Index: ksrc/skins/native/Kconfig
===================================================================
--- ksrc/skins/native/Kconfig	(Revision 2044)
+++ ksrc/skins/native/Kconfig	(Arbeitskopie)
@@ -1,4 +1,4 @@
-config XENO_SKIN_NATIVE
+menuconfig XENO_SKIN_NATIVE
 	depends on XENO_OPT_NUCLEUS
 	tristate "Native API"
 	default y
@@ -6,11 +6,10 @@ config XENO_SKIN_NATIVE
 
 	The native Xenomai API.
 
-if XENO_SKIN_NATIVE != n
+if XENO_SKIN_NATIVE
 
 config XENO_OPT_NATIVE_PERIOD
-	depends on XENO_OPT_TIMING_PERIODIC
-	int "Base period (us)"
+	int "Base period (us)" if XENO_OPT_TIMING_PERIODIC
 	default 0
 	help
 	Duration of the base periodic tick (in microseconds).
@@ -26,14 +25,6 @@ config XENO_OPT_NATIVE_PERIOD
 
 	If in doubt, leave zero in this field.
 
-if XENO_OPT_TIMING_PERIODIC != y
-
-config XENO_OPT_NATIVE_PERIOD
-	int
-	default 0
-
-endif
-
 config XENO_OPT_NATIVE_PIPE
 	bool "Message pipes"
 	select XENO_OPT_PIPE
@@ -91,7 +82,7 @@ config XENO_OPT_NATIVE_EVENT
 
 config XENO_OPT_NATIVE_MUTEX
 	bool "Mutexes"
-	default y if XENO_OPT_NATIVE_COND=y
+	default y if XENO_OPT_NATIVE_COND
 	help
 	
 	Mutexes are MUTual EXclusion objects, useful for protecting
Index: ksrc/skins/uitron/Kconfig
===================================================================
--- ksrc/skins/uitron/Kconfig	(Revision 2044)
+++ ksrc/skins/uitron/Kconfig	(Arbeitskopie)
@@ -1,8 +1,7 @@
-config XENO_SKIN_UITRON
+menuconfig XENO_SKIN_UITRON
 	depends on XENO_OPT_NUCLEUS
 	select XENO_OPT_TIMING_PERIODIC
 	tristate "uITRON API"
-	default n
 	help
 
 	This skin implements the uITRON API. 
Index: ksrc/skins/vxworks/Kconfig
===================================================================
--- ksrc/skins/vxworks/Kconfig	(Revision 2044)
+++ ksrc/skins/vxworks/Kconfig	(Arbeitskopie)
@@ -1,4 +1,4 @@
-config XENO_SKIN_VXWORKS
+menuconfig XENO_SKIN_VXWORKS
 	depends on XENO_OPT_NUCLEUS
 	select XENO_OPT_TIMING_PERIODIC
 	tristate "VxWorks emulator"
Index: ksrc/drivers/Kconfig
===================================================================
--- ksrc/drivers/Kconfig	(Revision 2044)
+++ ksrc/drivers/Kconfig	(Arbeitskopie)
@@ -1,3 +1,9 @@
+menu "Drivers"
+
+depends on XENO_OPT_NUCLEUS
+
 source "drivers/xenomai/serial/Kconfig"
 source "drivers/xenomai/testing/Kconfig"
 source "drivers/xenomai/can/Kconfig"
+
+endmenu
Index: ChangeLog
===================================================================
--- ChangeLog	(Revision 2044)
+++ ChangeLog	(Arbeitskopie)
@@ -1,3 +1,8 @@
+2007-01-03  Jan Kiszka  <jan.kiszka@domain.hid>
+
+	* ksrc/skins/*/Kconfig, ksrc/arch/*/Kconfig, ksrc/drivers/Kconfig:
+	Reorganise and clean up menus.
+
 2007-01-02  Gilles Chanteperdrix  <gilles.chanteperdrix@xenomai.org>
 
 	* ksrc/nucleus/pod.c (xnpod_announce_tick): Remove support for

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Xenomai-core] [PATCH] Reorder Kconfig menus
  2007-01-03 17:51 [Xenomai-core] [PATCH] Reorder Kconfig menus Jan Kiszka
@ 2007-01-03 18:29 ` Philippe Gerum
  0 siblings, 0 replies; 2+ messages in thread
From: Philippe Gerum @ 2007-01-03 18:29 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: xenomai-core

On Wed, 2007-01-03 at 18:51 +0100, Jan Kiszka wrote:
> The menu ordering bugged me, specifically under the skins menu with all
> the new period options. There was some bug /wrt RTDM and some redundancy
> around the drivers menu. Any concerns about this patch?

No, please merge.

> 
> Jan
> _______________________________________________
> Xenomai-core mailing list
> Xenomai-core@domain.hid
> https://mail.gna.org/listinfo/xenomai-core
-- 
Philippe.




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-01-03 18:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-01-03 17:51 [Xenomai-core] [PATCH] Reorder Kconfig menus Jan Kiszka
2007-01-03 18:29 ` Philippe Gerum

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.