[coreboot-gerrit] [L] Change in coreboot[master]: util/cbmem: add parsing of TPM logs per specs

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Arthur Heymans, Benjamin Doron, Christian Walter, Himanshu Sahdev, Julius Werner, Krystian Hebel, Maximilian Brune, Michał Kopeć, Michał Żygowski, Paul Menzel.

Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68749?usp=email )

Change subject: util/cbmem: add parsing of TPM logs per specs
......................................................................

Patch Set 19:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/68749/comment/c3b5c816_a27c3ced :
PS19, Line 9: coreboot is made to export the range allocated for the log.
> Sorry, I do not understand this sentence. […]

Thanks, the commit message and code have diverged.  Fixed.

https://review.coreboot.org/c/coreboot/+/68749/comment/a4582e26_3934183b :
PS19, Line 12:
> Could you please give some more background on the implementation. […]

Added more details and after juxtaposing outputs dropped debug print and fixed a discrepancy with formatting, thanks.

File util/cbmem/cbmem.c:

https://review.coreboot.org/c/coreboot/+/68749/comment/f9882403_8b589df3 :
PS19, Line 899: 			printf("\tEvent type: Unknown (0x%x)\n", event_type);
> `ARRAY_SIZE(tpm_event_types)` could also be printed.

Done