build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38423 )
Change subject: cbfs: Add metadata cache ......................................................................
Patch Set 11:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38423/11/src/lib/cbfs.c File src/lib/cbfs.c:
https://review.coreboot.org/c/coreboot/+/38423/11/src/lib/cbfs.c@45 PS11, Line 45: if (!(cbd = cbfs_get_boot_device(true))) do not use assignment in if condition
https://review.coreboot.org/c/coreboot/+/38423/11/src/lib/cbfs.c@367 PS11, Line 367: if (cbmem_possibly_online() && do not use assignment in if condition
https://review.coreboot.org/c/coreboot/+/38423/11/src/security/vboot/vboot_l... File src/security/vboot/vboot_loader.c:
https://review.coreboot.org/c/coreboot/+/38423/11/src/security/vboot/vboot_l... PS11, Line 100: if (cbmem_possibly_online() && do not use assignment in if condition