diff options
author | Aamir Bohra <aamir.bohra@intel.com> | 2017-05-25 14:12:01 +0530 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2017-06-05 00:32:48 +0200 |
commit | 842776e1dcb310e889d2ba922be9d7b81a1c2dd0 (patch) | |
tree | c6276bf0cffe9c8cd436689665b1814757cb92f6 /src/soc/intel/skylake | |
parent | 1fa16c9cb6c486cdd4b1bcb3734308b28b9c9a22 (diff) | |
download | coreboot-842776e1dcb310e889d2ba922be9d7b81a1c2dd0.tar.xz |
soc/intel/skylake: Use Intel timer common code
Use timer code from soc/intel/common. This code removes
monotonic timer refrence w.r.t MSR 24Mhz counter(0x637)
and use tsc timer.
Change-Id: I7fad620b11c9e5db128f646639c79ea58a0a574f
Signed-off-by: Aamir Bohra <aamir.bohra@intel.com>
Reviewed-on: https://review.coreboot.org/19912
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/soc/intel/skylake')
-rw-r--r-- | src/soc/intel/skylake/Kconfig | 8 | ||||
-rw-r--r-- | src/soc/intel/skylake/Makefile.inc | 12 | ||||
-rw-r--r-- | src/soc/intel/skylake/monotonic_timer.c | 39 | ||||
-rw-r--r-- | src/soc/intel/skylake/tsc_freq.c | 28 |
4 files changed, 2 insertions, 85 deletions
diff --git a/src/soc/intel/skylake/Kconfig b/src/soc/intel/skylake/Kconfig index 067e833090..c15e402052 100644 --- a/src/soc/intel/skylake/Kconfig +++ b/src/soc/intel/skylake/Kconfig @@ -62,6 +62,7 @@ config CPU_SPECIFIC_OPTIONS select SOC_INTEL_COMMON_BLOCK_SA select SOC_INTEL_COMMON_BLOCK_SATA select SOC_INTEL_COMMON_BLOCK_SMBUS + select SOC_INTEL_COMMON_BLOCK_TIMER select SOC_INTEL_COMMON_BLOCK_UART select SOC_INTEL_COMMON_BLOCK_XHCI select SOC_INTEL_COMMON_NHLT @@ -72,6 +73,7 @@ config CPU_SPECIFIC_OPTIONS select SSE2 select SUPPORT_CPU_UCODE_IN_CBFS select TSC_CONSTANT_RATE + select TSC_MONOTONIC_TIMER select TSC_SYNC_MFENCE select UDELAY_TSC select ACPI_NHLT @@ -158,12 +160,6 @@ config IED_REGION_SIZE hex default 0x400000 -config MONOTONIC_TIMER_MSR - def_bool y - select HAVE_MONOTONIC_TIMER - help - Provide a monotonic timer using the 24MHz MSR counter. - config PCR_BASE_ADDRESS hex default 0xfd000000 diff --git a/src/soc/intel/skylake/Makefile.inc b/src/soc/intel/skylake/Makefile.inc index 3ce88a73af..93eacfb0a8 100644 --- a/src/soc/intel/skylake/Makefile.inc +++ b/src/soc/intel/skylake/Makefile.inc @@ -18,33 +18,27 @@ bootblock-$(CONFIG_UART_DEBUG) += bootblock/uart.c bootblock-$(CONFIG_UART_DEBUG) += uart_debug.c bootblock-y += gpio.c bootblock-y += gspi.c -bootblock-y += monotonic_timer.c bootblock-y += pch.c bootblock-y += pmutil.c bootblock-y += spi.c -bootblock-y += tsc_freq.c verstage-y += gspi.c -verstage-y += monotonic_timer.c verstage-y += pch.c verstage-$(CONFIG_UART_DEBUG) += uart_debug.c verstage-y += pmutil.c verstage-y += i2c.c verstage-y += spi.c -verstage-y += tsc_freq.c romstage-y += gpio.c romstage-y += gspi.c romstage-y += i2c.c romstage-y += memmap.c -romstage-y += monotonic_timer.c romstage-y += me.c romstage-y += pch.c romstage-y += pei_data.c romstage-y += pmutil.c romstage-$(CONFIG_PLATFORM_USES_FSP2_0) += reset.c romstage-y += spi.c -romstage-y += tsc_freq.c romstage-$(CONFIG_UART_DEBUG) += uart_debug.c ramstage-$(CONFIG_HAVE_ACPI_TABLES) += acpi.c @@ -63,7 +57,6 @@ ramstage-y += irq.c ramstage-y += lpc.c ramstage-y += me.c ramstage-y += memmap.c -ramstage-y += monotonic_timer.c ramstage-$(CONFIG_PLATFORM_USES_FSP1_1) += opregion.c ramstage-y += pch.c ramstage-y += pei_data.c @@ -76,24 +69,19 @@ ramstage-y += smi.c ramstage-y += smmrelocate.c ramstage-y += spi.c ramstage-y += systemagent.c -ramstage-y += tsc_freq.c ramstage-y += uart.c ramstage-$(CONFIG_UART_DEBUG) += uart_debug.c ramstage-y += vr_config.c smm-y += cpu_info.c smm-y += gpio.c -smm-y += monotonic_timer.c smm-y += pch.c smm-y += pmutil.c smm-y += smihandler.c smm-$(CONFIG_SPI_FLASH_SMM) += spi.c -smm-y += tsc_freq.c smm-$(CONFIG_UART_DEBUG) += uart_debug.c postcar-y += memmap.c -postcar-y += monotonic_timer.c -postcar-y += tsc_freq.c postcar-$(CONFIG_UART_DEBUG) += uart_debug.c # cpu_microcode_bins += ??? diff --git a/src/soc/intel/skylake/monotonic_timer.c b/src/soc/intel/skylake/monotonic_timer.c deleted file mode 100644 index 7e3322340e..0000000000 --- a/src/soc/intel/skylake/monotonic_timer.c +++ /dev/null @@ -1,39 +0,0 @@ -/* - * This file is part of the coreboot project. - * - * Copyright (C) 2014 Google Inc. - * Copyright (C) 2015 Intel Corporation. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; version 2 of the License. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#include <stdint.h> -#include <cpu/x86/msr.h> -#include <timer.h> -#include <soc/msr.h> - -static inline uint32_t read_counter_msr(void) -{ - /* - * Even though the MSR is 64-bit it is assumed that the hardware - * is polled frequently enough to only use the lower 32-bits. - */ - msr_t counter_msr; - - counter_msr = rdmsr(MSR_COUNTER_24_MHZ); - - return counter_msr.lo; -} - -void timer_monotonic_get(struct mono_time *mt) -{ - /* Always increases. Don't normalize to 0 between stages. */ - mono_time_set_usecs(mt, read_counter_msr() / 24); -} diff --git a/src/soc/intel/skylake/tsc_freq.c b/src/soc/intel/skylake/tsc_freq.c deleted file mode 100644 index 8a4ff461b2..0000000000 --- a/src/soc/intel/skylake/tsc_freq.c +++ /dev/null @@ -1,28 +0,0 @@ -/* - * This file is part of the coreboot project. - * - * Copyright (C) 2014 Google Inc. - * Copyright (C) 2015 Intel Corporation. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; version 2 of the License. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#include <stdint.h> -#include <cpu/x86/msr.h> -#include <cpu/x86/tsc.h> -#include <soc/msr.h> - -unsigned long tsc_freq_mhz(void) -{ - msr_t platform_info; - - platform_info = rdmsr(MSR_PLATFORM_INFO); - return CONFIG_CPU_BCLK_MHZ * ((platform_info.lo >> 8) & 0xff); -} |