[coreboot-gerrit] [S] Change in coreboot[main]: util/cbfstool/eventlog: Add low battery event type

[Subrata Banik (Code Review)]

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

Change subject: util/cbfstool/eventlog: Add low battery event type
......................................................................

util/cbfstool/eventlog: Add low battery event type

This commit adds a new event type, `ELOG_TYPE_LOW_BATTERY_INDICATOR`,
to the event log. This event is logged when the system boots due to
a low battery condition.  It includes the reason for the shutdown,
currently only supporting "Poweroff: Low Battery".

BUG=b:339673254
TEST=Able to capture the eventlog for low battery boot event.

Change-Id: I5cc5e5f540657c7dfd174a4928e697a272da813a
Signed-off-by: Subrata Banik 
---
M src/commonlib/bsd/include/commonlib/bsd/elog.h
M util/cbfstool/eventlog.c
2 files changed, 15 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/23/86223/1

diff --git a/src/commonlib/bsd/include/commonlib/bsd/elog.h b/src/commonlib/bsd/include/commonlib/bsd/elog.h
index f7dc4a1..01f97f6 100644
--- a/src/commonlib/bsd/include/commonlib/bsd/elog.h
+++ b/src/commonlib/bsd/include/commonlib/bsd/elog.h
@@ -389,6 +389,9 @@
 #define  ELOG_FW_POST_RAM_CSE_SYNC         0x1
 #define  ELOG_FW_CSE_SYNC_AT_PAYLOAD       0x2

+#define ELOG_TYPE_LOW_BATTERY_INDICATOR   0xbe
+#define  ELOG_FW_ISSUE_SHUTDOWN            0x0
+
 /* Only the 7-LSB are used for size */
 #define ELOG_MAX_EVENT_SIZE                    0x7F

diff --git a/util/cbfstool/eventlog.c b/util/cbfstool/eventlog.c
index 4746670..c4e6b64 100644
--- a/util/cbfstool/eventlog.c
+++ b/util/cbfstool/eventlog.c
@@ -170,6 +170,7 @@
    	{ELOG_TYPE_PSR_DATA_LOST, "PSR data lost"},
    	{ELOG_TYPE_FW_SPLASH_SCREEN, "Firmware Splash Screen"},
    	{ELOG_TYPE_FW_CSE_SYNC, "Firmware CSE sync"},
+		{ELOG_TYPE_LOW_BATTERY_INDICATOR, "Low Battery Boot "},
    	{ELOG_TYPE_EOL, "End of log"},
    };

@@ -491,6 +492,11 @@
    	{0, NULL},
    };

+	static const struct valstr low_battery_path_types[] = {
+		{ELOG_FW_ISSUE_SHUTDOWN, "Poweroff: Low Battery"},
+		{0, NULL},
+	};
+
    size_t elog_type_to_min_size[] = {
    	[ELOG_TYPE_LOG_CLEAR]		= sizeof(uint16_t),
    	[ELOG_TYPE_BOOT]		= sizeof(uint32_t),
@@ -513,6 +519,7 @@
    	[ELOG_TYPE_PSR_DATA_BACKUP]	= sizeof(uint8_t),
    	[ELOG_TYPE_FW_SPLASH_SCREEN]	= sizeof(uint8_t),
    	[ELOG_TYPE_FW_CSE_SYNC]		= sizeof(uint8_t),
+		[ELOG_TYPE_LOW_BATTERY_INDICATOR]	= sizeof(uint8_t),
    	[0xff]				= 0,
    };

@@ -684,6 +691,11 @@
    	eventlog_printf("%s", val2str(*cse_event, cse_sync_path_types));
    	break;
    }
+	case ELOG_TYPE_LOW_BATTERY_INDICATOR: {
+		const uint8_t *sol_event = event_get_data(event);
+		eventlog_printf("%s", val2str(*sol_event, late_sol_path_types));
+		break;
+	}
    default:
    	break;
    }