[S] Change in coreboot[master]: soc/intel/alderlake: Add entries to eventLog on invocation of early SOL

[Julius Werner (Code Review)]

Julius Werner has submitted this change. ( https://review.coreboot.org/c/coreboot/+/71295 )

Change subject: soc/intel/alderlake: Add entries to eventLog on invocation of early SOL
......................................................................

soc/intel/alderlake: Add entries to eventLog on invocation of early SOL

If we show the user early signs of life during CSE FW sync or MRC
(re)training, log these to the eventLog (ELOG_TYPE_FW_EARLY_SOL).

These can be used to ensure persistence across global reset (e.g. after
CSE sync) so that they can be later retrieved in order to build things
such as test automation ensuring that we went through the SOL
path/display initialized.

BUG=b:264648959
TEST=event shows in eventlog after CSE sync and/or MRC

Change-Id: I8181370633a1ecff77b051d3110f593c3eb484a2
Signed-off-by: Tarun Tuli 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71295
Reviewed-by: Jérémy Compostella 
Reviewed-by: Subrata Banik 
Reviewed-by: Nick Vaccaro 
Tested-by: build bot (Jenkins) 
Reviewed-by: Julius Werner 
---
M src/soc/intel/alderlake/romstage/fsp_params.c
M src/soc/intel/alderlake/romstage/romstage.c
M src/soc/intel/alderlake/romstage/ux.c
M src/soc/intel/alderlake/romstage/ux.h
4 files changed, 39 insertions(+), 7 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Julius Werner: Looks good to me, approved
  Nick Vaccaro: Looks good to me, approved
  Subrata Banik: Looks good to me, approved
  Jérémy Compostella: Looks good to me, but someone else must approve

diff --git a/src/soc/intel/alderlake/romstage/fsp_params.c b/src/soc/intel/alderlake/romstage/fsp_params.c
index 6fec2c7..2e8a73c 100644
--- a/src/soc/intel/alderlake/romstage/fsp_params.c
+++ b/src/soc/intel/alderlake/romstage/fsp_params.c
@@ -6,6 +6,7 @@
 #include <cpu/intel/cpu_ids.h>
 #include <device/device.h>
 #include <drivers/wifi/generic/wifi.h>
+#include <elog.h>
 #include <fsp/fsp_debug_event.h>
 #include <fsp/util.h>
 #include <gpio.h>
@@ -423,9 +424,10 @@
     * training. Memory training can take a while so let's inform the end
     * user with an on-screen text message.
     */
-	if (!arch_upd->NvsBufferPtr)
-		ux_inform_user_of_update_operation("memory training");
-
+	if (!arch_upd->NvsBufferPtr) {
+		if (ux_inform_user_of_update_operation("memory training"))
+			elog_add_event_byte(ELOG_TYPE_FW_EARLY_SOL, ELOG_FW_EARLY_SOL_MRC);
+	}
    config = config_of_soc();

soc_memory_init_params(m_cfg, config);
diff --git a/src/soc/intel/alderlake/romstage/romstage.c b/src/soc/intel/alderlake/romstage/romstage.c
index 2ccadbc..2c0bdea 100644
--- a/src/soc/intel/alderlake/romstage/romstage.c
+++ b/src/soc/intel/alderlake/romstage/romstage.c
@@ -4,6 +4,7 @@
 #include <cbmem.h>
 #include <cf9_reset.h>
 #include <console/console.h>
+#include <elog.h>
 #include <fsp/util.h>
 #include <intelblocks/cfg.h>
 #include <intelblocks/cse.h>
@@ -149,7 +150,8 @@

void cse_fw_update_misc_oper(void)
 {
-	ux_inform_user_of_update_operation("CSE update");
+	if (ux_inform_user_of_update_operation("CSE update"))
+		elog_add_event_byte(ELOG_TYPE_FW_EARLY_SOL, ELOG_FW_EARLY_SOL_CSE_SYNC);
 }

void cse_board_reset(void)
diff --git a/src/soc/intel/alderlake/romstage/ux.c b/src/soc/intel/alderlake/romstage/ux.c
index 66b2bef..5dba194 100644
--- a/src/soc/intel/alderlake/romstage/ux.c
+++ b/src/soc/intel/alderlake/romstage/ux.c
@@ -6,13 +6,14 @@

#include "ux.h"

-void ux_inform_user_of_update_operation(const char *name)
+bool ux_inform_user_of_update_operation(const char *name)
 {
    if (!CONFIG(MAINBOARD_HAS_EARLY_LIBGFXINIT) ||
        !early_graphics_init())
-		return;
+		return false;

printk(BIOS_INFO, "Informing user on-display of %s.\n", name);
    vga_write_text(VGA_TEXT_CENTER, VGA_TEXT_HORIZONTAL_MIDDLE,
    	       "Your device is finishing an update. This may take 1-2 minutes.\nPlease do not turn off your device.");
+	return true;
 }
diff --git a/src/soc/intel/alderlake/romstage/ux.h b/src/soc/intel/alderlake/romstage/ux.h
index f09daed..e7e1d99 100644
--- a/src/soc/intel/alderlake/romstage/ux.h
+++ b/src/soc/intel/alderlake/romstage/ux.h
@@ -1,3 +1,3 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-void ux_inform_user_of_update_operation(const char *name);
+bool ux_inform_user_of_update_operation(const char *name);