commit bf73c498d448678fd19354d66c03d852fcb5ec1d
author Bora Guvendik <bora.guvendik@intel.com> Mon Nov 22 16:03:39 2021 -0800
committer Felix Held <felix-coreboot@felixheld.de> Fri Jan 21 22:42:19 2022 +0000
tree 68678157308ff8e00a2dc067b44a6378aa90fbd8
parent f33c9bf79a8ee80067920d7dd0ec15360b42703b

timestamp: Allow timestamp_add to accept a negative number

Change timestamp_add to accept negative values for events that took
place before coreboot started executing.

TEST=Boot to OS, check cbmem -t

Signed-off-by: Bora Guvendik <bora.guvendik@intel.com>
Change-Id: I90afc13a8e92693d86e3358f05e0a0cb7cdbca9b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59554
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>

src/lib/timestamp.c

diff --git a/src/lib/timestamp.c b/src/lib/timestamp.c
index 7347d07..f8e4ab8 100644
--- a/src/lib/timestamp.c
+++ b/src/lib/timestamp.c
@@ -102,7 +102,7 @@
 }
 
 static void timestamp_add_table_entry(struct timestamp_table *ts_table,
-				      enum timestamp_id id, uint64_t ts_time)
+				      enum timestamp_id id, int64_t ts_time)
 {
 	struct timestamp_entry *tse;
 
@@ -117,7 +117,7 @@
 		printk(BIOS_ERR, "ERROR: Timestamp table full\n");
 }
 
-void timestamp_add(enum timestamp_id id, uint64_t ts_time)
+void timestamp_add(enum timestamp_id id, int64_t ts_time)
 {
 	struct timestamp_table *ts_table;
 
@@ -135,7 +135,7 @@
 	timestamp_add_table_entry(ts_table, id, ts_time);
 
 	if (CONFIG(TIMESTAMPS_ON_CONSOLE))
-		printk(BIOS_INFO, "Timestamp - %s: %llu\n", timestamp_name(id), ts_time);
+		printk(BIOS_INFO, "Timestamp - %s: %lld\n", timestamp_name(id), ts_time);
 }
 
 void timestamp_add_now(enum timestamp_id id)