[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Add description to HMRFPO status

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38480 )

Change subject: soc/intel/common: Add description to HMRFPO status
......................................................................

soc/intel/common: Add description to HMRFPO status

Below changes are implemented:
1. Fix typos.
2. Rename 'padding' field of hmrfpo_get_status_resp struct to
   'reserved' to match with ME BWG Guide.
3. Add documentation for HMRFPO Status.

TEST=Build and boot hatch

Change-Id: I4db9bdf7386c48e17ed0373cf334ccff358d1951
Signed-off-by: Sridhar Siricilla 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38480
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
---
M src/soc/intel/common/block/cse/cse.c
M src/soc/intel/common/block/include/intelblocks/cse.h
2 files changed, 10 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: 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 8fbaba5..d323c76 100644
--- a/src/soc/intel/common/block/cse/cse.c
+++ b/src/soc/intel/common/block/cse/cse.c
@@ -683,7 +683,7 @@

/*
  * Sends HMRFPO Get Status command to CSE to get the HMRFPO status.
- * The status can be DISABLES/LOCKED/ENABLED
+ * The status can be DISABLED/LOCKED/ENABLED
  */
 int cse_hmrfpo_get_status(void)
 {
@@ -694,7 +694,7 @@
    struct hmrfpo_get_status_resp {
    	struct mkhi_hdr hdr;
    	uint8_t status;
-		uint8_t padding[3];
+		uint8_t reserved[3];
    } __packed;

struct hmrfpo_get_status_msg msg = {
diff --git a/src/soc/intel/common/block/include/intelblocks/cse.h b/src/soc/intel/common/block/include/intelblocks/cse.h
index 6233f7d..1377bd4 100644
--- a/src/soc/intel/common/block/include/intelblocks/cse.h
+++ b/src/soc/intel/common/block/include/intelblocks/cse.h
@@ -144,8 +144,16 @@
 #define CSE_RESET_ONLY	3

/* HMRFPO Status types */
+/* Host can't access ME region */
 #define MKHI_HMRFPO_DISABLED	0
+
+/*
+ * ME Firmware locked down HMRFPO Feature.
+ * Host can't access ME region.
+ */
 #define MKHI_HMRFPO_LOCKED	1
+
+/* Host can access ME region */
 #define MKHI_HMRFPO_ENABLED	2

/*