[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    [PATCH] modutils: remove redundant "select PLATFORM_LINUX" configs
From:       Kang-Che Sung <explorer09 () gmail ! com>
Date:       2017-01-31 17:17:50
Message-ID: CADDzAfO8BaOznF-=h=Li7cPJRCrs+4Xu+Df_vNxwbmAK8EimSw () mail ! gmail ! com
[Download RAW message or body]

It is enough to have only applets' configs select PLATFORM_LINUX.

Signed-off-by: Kang-Che Sung <explorer09@gmail.com>
---
 modutils/Config.src | 11 -----------
 modutils/lsmod.c    |  1 -
 modutils/modprobe.c |  1 -
 3 files changed, 13 deletions(-)

diff --git a/modutils/Config.src b/modutils/Config.src
index 1aee57ee7..36a0a7225 100644
--- a/modutils/Config.src
+++ b/modutils/Config.src
@@ -8,7 +8,6 @@ menu "Linux Module Utilities"
 config MODPROBE_SMALL
  bool "Simplified modutils"
  default y
- select PLATFORM_LINUX
  help
   Simplified modutils.

@@ -44,7 +43,6 @@ config FEATURE_2_4_MODULES
  bool "Support version 2.2/2.4 Linux kernels"
  default n
  depends on (INSMOD || RMMOD || LSMOD) && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   Support module loading for 2.2.x and 2.4.x Linux kernels.
   This increases size considerably. Say N unless you plan
@@ -54,7 +52,6 @@ config FEATURE_INSMOD_TRY_MMAP
  bool "Try to load module from a mmap'ed area"
  default n
  depends on INSMOD && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   This option causes module loading code to try to mmap
   module first. If it does not work (for example,
@@ -71,7 +68,6 @@ config FEATURE_INSMOD_VERSION_CHECKING
  bool "Enable module version checking"
  default n
  depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
- select PLATFORM_LINUX
  help
   Support checking of versions for modules. This is used to
   ensure that the kernel and module are made for each other.
@@ -80,7 +76,6 @@ config FEATURE_INSMOD_KSYMOOPS_SYMBOLS
  bool "Add module symbols to kernel symbol table"
  default n
  depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
- select PLATFORM_LINUX
  help
   By adding module symbols to the kernel symbol table, Oops messages
   occuring within kernel modules can be properly debugged. By enabling
@@ -92,7 +87,6 @@ config FEATURE_INSMOD_LOADINKMEM
  bool "In kernel memory optimization (uClinux only)"
  default n
  depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
- select PLATFORM_LINUX
  help
   This is a special uClinux only memory optimization that lets insmod
   load the specified kernel module directly into kernel space, reducing
@@ -103,7 +97,6 @@ config FEATURE_INSMOD_LOAD_MAP
  bool "Enable insmod load map (-m) option"
  default n
  depends on FEATURE_2_4_MODULES && INSMOD
- select PLATFORM_LINUX
  help
   Enabling this, one would be able to get a load map
   output on stdout. This makes kernel module debugging
@@ -115,7 +108,6 @@ config FEATURE_INSMOD_LOAD_MAP_FULL
  bool "Symbols in load map"
  default y
  depends on FEATURE_INSMOD_LOAD_MAP && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   Without this option, -m will only output section
   load map. With this option, -m will also output
@@ -125,7 +117,6 @@ config FEATURE_CHECK_TAINTED_MODULE
  bool "Support tainted module checking with new kernels"
  default y
  depends on (LSMOD || FEATURE_2_4_MODULES) && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   Support checking for tainted modules. These are usually binary
   only modules that will make the linux-kernel list ignore your
@@ -136,7 +127,6 @@ config FEATURE_MODUTILS_ALIAS
  bool "Support module.aliases file"
  default y
  depends on (DEPMOD || MODPROBE) && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   Generate and parse modules.alias containing aliases for bus
   identifiers:
@@ -153,7 +143,6 @@ config FEATURE_MODUTILS_SYMBOLS
  bool "Support module.symbols file"
  default y
  depends on (DEPMOD || MODPROBE) && !MODPROBE_SMALL
- select PLATFORM_LINUX
  help
   Generate and parse modules.symbols containing aliases for
   symbol_request() kernel calls, such as:
diff --git a/modutils/lsmod.c b/modutils/lsmod.c
index 9ab49f35b..24e5d35b9 100644
--- a/modutils/lsmod.c
+++ b/modutils/lsmod.c
@@ -18,7 +18,6 @@
 //config: bool "Pretty output"
 //config: default y
 //config: depends on LSMOD && !MODPROBE_SMALL
-//config: select PLATFORM_LINUX
 //config: help
 //config:  This option makes output format of lsmod adjusted to
 //config:  the format of module-init-tools for Linux kernel 2.6.
diff --git a/modutils/modprobe.c b/modutils/modprobe.c
index 09e3de6c3..cbec43888 100644
--- a/modutils/modprobe.c
+++ b/modutils/modprobe.c
@@ -19,7 +19,6 @@
 //config: bool "Blacklist support"
 //config: default y
 //config: depends on MODPROBE && !MODPROBE_SMALL
-//config: select PLATFORM_LINUX
 //config: help
 //config:  Say 'y' here to enable support for the 'blacklist' command in
 //config:  modprobe.conf. This prevents the alias resolver to resolve
-- 
2.11.0

["busybox-modutils-config-select.patch" (text/x-patch)]

From 10a54e192e9c1886b8d5a1dbb5eb46fe3f3a6a5e Mon Sep 17 00:00:00 2001
From: Kang-Che Sung <explorer09@gmail.com>
Date: Tue, 31 Jan 2017 14:27:01 +0800
Subject: [PATCH] modutils: remove redundant "select PLATFORM_LINUX" configs

It is enough to have only applets' configs select PLATFORM_LINUX.

Signed-off-by: Kang-Che Sung <explorer09@gmail.com>
---
 modutils/Config.src | 11 -----------
 modutils/lsmod.c    |  1 -
 modutils/modprobe.c |  1 -
 3 files changed, 13 deletions(-)

diff --git a/modutils/Config.src b/modutils/Config.src
index 1aee57ee7..36a0a7225 100644
--- a/modutils/Config.src
+++ b/modutils/Config.src
@@ -8,7 +8,6 @@ menu "Linux Module Utilities"
 config MODPROBE_SMALL
 	bool "Simplified modutils"
 	default y
-	select PLATFORM_LINUX
 	help
 	  Simplified modutils.
 
@@ -44,7 +43,6 @@ config FEATURE_2_4_MODULES
 	bool "Support version 2.2/2.4 Linux kernels"
 	default n
 	depends on (INSMOD || RMMOD || LSMOD) && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  Support module loading for 2.2.x and 2.4.x Linux kernels.
 	  This increases size considerably. Say N unless you plan
@@ -54,7 +52,6 @@ config FEATURE_INSMOD_TRY_MMAP
 	bool "Try to load module from a mmap'ed area"
 	default n
 	depends on INSMOD && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  This option causes module loading code to try to mmap
 	  module first. If it does not work (for example,
@@ -71,7 +68,6 @@ config FEATURE_INSMOD_VERSION_CHECKING
 	bool "Enable module version checking"
 	default n
 	depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
-	select PLATFORM_LINUX
 	help
 	  Support checking of versions for modules. This is used to
 	  ensure that the kernel and module are made for each other.
@@ -80,7 +76,6 @@ config FEATURE_INSMOD_KSYMOOPS_SYMBOLS
 	bool "Add module symbols to kernel symbol table"
 	default n
 	depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
-	select PLATFORM_LINUX
 	help
 	  By adding module symbols to the kernel symbol table, Oops messages
 	  occuring within kernel modules can be properly debugged. By enabling
@@ -92,7 +87,6 @@ config FEATURE_INSMOD_LOADINKMEM
 	bool "In kernel memory optimization (uClinux only)"
 	default n
 	depends on FEATURE_2_4_MODULES && (INSMOD || MODPROBE)
-	select PLATFORM_LINUX
 	help
 	  This is a special uClinux only memory optimization that lets insmod
 	  load the specified kernel module directly into kernel space, reducing
@@ -103,7 +97,6 @@ config FEATURE_INSMOD_LOAD_MAP
 	bool "Enable insmod load map (-m) option"
 	default n
 	depends on FEATURE_2_4_MODULES && INSMOD
-	select PLATFORM_LINUX
 	help
 	  Enabling this, one would be able to get a load map
 	  output on stdout. This makes kernel module debugging
@@ -115,7 +108,6 @@ config FEATURE_INSMOD_LOAD_MAP_FULL
 	bool "Symbols in load map"
 	default y
 	depends on FEATURE_INSMOD_LOAD_MAP && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  Without this option, -m will only output section
 	  load map. With this option, -m will also output
@@ -125,7 +117,6 @@ config FEATURE_CHECK_TAINTED_MODULE
 	bool "Support tainted module checking with new kernels"
 	default y
 	depends on (LSMOD || FEATURE_2_4_MODULES) && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  Support checking for tainted modules. These are usually binary
 	  only modules that will make the linux-kernel list ignore your
@@ -136,7 +127,6 @@ config FEATURE_MODUTILS_ALIAS
 	bool "Support module.aliases file"
 	default y
 	depends on (DEPMOD || MODPROBE) && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  Generate and parse modules.alias containing aliases for bus
 	  identifiers:
@@ -153,7 +143,6 @@ config FEATURE_MODUTILS_SYMBOLS
 	bool "Support module.symbols file"
 	default y
 	depends on (DEPMOD || MODPROBE) && !MODPROBE_SMALL
-	select PLATFORM_LINUX
 	help
 	  Generate and parse modules.symbols containing aliases for
 	  symbol_request() kernel calls, such as:
diff --git a/modutils/lsmod.c b/modutils/lsmod.c
index 9ab49f35b..24e5d35b9 100644
--- a/modutils/lsmod.c
+++ b/modutils/lsmod.c
@@ -18,7 +18,6 @@
 //config:	bool "Pretty output"
 //config:	default y
 //config:	depends on LSMOD && !MODPROBE_SMALL
-//config:	select PLATFORM_LINUX
 //config:	help
 //config:	  This option makes output format of lsmod adjusted to
 //config:	  the format of module-init-tools for Linux kernel 2.6.
diff --git a/modutils/modprobe.c b/modutils/modprobe.c
index 09e3de6c3..cbec43888 100644
--- a/modutils/modprobe.c
+++ b/modutils/modprobe.c
@@ -19,7 +19,6 @@
 //config:	bool "Blacklist support"
 //config:	default y
 //config:	depends on MODPROBE && !MODPROBE_SMALL
-//config:	select PLATFORM_LINUX
 //config:	help
 //config:	  Say 'y' here to enable support for the 'blacklist' command in
 //config:	  modprobe.conf. This prevents the alias resolver to resolve
-- 
2.11.0



_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic