diff options
| author | Julius Werner <jwerner@chromium.org> | 2019-03-05 16:47:25 -0800 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-03-07 17:06:28 +0000 |
| commit | 496ef1a9e9a746005d85fbb8363355894d7a16ad (patch) | |
| tree | 6596a00b07eb064915cf5215302d9419a315c9fb /payloads/libpayload | |
| parent | 55f0a1409ddc2889e092d0bbbe1f415f68353e91 (diff) | |
| download | coreboot-496ef1a9e9a746005d85fbb8363355894d7a16ad.tar.gz coreboot-496ef1a9e9a746005d85fbb8363355894d7a16ad.tar.bz2 coreboot-496ef1a9e9a746005d85fbb8363355894d7a16ad.zip | |
Add new CONFIG(XXX) macro to replace IS_ENABLED(CONFIG_XXX)
The IS_ENABLED() macro is pretty long and unwieldy for something so
widely used, and often forces line breaks just for checking two Kconfigs
in a row. Let's replace it with something that takes up less space to
make our code more readable. From now on,
if (IS_ENABLED(CONFIG_XXX))
#if IS_ENABLED(CONFIG_XXX)
shall become
if (CONFIG(XXX))
#if CONFIG(XXX)
Change-Id: I2468427b569b974303084574125a9e1d9f6db596
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31773
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'payloads/libpayload')
| -rw-r--r-- | payloads/libpayload/include/kconfig.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/payloads/libpayload/include/kconfig.h b/payloads/libpayload/include/kconfig.h index adb3403cf2c1..9cce6ea7068e 100644 --- a/payloads/libpayload/include/kconfig.h +++ b/payloads/libpayload/include/kconfig.h @@ -17,5 +17,7 @@ #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) +#define IS_ENABLED(option) config_enabled(option) /* deprecated */ +#define CONFIG(option) config_enabled(CONFIG_##option) + #endif |