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

[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/+/69159 )

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

Patch Set 4:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/69159/comment/b06cc68d_83dd5945 
PS1, Line 461: 	*tpm_family = 1;
> Judging by the differences in addresses they might have implemented their own protocol derived from  […]

if (TPM_INTF_CAPABILITY.InterfaceVersion == 0)
    tpm_family = 1     // (TIS <= 1.21, not used by TPM 2.0 devices)
else                   // (TIS 1.3, used both by TPM 1.2 and 2.0)
    if (TPM_STS.tpmFamily == 0)
        tpm_family = 1
    else
        tpm_family = 2

This procedure is wrong, I added an appropriate procedure in parent change as a comment.