Attention is currently required from: Arthur Heymans. Hello Arthur Heymans,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/63057
to review the following change.
Change subject: device/dram/common.h: Use C over CPP ......................................................................
device/dram/common.h: Use C over CPP
This fixes building with clang.
Change-Id: Ia8511ab46184aa0d8ee3a79c3ef22614aeb61298 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/include/device/dram/common.h 1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/63057/1
diff --git a/src/include/device/dram/common.h b/src/include/device/dram/common.h index 9475853..452f693 100644 --- a/src/include/device/dram/common.h +++ b/src/include/device/dram/common.h @@ -40,11 +40,11 @@ * disabled. * @{ */ -#if CONFIG(DEBUG_RAM_SETUP) -#define printram(x, ...) printk(BIOS_DEBUG, x, ##__VA_ARGS__) -#else -#define printram(x, ...) -#endif +#define printram(x, ...) \ + do { \ + if (CONFIG(DEBUG_RAM_SETUP)) \ + printk(BIOS_DEBUG, x, ##__VA_ARGS__); \ + } while (0); /** @} */
/** Result of the SPD decoding process */