[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Log CSE FW Status Registers before triggering recovery

[Sridhar Siricilla (Code Review)]

Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43537 )

Change subject: soc/intel/common: Log CSE FW Status Registers before triggering recovery
......................................................................

soc/intel/common: Log CSE FW Status Registers before triggering recovery

The patch log CSE Firmware Status Registers(FWSTS1, FWSTS2 & FWSTS3) before
triggering recovery to help debugging.

Signed-off-by: Sridhar Siricilla 
Change-Id: I90e9f5897408bfc37a69cf0bb23bff18a146b9e2
---
M src/soc/intel/common/block/cse/cse_lite.c
1 file changed, 8 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/43537/1

diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c
index a12f2d0..e51b8d6 100644
--- a/src/soc/intel/common/block/cse/cse_lite.c
+++ b/src/soc/intel/common/block/cse/cse_lite.c
@@ -153,8 +153,16 @@
    struct cse_bp_info bp_info;
 } __packed;

+static void cse_log_status_registers(void)
+{
+	printk(BIOS_DEBUG, "cse_lite: CSE status registers: HFSTS1:0x%x, HFSTS2:0x%x HFST3:0x%x\n",
+		me_read_config32(PCI_ME_HFSTS1), me_read_config32(PCI_ME_HFSTS2), me_read_config32(PCI_ME_HFSTS3));
+}
+
 static void cse_trigger_recovery(uint8_t rec_sub_code)
 {
+	/* Log CSE Firmware Status Registers to help debugging */
+	cse_log_status_registers();
    if (CONFIG(VBOOT)) {
    	struct vb2_context *ctx;
    	ctx = vboot_get_context();