Attention is currently required from: Michał Żygowski, Patrick Rudolph. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59519 )
Change subject: security/intel/txt: Fix HEAP_ACM format depending on number of ACMs in CBFS ......................................................................
Patch Set 8: Code-Review+1
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59519/comment/133072df_e878f5a3 PS8, Line 18: SINTI typo: SINIT
Patchset:
PS8: We could use a flexible array member in `struct txt_heap_acm_element`, but this prevents using a struct for the BDR contents. We would need to use a "raw" data buffer instead of the current "struct of structs".
File src/security/intel/txt/ramstage.c:
https://review.coreboot.org/c/coreboot/+/59519/comment/d30988e3_dcf0c794 PS8, Line 320: nit: drop blank line