Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38799 )
Change subject: soc/intel/common: Check prerequisites for HMRFPO_GET_STATUS command ......................................................................
soc/intel/common: Check prerequisites for HMRFPO_GET_STATUS command
Add code to check prerequisites before sending HMRFPO_GET_STATUS command.
TEST=Verified on hatch.
Change-Id: I4380e5096c6346d88aae6826d19a2f4ed1e97036 Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com --- M src/soc/intel/common/block/cse/cse.c 1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/99/38799/1
diff --git a/src/soc/intel/common/block/cse/cse.c b/src/soc/intel/common/block/cse/cse.c index 80f5a49..6035aaf 100644 --- a/src/soc/intel/common/block/cse/cse.c +++ b/src/soc/intel/common/block/cse/cse.c @@ -730,6 +730,11 @@
printk(BIOS_INFO, "HECI: Sending Get HMRFPO Status Command\n");
+ if (!cse_is_hfs1_cws_normal()) { + printk(BIOS_ERR, "HECI: CSE does not meet required prerequisites\n"); + return -1; + } + if (!heci_send_receive(&msg, sizeof(struct hmrfpo_get_status_msg), &resp, &resp_size)) { printk(BIOS_ERR, "HECI: HMRFPO send/receive fail\n");