[coreboot-gerrit] [M] Change in coreboot[master]: drivers/pc80/tpm: probe for TPM family of a device

[Michał Żygowski (Code Review)]

Attention is currently required from: Christian Walter, Julius Werner, Krystian Hebel, Sergii Dmytruk.

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69023 )

Change subject: drivers/pc80/tpm: probe for TPM family of a device
......................................................................

Patch Set 5:

(2 comments)

File src/drivers/pc80/tpm/tis.c:

https://review.coreboot.org/c/coreboot/+/69023/comment/b0753c69_ebd44e4f 
PS5, Line 432: 		family = TPM_2;
If intf_id == 1 that means CRB interface is active which is not compatible with pc80 TIS driver. We should also check for value 0 which will be TPM2 FIFO interface compatible with this driver, otherwise return TPM_DRIVER_ERR.

https://review.coreboot.org/c/coreboot/+/69023/comment/c3b94c20_1a3e9f34 
PS5, Line 458: 	family_str = (family == TPM_1 ? "TPM1" : "TPM2");
family_str = (family == TPM_1 ? "TPM 1.2" : "TPM 2.0");