Attention is currently required from: Philipp Deppenwiese, Michał Żygowski, Christian Walter, Julius Werner, Aaron Durbin, Piotr Król. Hello Philipp Deppenwiese, build bot (Jenkins), Michał Żygowski, Christian Walter, Aaron Durbin, Piotr Król,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/54641
to look at the new patch set (#3).
Change subject: tpm: Remove USER_TPMx options, make TPM1/TPM2 menuconfig visible ......................................................................
tpm: Remove USER_TPMx options, make TPM1/TPM2 menuconfig visible
We would like to have an easy way to completely disable TPM support on a board. For boards that don't pre-select a TPM protocol via the MAINBOARD_HAS_TPMx options, this is already possible with the USER_NO_TPM option. In order to make this available for all boards, this patch just removes the whole USER_TPMx option group and directly makes the TPM1 and TPM2 options visible to menuconfig. The MAINBOARD_HAS_TPMx options can still be used to select defaults and to prevent selection of a protocol that the TPM is known to not support, but the NO_TPM option always remains available.
Also fix some mainboards that selected TPM2 directly, which they're not supposed to do (that's what MAINBOARD_HAS_TPM2 is for), and add a missing dependency to TPM_CR50 so it is set correctly for a NO_TPM scenario.
Signed-off-by: Julius Werner jwerner@chromium.org Change-Id: Ib0a73da3c42fa4e8deffecb53f29ee38cbb51a93 --- M configs/config.asrock_b85m_pro4.tpm2_txt_placeholder_acms M configs/config.libretrend_lt1000 M src/mainboard/google/drallion/Kconfig M src/mainboard/protectli/vault_kbl/Kconfig M src/security/tpm/Kconfig M src/security/tpm/tss/vendor/cr50/Kconfig 6 files changed, 33 insertions(+), 43 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/54641/3