[coreboot-gerrit] [S] Change in coreboot[main]: util/cbfstool: Add eventLog support for ELOG_TYPE_FW_LATE_SOL

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83462?usp=email )

Change subject: util/cbfstool: Add eventLog support for ELOG_TYPE_FW_LATE_SOL
......................................................................

util/cbfstool: Add eventLog support for ELOG_TYPE_FW_LATE_SOL

In order to support logging events for when we show early signs
of life to the user during CSE FW syncs add support for the
ELOG_TYPE_FW_LATE_SOL type.

BUG=b:305898363
TEST=verify event shows in eventlog CSE sync.

Change-Id: I862db946f6ff622ac83072e6bf27832732c0c318
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/83462
Tested-by: build bot (Jenkins) 
Reviewed-by: Dinesh Gehlot 
---
M util/cbfstool/eventlog.c
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  Dinesh Gehlot: Looks good to me, approved
  build bot (Jenkins): Verified

diff --git a/util/cbfstool/eventlog.c b/util/cbfstool/eventlog.c
index e090ebe..5e76df2 100644
--- a/util/cbfstool/eventlog.c
+++ b/util/cbfstool/eventlog.c
@@ -169,6 +169,7 @@
    	{ELOG_TYPE_PSR_DATA_BACKUP, "PSR data backup"},
    	{ELOG_TYPE_PSR_DATA_LOST, "PSR data lost"},
    	{ELOG_TYPE_FW_SPLASH_SCREEN, "Firmware Splash Screen"},
+		{ELOG_TYPE_FW_LATE_SOL, "Late Sign of Life "},
    	{ELOG_TYPE_EOL, "End of log"},
    };

@@ -483,6 +484,11 @@
    	{0, NULL},
    };

+	static const struct valstr late_sol_path_types[] = {
+		{ELOG_FW_LATE_SOL_CSE_SYNC, "CSE Sync Late SOL Screen Shown"},
+		{0, NULL},
+	};
+
    size_t elog_type_to_min_size[] = {
    	[ELOG_TYPE_LOG_CLEAR]		= sizeof(uint16_t),
    	[ELOG_TYPE_BOOT]		= sizeof(uint32_t),
@@ -504,6 +510,7 @@
    	[ELOG_TYPE_FW_EARLY_SOL]	= sizeof(uint8_t),
    	[ELOG_TYPE_PSR_DATA_BACKUP]	= sizeof(uint8_t),
    	[ELOG_TYPE_FW_SPLASH_SCREEN]	= sizeof(uint8_t),
+		[ELOG_TYPE_FW_LATE_SOL]	= sizeof(uint8_t),
    	[0xff]				= 0,
    };

@@ -670,6 +677,11 @@
    	eventlog_printf("%s", *fw_splash_screen_event ? "Enabled" : "Disabled");
    	break;
    }
+	case ELOG_TYPE_FW_LATE_SOL: {
+		const uint8_t *sol_event = event_get_data(event);
+		eventlog_printf("%s", val2str(*sol_event, late_sol_path_types));
+		break;
+	}
    default:
    	break;
    }