All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: xenomai-core <xenomai@xenomai.org>
Subject: [Xenomai-core] [PATCH] Reorder Kconfig menus
Date: Wed, 03 Jan 2007 18:51:56 +0100	[thread overview]
Message-ID: <459BED3C.4080400@domain.hid> (raw)


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

             reply	other threads:[~2007-01-03 17:51 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-03 17:51 Jan Kiszka [this message]
2007-01-03 18:29 ` [Xenomai-core] [PATCH] Reorder Kconfig menus Philippe Gerum

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=459BED3C.4080400@domain.hid \
    --to=jan.kiszka@domain.hid \
    --cc=xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.