[coreboot-gerrit] New patch to review for coreboot: google/veyron_shark: update to new timestamps API

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Jul 3 16:41:25 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10787

-gerrit

commit 4d3a85ca57844a0cf4b56f65405af3654ff16a58
Author: Patrick Georgi <pgeorgi at google.com>
Date:   Fri Jul 3 16:16:17 2015 +0200

    google/veyron_shark: update to new timestamps API
    
    Change-Id: Ic7f6b107fa582ff87de13e45a523b2cac5e959c3
    Signed-off-by: Patrick Georgi <pgeorgi at google.com>
---
 src/mainboard/google/veyron_shark/romstage.c | 26 +++++++-------------------
 1 file changed, 7 insertions(+), 19 deletions(-)

diff --git a/src/mainboard/google/veyron_shark/romstage.c b/src/mainboard/google/veyron_shark/romstage.c
index 68ab661..07170b9 100644
--- a/src/mainboard/google/veyron_shark/romstage.c
+++ b/src/mainboard/google/veyron_shark/romstage.c
@@ -86,13 +86,7 @@ static void sdmmc_power_off(void)
 
 void main(void)
 {
-#if CONFIG_COLLECT_TIMESTAMPS
-	uint64_t start_romstage_time;
-	uint64_t before_dram_time;
-	uint64_t after_dram_time;
-	uint64_t base_time = timestamp_get();
-	start_romstage_time = timestamp_get();
-#endif
+	timestamp_add_now(TS_START_ROMSTAGE);
 
 	console_init();
 	configure_l2ctlr();
@@ -103,13 +97,13 @@ void main(void)
 
 	/* vdd_log 1200mv is enough for ddr run 666Mhz */
 	regulate_vdd_log(1200);
-#if CONFIG_COLLECT_TIMESTAMPS
-	before_dram_time = timestamp_get();
-#endif
+
+	timestamp_add_now(TS_BEFORE_INITRAM);
+
 	sdram_init(get_sdram_config());
-#if CONFIG_COLLECT_TIMESTAMPS
-	after_dram_time = timestamp_get();
-#endif
+
+	timestamp_add_now(TS_AFTER_INITRAM);
+
 
 	/* Now that DRAM is up, add mappings for it and DMA coherency buffer. */
 	mmu_config_range((uintptr_t)_dram/MiB,
@@ -119,13 +113,7 @@ void main(void)
 
 	cbmem_initialize_empty();
 
-#if CONFIG_COLLECT_TIMESTAMPS
-	timestamp_init(base_time);
-	timestamp_add(TS_START_ROMSTAGE, start_romstage_time);
-	timestamp_add(TS_BEFORE_INITRAM, before_dram_time);
-	timestamp_add(TS_AFTER_INITRAM, after_dram_time);
 	timestamp_add_now(TS_END_ROMSTAGE);
-#endif
 
 	run_ramstage();
 }



More information about the coreboot-gerrit mailing list