Joel Kitching has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37261 )
Change subject: security/vboot: Use persistent context to read GBB flags ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Awesome! It didn't even occur to me that we could get rid of this with persistent context.
LGTM with nit about bool.
https://review.coreboot.org/c/coreboot/+/37261/1/src/security/vboot/misc.h File src/security/vboot/misc.h:
https://review.coreboot.org/c/coreboot/+/37261/1/src/security/vboot/misc.h@5... PS1, Line 58: bool Are we doing bools now? I have only ever seen int used for booleans up until this point.