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>
		
			
				
	
	
		
			24 lines
		
	
	
		
			944 B
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			24 lines
		
	
	
		
			944 B
		
	
	
	
		
			C
		
	
	
	
	
	
| #ifndef __KCONFIG_H__
 | |
| #define __KCONFIG_H__
 | |
| 
 | |
| #include <libpayload-config.h>
 | |
| 
 | |
| /*
 | |
|  * Getting something that works in C and CPP for an arg that may or may
 | |
|  * not be defined is tricky.  Here, if we have "#define CONFIG_LP_BOOGER 1"
 | |
|  * we match on the placeholder define, insert the "0," for arg1 and generate
 | |
|  * the triplet (0, 1, 0).  Then the last step cherry picks the 2nd arg (a one).
 | |
|  * When CONFIG_LP_BOOGER is not defined, we generate a (... 1, 0) pair, and when
 | |
|  * the last step cherry picks the 2nd arg, we get a zero.
 | |
|  */
 | |
| #define __ARG_PLACEHOLDER_1 0,
 | |
| #define config_enabled(cfg) _config_enabled(cfg)
 | |
| #define _config_enabled(value) __config_enabled(__ARG_PLACEHOLDER_##value)
 | |
| #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
 |