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