Attention is currently required from: Christian Walter, Subrata Banik.
Tim Van Patten has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79736?usp=email )
Change subject: security/tpm: Retrieve factory configuration for TI50 devices ......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/79736/comment/0c40b208_d4acc5aa : PS1, Line 9: This patch enables retrieval of factory configuration data from : TI50 TPM devices. I think it's worth checking with the GSC team to verify the validity of that comment.
I trust go/cr50-board-id-in-factory more though, since it's explicit about version numbers:
To get/set the factory config, it requires Cr50 firmware version >= 0.{5,6}.170, or Ti50 firmware version >= 0.{23,24}.30.
For reference, Monkey Island is running CR50 `0.5.201`, which is new enough to support the command as well (based on the values listed).
Further evidence it's supported by the CR50 is the CL that added those values:
cr50: Add get/set factory config command codes.
The bug that added them (b/275356839) also mentions both the CR50 and TI50.