Attention is currently required from: Tarun Tuli, Subrata Banik, Dinesh Gehlot, Sridhar Siricilla.
Kangheui Won has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74256 )
Change subject: {commonlib, soc/intel/cmn/cse}: Store CSE firmware version into CBMEM ......................................................................
Patch Set 19:
(1 comment)
Patchset:
PS19: @Sridhar
Otherway this can be done by just checking the CSE Lite version in the CSE RW blob's metadata. In this scenario, coreboot need not interact with CSE and checking CSE blob only once is required. We plan to add the change later one this patchtrain get merged.
Again, it is intentional to talk to CSE and get the ISH version. Parsing BPDT and sub-partition headers is not 100% correct - e.g. if signature verification is failed CSE will silently fail and not load ISH firmware without any indication.
So we think getting the running ISH firmware version directly from CSE is necessary to ensure that SoC is running the ISH firmware we intended.