From 662237614b125c4039288a06df7af2a32317a554 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sun, 31 Jan 2016 10:34:13 -0700 Subject: 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 Reviewed-on: https://review.coreboot.org/13538 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- util/lint/kconfig_lint_README | 1 + 1 file changed, 1 insertion(+) (limited to 'util/lint/kconfig_lint_README') 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 -- cgit v1.2.3