Attention is currently required from: Marshall Dawson, Paul Menzel, Jan Dabros, Felix Held. Hello build bot (Jenkins), Raul Rangel, Marshall Dawson, Karthik Ramasubramanian, Felix Held,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/61965
to look at the new patch set (#5).
Change subject: mb/google/guybrush: Enable SOC_AMD_COMMON_BLOCK_I2C_TPM_SHARED_WITH_PSP ......................................................................
mb/google/guybrush: Enable SOC_AMD_COMMON_BLOCK_I2C_TPM_SHARED_WITH_PSP
Guybrush platforms have I2C3 controller which is shared between PSP and X86. In order to enable cooperation, PSP acts as an arbitrator. Enable SOC_AMD_COMMON_BLOCK_I2C_TPM_SHARED_WITH_PSP, so that proper driver is binded on the OS side.
With this change in place it is important to use correct kernel version which has I2C-amdpsp driver [1] enabled. Otherwise, we won't have I2C3 available and thus TPM device available in OS, what may end up as a serious error - guybrush refuses to boot without access to TPM.
BUG=b:204508404 BRANCH=guybrush TEST=Build proper kernel and firmware. Run on guybrush and verify TPM functionality.
[1]: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?...
Signed-off-by: Jan Dabros jsd@semihalf.com Change-Id: I9dd94e47e1a02e790427b67adff84de3eb3ee387 --- M src/mainboard/google/guybrush/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/61965/5