[coreboot-gerrit] [M] Change in coreboot[master]: soc/intel/: Store CSE firmware version into cbmem table

[Sridhar Siricilla (Code Review)]

Attention is currently required from: Tarun Tuli, Subrata Banik, Dinesh Gehlot.

Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74256 )

Change subject: soc/intel/: Store CSE firmware version into cbmem table
......................................................................

Patch Set 7:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/74256/comment/77377d6c_d5b69a41 
PS7, Line 1089: 
              : 	const struct cse_bp_entry *cse_bp = cse_get_bp_entry(RW, &cse_bp_info.bp_info);
              : 	struct cse_fw_partition_info *version;
              : 	version = cbmem_add(CBMEM_ID_CSE_PARTITION_VERSION, sizeof(*version));
              : 	version->cur_cse_fw_version.major = cse_bp->fw_ver.major;
              : 	version->cur_cse_fw_version.minor = cse_bp->fw_ver.minor;
              : 	version->cur_cse_fw_version.hotfix = cse_bp->fw_ver.hotfix;
              : 	version->cur_cse_fw_version.build = cse_bp->fw_ver.build;
              : }
              : 
Can't this be handled in the cse_get_bp_info() itself? Doing so, we can avoid resending the command to CSE.