[B.A.T.M.A.N.] [PATCH v2 1/2] batman-adv: Remove "default n" in Kconfig

[B.A.T.M.A.N.] [PATCH v2 1/2] batman-adv: Remove "default n" in Kconfig

[Sven Eckelmann]

The "default n" is the default value for any bool or tristate Kconfig
setting. It is therefore not necessary to add it to a config entry.

Reported-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
v2: changed "the an config entry" to "a config entry" in commit message

 net/batman-adv/Kconfig | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/net/batman-adv/Kconfig b/net/batman-adv/Kconfig
index de8034d80623..41bb67d70c83 100644
--- a/net/batman-adv/Kconfig
+++ b/net/batman-adv/Kconfig
@@ -24,7 +24,6 @@ config BATMAN_ADV
 	depends on NET
 	select CRC16
 	select LIBCRC32C
-        default n
 	help
           B.A.T.M.A.N. (better approach to mobile ad-hoc networking) is
           a routing protocol for multi-hop ad-hoc mesh networks. The
@@ -60,7 +59,6 @@ config BATMAN_ADV_BLA
 config BATMAN_ADV_DAT
 	bool "Distributed ARP Table"
 	depends on BATMAN_ADV && INET
-	default n
 	help
 	  This option enables DAT (Distributed ARP Table), a DHT based
 	  mechanism that increases ARP reliability on sparse wireless
@@ -70,7 +68,6 @@ config BATMAN_ADV_DAT
 config BATMAN_ADV_NC
 	bool "Network Coding"
 	depends on BATMAN_ADV
-	default n
 	help
 	  This option enables network coding, a mechanism that aims to
 	  increase the overall network throughput by fusing multiple
@@ -84,7 +81,6 @@ config BATMAN_ADV_NC
 config BATMAN_ADV_MCAST
 	bool "Multicast optimisation"
 	depends on BATMAN_ADV && INET && !(BRIDGE=m && BATMAN_ADV=y)
-	default n
 	help
 	  This option enables the multicast optimisation which aims to
 	  reduce the air overhead while improving the reliability of
@@ -94,7 +90,6 @@ config BATMAN_ADV_DEBUGFS
 	bool "batman-adv debugfs entries"
 	depends on BATMAN_ADV
 	depends on DEBUG_FS
-	default n
 	help
 	  Enable this to export routing related debug tables via debugfs.
 	  The information for each soft-interface and used hard-interface can be
-- 
2.17.0

[B.A.T.M.A.N.] [PATCH v2 2/2] batman-adv: Drop "experimental" from BATMAN_V Kconfig

[Sven Eckelmann]

The Kconfig option BATMAN_ADV_BATMAN_V is now enabled by default when the
BATMAN_ADV is enabled. A feature which is enabled by default for a module
should not be considered experimental.

Reported-by: Joe Perches <joe@perches.com>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
v2: no change

 net/batman-adv/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/batman-adv/Kconfig b/net/batman-adv/Kconfig
index 41bb67d70c83..da0b7aa98be9 100644
--- a/net/batman-adv/Kconfig
+++ b/net/batman-adv/Kconfig
@@ -32,7 +32,7 @@ config BATMAN_ADV
           tools.
 
 config BATMAN_ADV_BATMAN_V
-	bool "B.A.T.M.A.N. V protocol (experimental)"
+	bool "B.A.T.M.A.N. V protocol"
 	depends on BATMAN_ADV && !(CFG80211=m && BATMAN_ADV=y)
 	default y
 	help
-- 
2.17.0

Re: [B.A.T.M.A.N.] [PATCH v2 1/2] batman-adv: Remove "default n" in Kconfig

[Sven Eckelmann]

[-- Attachment #1: Type: text/plain, Size: 549 bytes --]

On Samstag, 26. Mai 2018 11:40:31 CEST Sven Eckelmann wrote:
> The "default n" is the default value for any bool or tristate Kconfig
> setting. It is therefore not necessary to add it to a config entry.
> 
> Reported-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
> ---
> v2: changed "the an config entry" to "a config entry" in commit message

Added as ba1a986d4af5 [1]

Kind regards,
	Sven

[1] https://git.open-mesh.org/batman-adv.git/commit/ba1a986d4af5ed4182966dcbd0fd04189b5e13cb

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [B.A.T.M.A.N.] [PATCH v2 2/2] batman-adv: Drop "experimental" from BATMAN_V Kconfig

[Sven Eckelmann]

[-- Attachment #1: Type: text/plain, Size: 519 bytes --]

On Samstag, 26. Mai 2018 11:40:32 CEST Sven Eckelmann wrote:
> The Kconfig option BATMAN_ADV_BATMAN_V is now enabled by default when the
> BATMAN_ADV is enabled. A feature which is enabled by default for a module
> should not be considered experimental.
> 
> Reported-by: Joe Perches <joe@perches.com>
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
> ---
> v2: no change

Added as c812861ac3d3 [1].

Kind regards,
	Sven

[1] https://git.open-mesh.org/batman-adv.git/commit/c812861ac3d391243e8059955c99b9b20ca7a82e

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]