[coreboot-gerrit] [XL] Change in coreboot[main]: treewide: convert to tpm_result_t

[Jon Murphy (Code Review)]

Attention is currently required from: Andrey Pronin, Christian Walter, Eric Lai, Erik van den Bogaert, Felix Held, Frans Hendriks, Fred Reitberger, Jason Glenesk, Julius Werner, Karthik Ramasubramanian, Martin L Roth, Matt DeVillier, Nick Vaccaro, Subrata Banik, Tim Van Patten, Yu-Ping Wu.

Jon Murphy has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77666?usp=email )

Change subject: treewide: convert to tpm_result_t
......................................................................

Patch Set 28:

(5 comments)

File src/drivers/i2c/tpm/cr50.c:

https://review.coreboot.org/c/coreboot/+/77666/comment/695abc61_88db69c2 :
PS27, Line 277: 	if(rc)
> `space required before the open parenthesis '('`

Please fix.

File src/security/vboot/vboot_logic.c:

https://review.coreboot.org/c/coreboot/+/77666/comment/3e461e94_dcbcb6c2 :
PS27, Line 226: error(0x%x)
> nit: `error (%#x)`

Done

https://review.coreboot.org/c/coreboot/+/77666/comment/69e7f76f_10698d38 :
PS27, Line 374: 					tpm_rc);
> No matter what style you choose, the indentation level should be the same as `VB2_RECOVERY_RO_TPM_U_ […]

Done

https://review.coreboot.org/c/coreboot/+/77666/comment/b145e2ac_ef28b67a :
PS27, Line 384: %x
> Let's take the chance to change it to `%#x`?

Done

https://review.coreboot.org/c/coreboot/+/77666/comment/51dbca50_3a52b552 :
PS27, Line 394: %x
> Let's take the chance to change it to `%#x`?

Done