[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/cse: Add function to perform global reset lock

[Subrata Banik (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Angel Pons, Sridhar Siricilla, Werner Zeh, Patrick Rudolph, EricR Lai.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61520 )

Change subject: soc/intel/common/cse: Add function to perform global reset lock
......................................................................

Patch Set 9:

(1 comment)

File src/soc/intel/common/block/cse/cse.c:

https://review.coreboot.org/c/coreboot/+/61520/comment/5527801a_66a17217 
PS8, Line 1022: if (cse_is_hfs1_spi_protected())
> nit: I'd write the check as 
> `if (CONFIG(CONFIG_SOC_INTEL_CSE_LITE_SKU) || cse_cse_is_hfs1_spi_protected())` so that the function call can get optimised out for CSE Lite.

Yes, that makes sense to me as well because reading the recommendation from Sridhar looks like for CSE lite SKU, those original ME BWG recommendation is not relevant.