Vadim Bendebury has posted comments on this change. ( https://review.coreboot.org/22554 )
Change subject: drivers/i2c/tpm/cr50: Simplify and increase init delay to 30 seconds ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/22554/1/src/drivers/i2c/tpm/cr50.c File src/drivers/i2c/tpm/cr50.c:
https://review.coreboot.org/#/c/22554/1/src/drivers/i2c/tpm/cr50.c@450 PS1, Line 450: : if (ENV_VERSTAGE || ENV_BOOTBLOCK) : if (request_locality(chip)) : return -1;
As a sanity, do we want to check if locality is still claimed in other stag
actually, this is a good point, not as just a sanity issue: the old RO would release locality, so the new RW needs to claim it if there is RO/RW version mismatch.