lkml.org 
[lkml]   [2018]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] c6x: remove redundant 'default n' from Kconfig-s
Date
'default n' is the default value for any bool or tristate Kconfig
setting so there is no need to write it explicitly.

Also since commit f467c5640c29 ("kconfig: only write '# CONFIG_FOO
is not set' for visible symbols") the Kconfig behavior is the same
regardless of 'default n' being present or not:

...
One side effect of (and the main motivation for) this change is making
the following two definitions behave exactly the same:

config FOO
bool

config FOO
bool
default n

With this change, neither of these will generate a
'# CONFIG_FOO is not set' line (assuming FOO isn't selected/implied).
That might make it clearer to people that a bare 'default n' is
redundant.
...

Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
---
arch/c6x/Kconfig | 2 --
arch/c6x/platforms/Kconfig | 5 -----
2 files changed, 7 deletions(-)

Index: b/arch/c6x/Kconfig
===================================================================
--- a/arch/c6x/Kconfig 2018-10-09 15:58:36.295122871 +0200
+++ b/arch/c6x/Kconfig 2018-10-16 15:28:50.623630380 +0200
@@ -70,14 +70,12 @@ config CMDLINE
config CMDLINE_FORCE
bool "Force default kernel command string"
depends on CMDLINE_BOOL
- default n
help
Set this to have arguments from the default kernel command string
override those passed by the boot loader.

config CPU_BIG_ENDIAN
bool "Build big-endian kernel"
- default n
help
Say Y if you plan on running a kernel in big-endian mode.
Note that your board must be properly built and your board
Index: b/arch/c6x/platforms/Kconfig
===================================================================
--- a/arch/c6x/platforms/Kconfig 2018-09-03 18:11:11.249792456 +0200
+++ b/arch/c6x/platforms/Kconfig 2018-10-16 15:29:04.891630739 +0200
@@ -2,20 +2,15 @@

config SOC_TMS320C6455
bool "TMS320C6455"
- default n

config SOC_TMS320C6457
bool "TMS320C6457"
- default n

config SOC_TMS320C6472
bool "TMS320C6472"
- default n

config SOC_TMS320C6474
bool "TMS320C6474"
- default n

config SOC_TMS320C6678
bool "TMS320C6678"
- default n
\
 
 \ /
  Last update: 2018-10-16 15:31    [W:0.025 / U:6.872 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site