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

[Sergii Dmytruk (Code Review)]

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 27:

(1 comment)

Patchset:

PS27:
> With this patch we get problems on timeless build with mainboard siemens/mc_ehl1. […]

I tried building the board with `BUILD_TIMELESS=1` set and it succeeded but of course you probably customize config.  Also don't see timeless build affecting section sizes.

This patch doesn't add much code (some parameters and an error string), so you were probably running out of space there and many things could have caused the overlap (like toolchain changes) and it just happened to be this change.  I don't really see what could be thrown away to reduce size (would returning a struct instead of using output parameter help?).  I'd look into why timeless builds have tighter space requirements and whether that could be lifted.