summaryrefslogtreecommitdiff
path: root/util/lint/kconfig_lint_README
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2016-01-31 10:34:13 -0700
committerMartin Roth <martinroth@google.com>2016-02-09 21:51:46 +0100
commit662237614b125c4039288a06df7af2a32317a554 (patch)
tree4474f01ab98dca05ace3be2d98698c1a6253b58a /util/lint/kconfig_lint_README
parent609bd9445ed1cc76496a9d65ad1d158904d3cf47 (diff)
downloadcoreboot-662237614b125c4039288a06df7af2a32317a554.tar.xz
kconfig_lint: Check for IS_ENABLED used on symbols without CONFIG_
This looks at the coreboot codebase for the IS_ENABLED macro, and gives an error if there is a symbol used without the CONFIG_ prefix. This only works for symbols of type bool. A future check will be added for all symbols, but that will take a significant amount of time to run, because each symbol will need to be searched for individually. Change-Id: I92f2de2d231610d1a788da965f21966d89c2f25c Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/13538 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'util/lint/kconfig_lint_README')
-rw-r--r--util/lint/kconfig_lint_README1
1 files changed, 1 insertions, 0 deletions
diff --git a/util/lint/kconfig_lint_README b/util/lint/kconfig_lint_README
index 9076f4ca41..3c638f5e10 100644
--- a/util/lint/kconfig_lint_README
+++ b/util/lint/kconfig_lint_README
@@ -93,5 +93,6 @@ Errors in coreboot source files:
defined in coreboot's version of Kconfig.
- The IS_ENABLED macro is only valid for bool symbols.
- The IS_ENABLED used on unknown CONFIG_ value, like an obsolete symbol.
+- The IS_ENABLED macro is used on a symbol without the CONFIG_ prefix.
TODO: check for choice entries at the top level