[coreboot] New patch to review for coreboot: 46a3788 Store timestamps before resuming from suspend

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Wed Nov 7 01:37:23 CET 2012

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1781


commit 46a3788ce2543d47c02a2ed721d0beec9202ff4d
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Oct 15 13:41:56 2012 -0700

    Store timestamps before resuming from suspend
    in the resume case, timestamps were collected in RAM stage
    but not stored in CBMEM. This leads to only a single time stamp
    covering 200ms being available for all of ram stage.
    Change-Id: Ibf0bb92caf5e032c12fe4e1b9b84b3624d499511
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
 src/boot/hardwaremain.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/boot/hardwaremain.c b/src/boot/hardwaremain.c
index 8348493..b08fe79 100644
--- a/src/boot/hardwaremain.c
+++ b/src/boot/hardwaremain.c
@@ -121,10 +121,6 @@ void hardwaremain(int boot_complete)
-	suspend_resume();
-	post_code(0x8a);
 	timestamp_add(TS_START_RAMSTAGE, timestamps[0]);
 	timestamp_add(TS_DEVICE_ENUMERATE, timestamps[1]);
@@ -132,6 +128,12 @@ void hardwaremain(int boot_complete)
 	timestamp_add(TS_DEVICE_ENABLE, timestamps[3]);
 	timestamp_add(TS_DEVICE_INITIALIZE, timestamps[4]);
 	timestamp_add(TS_DEVICE_DONE, timestamps[5]);
+	suspend_resume();
+	post_code(0x8a);

More information about the coreboot mailing list