[coreboot-gerrit] New patch to review for coreboot: lib: Add time stamp when starting to finalize chips

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Mon Dec 5 23:35:29 CET 2016


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17725

-gerrit

commit 907c8aa5922ad75912242a58a81c99718c72c2b8
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Mon Dec 5 23:06:37 2016 +0100

    lib: Add time stamp when starting to finalize chips
    
    Add the new time stamps *finalize chips* to track, when the method
    `dev_finalize_chips()` is called, so that the real time of
    `write_tables()` is known.
    
    Change-Id: I65ca0ec4c07c2eaa14982989cb206d23d8a2293f
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/commonlib/include/commonlib/timestamp_serialized.h | 2 ++
 src/lib/hardwaremain.c                                 | 1 +
 2 files changed, 3 insertions(+)

diff --git a/src/commonlib/include/commonlib/timestamp_serialized.h b/src/commonlib/include/commonlib/timestamp_serialized.h
index 29f5e0d..ee5cd6f 100644
--- a/src/commonlib/include/commonlib/timestamp_serialized.h
+++ b/src/commonlib/include/commonlib/timestamp_serialized.h
@@ -56,6 +56,7 @@ enum timestamp_id {
 	TS_DEVICE_DONE = 70,
 	TS_CBMEM_POST = 75,
 	TS_WRITE_TABLES = 80,
+	TS_FINALIZE_CHIPS = 85,
 	TS_LOAD_PAYLOAD = 90,
 	TS_ACPI_WAKE_JUMP = 98,
 	TS_SELFBOOT_JUMP = 99,
@@ -141,6 +142,7 @@ static const struct timestamp_id_to_name {
 	{ TS_DEVICE_DONE,	"device setup done" },
 	{ TS_CBMEM_POST,	"cbmem post" },
 	{ TS_WRITE_TABLES,	"write tables" },
+	{ TS_FINALIZE_CHIPS,	"finalize chips" },
 	{ TS_LOAD_PAYLOAD,	"load payload" },
 	{ TS_ACPI_WAKE_JUMP,	"ACPI wake jump" },
 	{ TS_SELFBOOT_JUMP,	"selfboot jump" },
diff --git a/src/lib/hardwaremain.c b/src/lib/hardwaremain.c
index ef78960..7e20988 100644
--- a/src/lib/hardwaremain.c
+++ b/src/lib/hardwaremain.c
@@ -212,6 +212,7 @@ static boot_state_t bs_write_tables(void *arg)
 	 */
 	write_tables();
 
+	timestamp_add_now(TS_FINALIZE_CHIPS);
 	dev_finalize_chips();
 
 	return BS_PAYLOAD_LOAD;



More information about the coreboot-gerrit mailing list