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