Patch from Yann E. Morin to reorganize module config options.

diff --git a/modutils/Config.in b/modutils/Config.in
index bf19a3e6718aee7cdeb141e5e995f2d85330e14a..8f23be83671f09692ec32c85f316e02a9eaac04c 100644 (file)
--- a/modutils/Config.in
+++ b/modutils/Config.in
@@ -11,22 +11,8 @@ config CONFIG_INSMOD
        help
          insmod is used to load specified modules in the running kernel.
 
-config CONFIG_FEATURE_2_4_MODULES
-       bool "  Support version 2.2.x to 2.4.x Linux kernels"
-       default y
-       depends on CONFIG_INSMOD
-       help
-         Support module loading for 2.2.x and 2.4.x Linux kernels.
-
-config CONFIG_FEATURE_2_6_MODULES
-       bool "  Support version 2.6.x Linux kernels"
-       default n
-       depends on CONFIG_INSMOD 
-       help
-         Support module loading for newer 2.6.x Linux kernels.
-
 config CONFIG_FEATURE_INSMOD_VERSION_CHECKING
-       bool "  Module version checking"
+       bool "Module version checking"
        default n
        depends on CONFIG_INSMOD && CONFIG_FEATURE_2_4_MODULES
        help
@@ -34,7 +20,7 @@ config CONFIG_FEATURE_INSMOD_VERSION_CHECKING
          ensure that the kernel and module are made for each other.
 
 config CONFIG_FEATURE_INSMOD_KSYMOOPS_SYMBOLS
-       bool "  Add module symbols to kernel symbol table"
+       bool "Add module symbols to kernel symbol table"
        default n
        depends on CONFIG_INSMOD && CONFIG_FEATURE_2_4_MODULES
        help
@@ -45,7 +31,7 @@ config CONFIG_FEATURE_INSMOD_KSYMOOPS_SYMBOLS
          Oops messages from kernel modules, say N.
 
 config CONFIG_FEATURE_INSMOD_LOADINKMEM
-       bool "  In kernel memory optimization (uClinux only)"
+       bool "In kernel memory optimization (uClinux only)"
        default n
        depends on CONFIG_INSMOD && CONFIG_FEATURE_2_4_MODULES
        help
@@ -55,7 +41,7 @@ config CONFIG_FEATURE_INSMOD_LOADINKMEM
          being loaded into memory.
 
 config CONFIG_FEATURE_INSMOD_LOAD_MAP
-       bool "  Enable load map (-m) option"
+       bool "Enable load map (-m) option"
        default n
        depends on CONFIG_INSMOD && CONFIG_FEATURE_2_4_MODULES
        help
@@ -66,7 +52,7 @@ config CONFIG_FEATURE_INSMOD_LOAD_MAP
          don't need this option.
 
 config CONFIG_FEATURE_INSMOD_LOAD_MAP_FULL
-       bool "  Symbols in load map"
+       bool "Symbols in load map"
        default y
        depends on CONFIG_FEATURE_INSMOD_LOAD_MAP
        help
@@ -74,6 +60,12 @@ config CONFIG_FEATURE_INSMOD_LOAD_MAP_FULL
          load map.  With this option, -m will also output
          symbols load map.
 
+config CONFIG_RMMOD
+       bool "rmmod"
+       default n
+       help
+         rmmod is used to unload specified modules from the kernel.
+
 config CONFIG_LSMOD
        bool "lsmod"
        default n
@@ -81,18 +73,13 @@ config CONFIG_LSMOD
          lsmod is used to display a list of loaded modules.
 
 config CONFIG_FEATURE_LSMOD_PRETTY_2_6_OUTPUT
-       bool "  lsmod pretty output for 2.6.x Linux kernels "
+       bool "lsmod pretty output for 2.6.x Linux kernels "
        default n
        depends on CONFIG_LSMOD 
        help
          This option makes output format of lsmod adjusted to 
          the format of module-init-tools for Linux kernel 2.6.
 
-config CONFIG_FEATURE_QUERY_MODULE_INTERFACE
-       bool
-       default y
-       depends on CONFIG_FEATURE_2_4_MODULES && !CONFIG_FEATURE_2_6_MODULES
-
 config CONFIG_MODPROBE
        bool "modprobe"
        default n
@@ -100,33 +87,31 @@ config CONFIG_MODPROBE
          Handle the loading of modules, and their dependancies on a high
          level.
 
-         Note that, in the state it is, modprobe can pass only one option
-         to the modules it loads. See option below.
+         Note that in the state, modprobe does not understand multiple
+         module options from the configuration file. See option below.
 
-config CONFIG_MODPROBE_MULTIPLE_OPTIONS
+config CONFIG_FEATURE_MODPROBE_MULTIPLE_OPTIONS
        bool "Multiple options parsing"
        default y
        depends on CONFIG_MODPROBE
        help
          Allow modprobe to understand more than one option to pass to
          modules.
-         
+
          This is a WIP, while waiting for a common argument parsing
          common amongst all BB applets (shell, modprobe, etc...) and
          adds around 600 bytes on x86, 700 bytes on ARM. The code is
          biggish and uggly, but just works.
-         
+
          Saying Y here is not a bad idea if you're not that short
          on storage capacity.
 
-config CONFIG_RMMOD
-       bool "rmmod"
-       default n
-       help
-         rmmod is used to unload specified modules from the kernel.
+comment "Options common to multiple modutils"
+       depends on CONFIG_INSMOD || CONFIG_RMMOD || CONFIG_MODPROBE || CONFIG_LSMOD
 
 config CONFIG_FEATURE_CHECK_TAINTED_MODULE
-       bool "Support tainted module checking with new kernels"
+       # Simulate indentation
+       bool "  Support tainted module checking with new kernels"
        default y
        depends on CONFIG_INSMOD || CONFIG_LSMOD
        help
@@ -135,6 +120,28 @@ config CONFIG_FEATURE_CHECK_TAINTED_MODULE
          support request.
          This option is required to support GPLONLY modules.
 
+config CONFIG_FEATURE_2_4_MODULES
+       # Simulate indentation
+       bool "  Support version 2.2.x to 2.4.x Linux kernels"
+       default y
+       depends on CONFIG_INSMOD || CONFIG_RMMOD
+       help
+         Support module loading for 2.2.x and 2.4.x Linux kernels.
+
+config CONFIG_FEATURE_2_6_MODULES
+       # Simulate indentation
+       bool "  Support version 2.6.x Linux kernels"
+       default y
+       depends on CONFIG_INSMOD || CONFIG_RMMOD || CONFIG_MODPROBE
+       help
+         Support module loading for newer 2.6.x Linux kernels.
+
+
+config CONFIG_FEATURE_QUERY_MODULE_INTERFACE
+       bool
+       default y
+       depends on CONFIG_FEATURE_2_4_MODULES && !CONFIG_FEATURE_2_6_MODULES
+
 
 endmenu