Attention is currently required from: Raul Rangel. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60253 )
Change subject: [RFC] smp/spinlock: Use compiler builtins ......................................................................
Patch Set 2:
(2 comments)
File src/include/smp/spinlock.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-135973): https://review.coreboot.org/c/coreboot/+/60253/comment/7a3aff86_1587c43f PS2, Line 11: #define DECLARE_SPIN_LOCK(x) static struct spinlock x; macros should not use a trailing semicolon
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-135973): https://review.coreboot.org/c/coreboot/+/60253/comment/05fbc824_7b976802 PS2, Line 27: while (!spinlock_try_lock(lock)) { braces {} are not necessary for single statement blocks
-
build bot (Jenkins) (Code Review)