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

[Felix Held (Code Review)]

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

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

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

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

Patch Set 22:

(1 comment)

File util/cbmem/cbmem.c:

https://review.coreboot.org/c/coreboot/+/68749/comment/bf8e065a_d7688452 :
PS22, Line 961: 			die("Unknown hash algorithm\n");
is it necessary to call die() here or would printing an error (and telling the caller that something went wrong, so that it can exit early) be sufficient here? there has been some discussion a few months ago in the leadership meeting about only using die() when it can't be avoided