From acbf1541eeedec58ce8a31c6847b2296f40bf4bd Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 13 Jul 2021 16:44:18 +0200 Subject: src: use mca_clear_status function instead of open coding Change-Id: I53413b4051b79d7c2f24b1191ce877155e654400 Signed-off-by: Felix Held Reviewed-on: https://review.coreboot.org/c/coreboot/+/56259 Reviewed-by: Raul Rangel Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/cpu/cpulib.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'src/soc/intel/common/block') diff --git a/src/soc/intel/common/block/cpu/cpulib.c b/src/soc/intel/common/block/cpu/cpulib.c index a245824821..93725132b2 100644 --- a/src/soc/intel/common/block/cpu/cpulib.c +++ b/src/soc/intel/common/block/cpu/cpulib.c @@ -336,17 +336,14 @@ uint32_t cpu_get_max_turbo_ratio(void) void mca_configure(void) { - msr_t msr; int i; const unsigned int num_banks = mca_get_bank_count(); printk(BIOS_DEBUG, "Clearing out pending MCEs\n"); - msr.lo = msr.hi = 0; + mca_clear_status(); for (i = 0; i < num_banks; i++) { - /* Clear the machine check status */ - wrmsr(IA32_MC_STATUS(i), msr); /* Initialize machine checks */ wrmsr(IA32_MC_CTL(i), (msr_t) {.lo = 0xffffffff, .hi = 0xffffffff}); -- cgit v1.2.3