From 74203de85185842f90ff1708b4f16445cc279688 Mon Sep 17 00:00:00 2001 From: Patrick Rudolph Date: Mon, 20 Nov 2017 11:57:01 +0100 Subject: intel/sandybridge: Don't hardcode platform type * Add a function to return CPU platform ID bits * Add a function to return platform type ** Platform id is 4 on Lenovo T430 (mobile) ** Platform id is 1 on HP8200 (desktop) * Use introduced method to handle platform specific code * Use enum for platform type * Report platform ID Change-Id: Ifbfc64c8cec98782d6efc987a4d4d5aeab1402ba Signed-off-by: Patrick Rudolph Reviewed-on: https://review.coreboot.org/22530 Tested-by: build bot (Jenkins) Reviewed-by: Felix Held --- src/cpu/intel/model_206ax/Makefile.inc | 4 ++++ src/cpu/intel/model_206ax/common.c | 31 ++++++++++++++++++++++++++++ src/cpu/intel/model_206ax/model_206ax.h | 1 + src/cpu/intel/model_206ax/model_206ax_init.c | 3 +++ 4 files changed, 39 insertions(+) create mode 100644 src/cpu/intel/model_206ax/common.c (limited to 'src/cpu/intel/model_206ax') diff --git a/src/cpu/intel/model_206ax/Makefile.inc b/src/cpu/intel/model_206ax/Makefile.inc index f6084ee202..2c1d961deb 100644 --- a/src/cpu/intel/model_206ax/Makefile.inc +++ b/src/cpu/intel/model_206ax/Makefile.inc @@ -5,6 +5,10 @@ subdirs-y += ../common ramstage-y += acpi.c +ramstage-y += common.c +romstage-y += common.c +smm-$(CONFIG_HAVE_SMI_HANDLER) += common.c + ramstage-y += tsc_freq.c romstage-y += tsc_freq.c smm-$(CONFIG_HAVE_SMI_HANDLER) += tsc_freq.c diff --git a/src/cpu/intel/model_206ax/common.c b/src/cpu/intel/model_206ax/common.c new file mode 100644 index 0000000000..9775efb30d --- /dev/null +++ b/src/cpu/intel/model_206ax/common.c @@ -0,0 +1,31 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2009 coresystems GmbH + * Copyright (C) 2011 The Chromium OS Authors. All rights reserved. + * + * 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 +#include +#include "model_206ax.h" + +#define IA32_PLATFORM_ID 0x17 + +int get_platform_id(void) +{ + msr_t msr; + + msr = rdmsr(IA32_PLATFORM_ID); + /* Read Platform Id Bits 52:50 */ + return (msr.hi >> 18) & 0x7; +} diff --git a/src/cpu/intel/model_206ax/model_206ax.h b/src/cpu/intel/model_206ax/model_206ax.h index 962b8302eb..7cb4069049 100644 --- a/src/cpu/intel/model_206ax/model_206ax.h +++ b/src/cpu/intel/model_206ax/model_206ax.h @@ -123,5 +123,6 @@ void set_power_limits(u8 power_limit_1_time); int cpu_config_tdp_levels(void); void smm_relocate(void); #endif +int get_platform_id(void); #endif diff --git a/src/cpu/intel/model_206ax/model_206ax_init.c b/src/cpu/intel/model_206ax/model_206ax_init.c index 5c60ed77b5..4be3d87aa0 100644 --- a/src/cpu/intel/model_206ax/model_206ax_init.c +++ b/src/cpu/intel/model_206ax/model_206ax_init.c @@ -514,6 +514,9 @@ static void model_206ax_init(struct device *cpu) fill_processor_name(processor_name); printk(BIOS_INFO, "CPU: %s.\n", processor_name); + /* Print platform ID */ + printk(BIOS_INFO, "CPU: platform id %x\n", get_platform_id()); + /* Setup MTRRs based on physical address size */ x86_setup_mtrrs_with_detect(); x86_mtrr_check(); -- cgit v1.2.3