[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Check prerequisites for HMRFPO_GET_STATUS command

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( 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

Send HMRFPO_GET_STATUS command when CSE's current working state is Normal.

TEST=Verified on hatch.

Change-Id: I4380e5096c6346d88aae6826d19a2f4ed1e97036
Signed-off-by: Sridhar Siricilla 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38799
Tested-by: build bot (Jenkins) 
Reviewed-by: Rizwan Qureshi 
---
M src/soc/intel/common/block/cse/cse.c
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Rizwan Qureshi: Looks good to me, approved

diff --git a/src/soc/intel/common/block/cse/cse.c b/src/soc/intel/common/block/cse/cse.c
index 39c30e9..afce985 100644
--- a/src/soc/intel/common/block/cse/cse.c
+++ b/src/soc/intel/common/block/cse/cse.c
@@ -713,6 +713,11 @@

printk(BIOS_INFO, "HECI: Sending Get HMRFPO Status Command\n");

+	if (!cse_is_hfs1_cws_normal()) {
+		printk(BIOS_ERR, "HECI: CSE's current working state is not Normal\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");