[coreboot-gerrit] [M] Change in coreboot[main]: security/tpm: make tis_probe() return tpm_family

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Christian Walter, Martin L Roth, Michał Żygowski.

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

Change subject: security/tpm: make tis_probe() return tpm_family
......................................................................

Patch Set 26:

(1 comment)

File src/security/tpm/tss/tcg-1.2/tss.c:

https://review.coreboot.org/c/coreboot/+/69159/comment/b94d5e06_b354462f :
PS26, Line 166: 		tis_sendrecv = NULL;
> Why are we not reporting an error here?

For consistency, because nothing is printed above for `TPM_CB_NO_DEVICE` error.  `tcg-2.0/tss.c` does print in that case, so I added a print for the new error as well.  The follow-up CB:69160 replaces both of these functions with a single one which reports errors, hence not trying to generally improve this implementation.