build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55189 )
Change subject: cpu/x86/lapic: Redo DEBUG_HALT_SELF ......................................................................
Patch Set 1:
(1 comment)
File src/cpu/x86/lapic/lapic_cpu_init.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-120695): https://review.coreboot.org/c/coreboot/+/55189/comment/9870e793_1f7ae471 PS1, Line 321: #define dprintk(LEVEL, args...) do { printk(LEVEL, ##args); } while (0) Single statement macros should not use a do {} while (0) loop