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.






-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
https://ticket.coreboot.org/my/account
_______________________________________________
coreboot mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to