Attention is currently required from: Subrata Banik, Nick Vaccaro. Hello Subrata Banik, Nick Vaccaro,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/62964
to look at the new patch set (#2).
Change subject: drivers/tpm/cr50: Use cr50_get_firmware_version in get_board_cfg ......................................................................
drivers/tpm/cr50: Use cr50_get_firmware_version in get_board_cfg
cr50_get_board_cfg() may be called in ramstage for some mainboards in order to determine the BOARD_CFG register's value. The code was written assuming that the firmware version was already retrieved, but for boards calling this in ramstage, this is not the case. Therefore, instead of using the cached cr50_firmware_version (which is all 0s in ramstage at that time), use the cr50_get_firmware_version function instead.
BUG=b:225206079, b:220685274 TEST=boot on brya0 and see: [INFO ] Firmware version: B2-C:0 RO_B:0.0.11/4d655eab RW_B:0.6.93/cr50_v3.94 [INFO ] Enabling GPIO PM b/c CR50 has long IRQ pulse support in the logs.
Signed-off-by: Tim Wawrzynczak twawrzynczak@chromium.org Change-Id: Ia6e5f4965a8852793d2f95e6eb21ea87860335a9 --- M src/drivers/tpm/cr50.c 1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/62964/2