[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