[coreboot-gerrit] [M] Change in coreboot[master]: security/tpm: replace CONFIG(TPMx) checks with runtime check

[Michał Żygowski (Code Review)]

Attention is currently required from: Lance Zhao, Christian Walter, Tim Wawrzynczak, Julius Werner, Krystian Hebel, Yu-Ping Wu, Sergii Dmytruk.

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

Change subject: security/tpm: replace CONFIG(TPMx) checks with runtime check
......................................................................

Patch Set 10:

(1 comment)

File src/drivers/crb/tis.c:

https://review.coreboot.org/c/coreboot/+/69161/comment/2165ad1e_b40fec9e 
PS6, Line 144: 	if (tlcl_get_family() == TPM_1)
             : 		return get_smbios_data(dev, handle, current);
> It was done this way, because the function is invoked like: […]

CRB is always TPM 2.0 AFAIK, thus TPM_1 is not a valid family to run this function. Thus we should return 0 and leave handle and current unmodfied.