[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linuxbios
Subject:    [coreboot] [coreboot - Feature #433] Unify TPM drivers in coreboot
From:       Maciej Pijanowski <coreboot () fe80 ! eu>
Date:       2022-10-26 21:27:28
Message-ID: redmine.journal-1226.20221026212727.700 () fe80 ! eu
[Download RAW message or body]

Issue #433 has been updated by Maciej Pijanowski.





Sergii Dmytruk wrote in #note-3:

> > Add an option to compile all drivers for TPM 1.2, 2.0 TIS and CRB.

> 

> An option like "compile all of them" or just allow selecting any subset of the \
> drivers?



I'd say select which drivers to include. Currently, one can include only one IIUC.

There would be no point in including driver if we know that given platform

will never come with given set of TPMs.



For example, we have a laptop which can use either discrete TPM 2.0 or fTPM.

We could give user selection which TPM should be used. Or, for example,

use fTPM if discrete TPM 2.0 module is not present. Etc. 



----------------------------------------

Feature #433: Unify TPM drivers in coreboot

https://ticket.coreboot.org/issues/433#change-1226



* 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 -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-leave@coreboot.org


[prev in list] [next in list] [prev in thread] [next in thread]