x86: kconfig: Remove meaningless 'select n'
authorUlf Magnusson <ulfalizer@gmail.com>
Tue, 30 Jan 2018 12:59:03 +0000 (13:59 +0100)
committerBin Meng <bmeng.cn@gmail.com>
Tue, 30 Jan 2018 14:34:38 +0000 (22:34 +0800)
'select n' selects a constant symbol, which is meaningless and has no
effect. Maybe this was meant to be a 'default n', though bool and
tristate symbols already implicitly default to n.

Discovered in Kconfiglib (https://github.com/ulfalizer/Kconfiglib),
which does more strict checking here:

kconfiglib.KconfigSyntaxError: board/google/Kconfig:34: Couldn't parse ' select n': expected nonconstant symbol

Signed-off-by: Ulf Magnusson <ulfalizer@gmail.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
board/google/Kconfig

index e56c026..766db1b 100644 (file)
@@ -31,7 +31,6 @@ config TARGET_CHROMEBOOK_LINK64
 
 config TARGET_CHROMEBOX_PANTHER
        bool "Chromebox panther (not available)"
-       select n
        help
          Note: At present this must be used with coreboot. See README.x86
          for instructions.