summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElyes HAOUAS <ehaouas@noos.fr>2018-09-28 09:06:43 +0200
committerMartin Roth <martinroth@google.com>2018-10-11 21:05:07 +0000
commit603963e1ba4147ef31a72b94304708ab416e3b6a (patch)
treea6ea32124fe96635dd637333859e7daa6a6e5b73
parentde5d04011cb2f3de4e93381d58a4d01d5682a3f9 (diff)
downloadcoreboot-603963e1ba4147ef31a72b94304708ab416e3b6a.tar.xz
src: Replace MSR addresses with macros
Change-Id: I849dd406f5ccc733d4957eaf1c774745782f531a Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/28784 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Werner Zeh <werner.zeh@siemens.com>
-rw-r--r--src/cpu/intel/microcode/microcode.c10
-rw-r--r--src/northbridge/intel/haswell/report_platform.c4
-rw-r--r--src/soc/intel/broadwell/romstage/report_platform.c4
-rw-r--r--src/soc/intel/skylake/bootblock/report_platform.c4
4 files changed, 11 insertions, 11 deletions
diff --git a/src/cpu/intel/microcode/microcode.c b/src/cpu/intel/microcode/microcode.c
index 60342359c2..ae34347e26 100644
--- a/src/cpu/intel/microcode/microcode.c
+++ b/src/cpu/intel/microcode/microcode.c
@@ -102,7 +102,7 @@ void intel_microcode_load_unlocked(const void *microcode_patch)
msr.lo = (unsigned long)m + sizeof(struct microcode);
msr.hi = 0;
- wrmsr(0x79, msr);
+ wrmsr(IA32_BIOS_UPDT_TRIG, msr);
#if !defined(__ROMCC__)
printk(BIOS_DEBUG, "microcode: updated to revision "
@@ -158,12 +158,12 @@ const void *intel_microcode_find(void)
return NULL;
#endif
- /* CPUID sets MSR 0x8B iff a microcode update has been loaded. */
+ /* CPUID sets MSR 0x8B if a microcode update has been loaded. */
msr.lo = 0;
msr.hi = 0;
- wrmsr(0x8B, msr);
+ wrmsr(IA32_BIOS_SIGN_ID, msr);
eax = cpuid_eax(1);
- msr = rdmsr(0x8B);
+ msr = rdmsr(IA32_BIOS_SIGN_ID);
rev = msr.hi;
x86_model = (eax >> 4) & 0x0f;
x86_family = (eax >> 8) & 0x0f;
@@ -171,7 +171,7 @@ const void *intel_microcode_find(void)
pf = 0;
if ((x86_model >= 5) || (x86_family > 6)) {
- msr = rdmsr(0x17);
+ msr = rdmsr(IA32_PLATFORM_ID);
pf = 1 << ((msr.hi >> 18) & 7);
}
#if !defined(__ROMCC__)
diff --git a/src/northbridge/intel/haswell/report_platform.c b/src/northbridge/intel/haswell/report_platform.c
index 5b738440f5..04ef3d5ec9 100644
--- a/src/northbridge/intel/haswell/report_platform.c
+++ b/src/northbridge/intel/haswell/report_platform.c
@@ -50,9 +50,9 @@ static void report_cpu_info(void)
microcode_ver.lo = 0;
microcode_ver.hi = 0;
- wrmsr(0x8B, microcode_ver);
+ wrmsr(IA32_BIOS_SIGN_ID, microcode_ver);
cpuidr = cpuid(1);
- microcode_ver = rdmsr(0x8b);
+ microcode_ver = rdmsr(IA32_BIOS_SIGN_ID);
printk(BIOS_DEBUG, "CPU id(%x) ucode:%08x %s\n", cpuidr.eax, microcode_ver.hi, cpu_name);
aes = (cpuidr.ecx & (1 << 25)) ? 1 : 0;
txt = (cpuidr.ecx & (1 << 6)) ? 1 : 0;
diff --git a/src/soc/intel/broadwell/romstage/report_platform.c b/src/soc/intel/broadwell/romstage/report_platform.c
index fc8b37c733..44a3d2714c 100644
--- a/src/soc/intel/broadwell/romstage/report_platform.c
+++ b/src/soc/intel/broadwell/romstage/report_platform.c
@@ -113,9 +113,9 @@ static void report_cpu_info(void)
microcode_ver.lo = 0;
microcode_ver.hi = 0;
- wrmsr(0x8B, microcode_ver);
+ wrmsr(IA32_BIOS_SIGN_ID, microcode_ver);
cpuidr = cpuid(1);
- microcode_ver = rdmsr(0x8b);
+ microcode_ver = rdmsr(IA32_BIOS_SIGN_ID);
/* Look for string to match the name */
for (i = 0; i < ARRAY_SIZE(cpu_table); i++) {
diff --git a/src/soc/intel/skylake/bootblock/report_platform.c b/src/soc/intel/skylake/bootblock/report_platform.c
index a652b5268f..b59d351176 100644
--- a/src/soc/intel/skylake/bootblock/report_platform.c
+++ b/src/soc/intel/skylake/bootblock/report_platform.c
@@ -145,9 +145,9 @@ static void report_cpu_info(void)
microcode_ver.lo = 0;
microcode_ver.hi = 0;
- wrmsr(0x8B, microcode_ver);
+ wrmsr(IA32_BIOS_SIGN_ID, microcode_ver);
cpuidr = cpuid(1);
- microcode_ver = rdmsr(0x8b);
+ microcode_ver = rdmsr(IA32_BIOS_SIGN_ID);
/* Look for string to match the name */
for (i = 0; i < ARRAY_SIZE(cpu_table); i++) {