Issue #433 has been updated by Michał Żygowski.
Sergii Dmytruk wrote in #note-5:
https://review.coreboot.org/c/coreboot/+/69162 was merged today, so I think this can be closed.
I have added 3 more patches that ought to be merged before we can consider it closed: https://review.coreboot.org/c/coreboot/+/80453 https://review.coreboot.org/c/coreboot/+/80454 https://review.coreboot.org/c/coreboot/+/80455
Without them, one cannot really use either fTPM or dTPM on Intel platform.
---------------------------------------- Feature #433: Unify TPM drivers in coreboot https://ticket.coreboot.org/issues/433#change-1808
* Author: Michał Żygowski * Status: New * Priority: Normal * Target version: none * Start date: 2022-10-24 ---------------------------------------- Add an option to compile all drivers for TPM 1.2, 2.0 TIS and CRB. The motivation is to not build multiple coreboot ROMs for each possible TPM supported by the platform.
The tasks would include: - runtime TPM detection (probing TPM_INTF_CAPABILITY and TPM_INTERFACE_ID) - rename the TPM driver functions, make them static and expose them as a driver structure, e.g.
struct tpm_driver { void (*init)(void); int (*open)(void); int (*close)(void); int (*sendrecv)(const uint8_t *sendbuf, size_t send_size, uint8_t *recvbuf, size_t *recv_len); }
- based on the detected TPM, hook the tpm_driver functions to provide the global TPM API: tis_open, tis_close, tis_init, tis_sendrecv. Some additional API to get vendor/device name could also be considered.