[coreboot-gerrit] New patch to review for coreboot: soc/intel/quark: Add monotonic timer support
Lee Leahy (leroy.p.leahy@intel.com)
gerrit at coreboot.org
Sat Dec 31 18:02:16 CET 2016
Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18002
-gerrit
commit 100a6b77fd6066d681bf74567efd085096455a54
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date: Sat Dec 31 08:21:56 2016 -0800
soc/intel/quark: Add monotonic timer support
Add the Kconfig value HAVE_MONOTONIC_TIMER and the routine to read the
TSC for the monotonic timer. Simplify the routine to get the TSC
frequency.
TEST=Build and run on Galileo Gen2
Change-Id: I806fb864b01e39277bf2d6276254b0543930c2f6
Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
---
src/soc/intel/quark/Kconfig | 1 +
src/soc/intel/quark/tsc_freq.c | 16 +++++++---------
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/soc/intel/quark/Kconfig b/src/soc/intel/quark/Kconfig
index c5efd4a..8182caa 100644
--- a/src/soc/intel/quark/Kconfig
+++ b/src/soc/intel/quark/Kconfig
@@ -29,6 +29,7 @@ config CPU_SPECIFIC_OPTIONS
select BOOTBLOCK_SAVE_BIST_AND_TIMESTAMP
select C_ENVIRONMENT_BOOTBLOCK
select HAVE_HARD_RESET
+ select HAVE_MONOTONIC_TIMER
select NO_MMCONF_SUPPORT
select REG_SCRIPT
select RELOCATABLE_RAMSTAGE
diff --git a/src/soc/intel/quark/tsc_freq.c b/src/soc/intel/quark/tsc_freq.c
index eb8c46e..e6d0369 100644
--- a/src/soc/intel/quark/tsc_freq.c
+++ b/src/soc/intel/quark/tsc_freq.c
@@ -17,20 +17,18 @@
#include <stdint.h>
#include <cpu/x86/msr.h>
#include <cpu/x86/tsc.h>
+#include <timer.h>
-static unsigned long bus_freq_khz(void)
+unsigned long tsc_freq_mhz(void)
{
/* CPU freq = 400 MHz */
- return 400 * 1000;
+ return 400;
}
-unsigned long tsc_freq_mhz(void)
+void timer_monotonic_get(struct mono_time *mt)
{
- /* assume ratio=1 */
- unsigned bclk_khz = bus_freq_khz();
-
- if (!bclk_khz)
- return 0;
+ uint64_t tsc_value;
- return (bclk_khz * 1) / 1000;
+ tsc_value = rdtscll();
+ mt->microseconds = tsc_value / tsc_freq_mhz();
}
More information about the coreboot-gerrit
mailing list