Attention is currently required from: Tim Wawrzynczak, Sridhar Siricilla, Bernardo Perez Priego, Patrick Rudolph. Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58158 )
Change subject: soc/intel/common/cse: Split CSE metadata into two CBFS files ......................................................................
Patch Set 9:
(1 comment)
File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/58158/comment/8aca6bbd_a0b5a999 PS9, Line 598: if (!read_ver_field(version_str, &ptr, size, &cbfs_rw_version.major)) : return CSE_UPDATE_METADATA_ERROR; : if (!read_ver_field(version_str, &ptr, size, &cbfs_rw_version.minor)) : return CSE_UPDATE_METADATA_ERROR; : if (!read_ver_field(version_str, &ptr, size, &cbfs_rw_version.hotfix)) : return CSE_UPDATE_METADATA_ERROR; : if (!read_ver_field(version_str, &ptr, size, &cbfs_rw_version.build)) : return CSE_UPDATE_METADATA_ERROR;
I think version_str still needs to be unmapped here on the errors?, maybe reorganizing it would work […]
😢. Yes, that's required! Fixed in latest patchset.