build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25182 )
Change subject: smmstore: Add a key/val store facility in flash, mediated through SMM ......................................................................
Patch Set 10:
(5 comments)
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c File src/drivers/smmstore/store.c:
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c@185 PS10, Line 185: if (rdev_writeat(&store, &key_sz, end, 4) != 4) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c@189 PS10, Line 189: if (rdev_writeat(&store, &value_sz, end, 4) != 4) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c@193 PS10, Line 193: if (rdev_writeat(&store, key, end, key_sz) != key_sz) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c@197 PS10, Line 197: if (rdev_writeat(&store, value, end, value_sz) != value_sz) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25182/10/src/drivers/smmstore/store.c@202 PS10, Line 202: if (rdev_writeat(&store, &nul, end, 1) != 1) { braces {} are not necessary for single statement blocks