Attention is currently required from: Subrata Banik.
Dinesh Gehlot has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74209 )
Change subject: soc/intel/cmn/block/cse: Implement an API to get ISH version ......................................................................
Patch Set 4:
(1 comment)
File src/soc/intel/common/block/cse/cse.c:
https://review.coreboot.org/c/coreboot/+/74209/comment/b8913879_6debd2a3 PS3, Line 1233: if (memcmp(&version->ish_partition_info.prev_cse_fw_version, : &version->cur_cse_fw_version, size)) {
i still don't understand why u need to check the cse version here ? what is the value? […]
Acknowledged. Initially I kept it to ensure the stored ISH version data is not outdated, but yes there is no software reason for the same.