Sridhar Siricilla has uploaded this change for review.

View Change

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 <sridhar.siricilla@intel.com>
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();

To view, visit change 43537. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I90e9f5897408bfc37a69cf0bb23bff18a146b9e2
Gerrit-Change-Number: 43537
Gerrit-PatchSet: 1
Gerrit-Owner: Sridhar Siricilla <sridhar.siricilla@intel.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-MessageType: newchange