Nico Huber merged this change.

View Change

kconfig: Drop IS_ENABLED() macro

We keep its definition in libpayload, though, to maintain compatibility
with existing payload code. For now.

Change-Id: I8fc0d0136ba2316ef393c5c17f2b3ac3a9c6328d
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32230
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
---
M Documentation/getting_started/kconfig.md
M src/include/kconfig.h
2 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/Documentation/getting_started/kconfig.md b/Documentation/getting_started/kconfig.md
index 249fd46..852ca08 100644
--- a/Documentation/getting_started/kconfig.md
+++ b/Documentation/getting_started/kconfig.md
@@ -1165,8 +1165,6 @@
 - coreboot has added the glob operator '*' for the 'source' keyword.
 - coreboot’s Kconfig always defines variables except for strings. In other
   Kconfig implementations, bools set to false/0/no are not defined.
-- IS_ENABLED() is ‘false’ for undefined variables and ‘0’ variables. In Linux
-  (where the macro comes from) it’s ‘true’ as soon as the variable is defined.
 - coreboot’s version of Kconfig adds the KCONFIG_STRICT environment variable to
   error out if there are any issues in the Kconfig files.  In the Linux kernel,
   Kconfig will generate a warning, but will still output an updated .config or
diff --git a/src/include/kconfig.h b/src/include/kconfig.h
index 0478548..50ef302 100644
--- a/src/include/kconfig.h
+++ b/src/include/kconfig.h
@@ -17,7 +17,6 @@
 #define __config_enabled(arg1_or_junk) ___config_enabled(arg1_or_junk 1, 0, 0)
 #define ___config_enabled(__ignored, val, ...) val
 
-#define IS_ENABLED(option) config_enabled(option)	/* deprecated */
 #define CONFIG(option) config_enabled(CONFIG_##option)
 
 #endif

To view, visit change 32230. To unsubscribe, or for help writing mail filters, visit settings.