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

[Julius Werner (Code Review)]

Attention is currently required from: Michał Żygowski, Christian Walter, Krystian Hebel, Sergii Dmytruk.

Julius Werner 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 1:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/69159/comment/c3382c95_2ba23633 
PS1, Line 461: 	*tpm_family = 1;
Are you sure this is guaranteed? I thought I recall that 9645 can be 2.0 as well but I might be mixing stuff up...

Is there no way to detect the version dynamically here as well?