Attention is currently required from: Nico Huber, Subrata Banik, Patrick Rudolph. Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61380 )
Change subject: soc/inte/common: Add support to control coreboot and Intel SoC features ......................................................................
Patch Set 2:
(1 comment)
File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/61380/comment/3a6d6281_b88c76ea PS2, Line 666: is_cse_fw_update_enabled
is there any restriction that on after FSI, we can't read the softstrap ? […]
So, if I read it correctly there is no restriction while reading OEM sections prior or post FSI. hence, why we would like to introduce a Kconfig to guard the read operation which would eventually get drop during FSI. Any owner for ensuring the same across program ?
I don't see any benefit running this code since it is not useful once RO is locked. Hence, I want to guard the code and remove it before FSI.
I could sense there are some underlying assumption which is not documented anything, either please raise a bug or write a .md file to clear out those assumptions.
There are no assumptions or hidden things in this regard, I will update commit message with more information.