From 51e4c1a76cafa0ddd429ffa78d0e6fdee179f731 Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Wed, 24 Jan 2018 17:42:51 -0700 Subject: soc/amd/stoneyridge: remove dependence on TSC The TSC rate is empirically swinging during early boot. That leaves timestamps and udelay()s to not be correct. To rectify this stop using TSC for all of these time sources. Instead use the performance TSC which is at a fixed 100MHz clock. That provides stable time sources and legit timestamps. BUG=b:72378235,b:72170796 Change-Id: Ia2693c415c557aac687bcb48ee69358ea1c53d67 Signed-off-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/23424 Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh --- src/soc/amd/stoneyridge/bootblock/bootblock.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/soc/amd/stoneyridge/bootblock') diff --git a/src/soc/amd/stoneyridge/bootblock/bootblock.c b/src/soc/amd/stoneyridge/bootblock/bootblock.c index 4b58287ba5..f9ae365c0d 100644 --- a/src/soc/amd/stoneyridge/bootblock/bootblock.c +++ b/src/soc/amd/stoneyridge/bootblock/bootblock.c @@ -28,6 +28,7 @@ #include #include #include +#include asmlinkage void bootblock_c_entry(uint64_t base_timestamp) { @@ -40,7 +41,8 @@ asmlinkage void bootblock_c_entry(uint64_t base_timestamp) if (!boot_cpu()) bootblock_soc_early_init(); /* APs will not return */ - bootblock_main_with_timestamp(base_timestamp); + /* TSC cannot be relied upon. Override the TSC value passed in. */ + bootblock_main_with_timestamp(timestamp_get()); } /* Set the MMIO Configuration Base Address and Bus Range. */ -- cgit v1.2.3