Bug #429
closedI2C CR50 TPM fails to initialize
0%
Description
On several (but not all) Chromebook platforms which use an I2C interface for the CR50 TPM, the TPM fails to initialize due to I2C transaction errors.
The following boards with I2C CR50 TPM are known to be affected:
google/brya (banshee variant confirmed, others untested)
google/drallion
google/poppy (soraka and nautilus variants)
google/reef (all variants)
The following boards with I2C CR50 TPM are known to be working:
google/eve
google/guybrush
google/kahlee
google/zork
cbmem shows the following, with the i2c transactions repeating 100x until failing. This causes a significant increase in boot time.
[INFO ] Probing TPM I2C: i2c 2:50 W 1 bytes : 06
[ERROR] I2C TX abort detected (00000001)
[ERROR] cr50_i2c_read: Address write failed
[INFO ] .i2c 2:50 W 1 bytes : 06
[ERROR] I2C TX abort detected (00000001)
[ERROR] cr50_i2c_read: Address write failed
...
soraka/nautilus show slightly different output:
[INFO ] Probing TPM I2C: Cr50 TPM IRQ timeout!
[INFO ] .Cr50 TPM IRQ timeout!
[INFO ] .Cr50 TPM IRQ timeout!
[INFO ] .Cr50 TPM IRQ timeout!
...
Updated by Matt DeVillier about 2 years ago
did some more testing, and confirmed that this issue is not present when vboot is used (RO only is fine) - so the issue potentially is the lack of early init by vboot leading to failure to init during ramstage. Will take a look and see what's different about the two
edit: issue is that TPM init in ramstage is occurring before the i2c bus has been initialized. Will push a fix
Updated by Matt DeVillier about 2 years ago
proposed fix: https://review.coreboot.org/c/coreboot/+/68550