Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35402 )
Change subject: soc/intel/common/block/cse: Add boot partition related APIs ......................................................................
Patch Set 16:
(5 comments)
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... File src/soc/intel/common/block/cse/cse_bp.c:
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... PS14, Line 59: static struct cse_boot_partition_info cse_bp_info; : static bool cse_bp_init = false;
With this approach, complete response structure will be static. […]
Done
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... PS14, Line 125: memcpy((void *)&cse_bp_info, (void *)&info_resp.bp_info, : sizeof(struct cse_boot_partition_info));
Agreed, please see my earlier response.
Done
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... PS14, Line 128: return 1;
Please see my earlier response.
Done
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... PS14, Line 135: static uint8_t cse_init_bp_info(bool force_cse_init) : { : if (cse_bp_init == false || force_cse_init == true) { : if (!cse_update_boot_partition_info()) { : printk(BIOS_ERR, "Boot partition update fail\n"); : return 0; : } : cse_print_boot_partition_info(); : } : cse_bp_init = true; : return 1; : }
This function checks if it needs to update boot partition info or not. […]
Done
https://review.coreboot.org/c/coreboot/+/35402/14/src/soc/intel/common/block... PS14, Line 214: if (!cse_init_bp_info(false)) : return 0;
Please see my above response.
Done