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/cpu/amd/agesa/family14/model_14_init.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'src/cpu/amd/agesa/family14') diff --git a/src/cpu/amd/agesa/family14/model_14_init.c b/src/cpu/amd/agesa/family14/model_14_init.c index 76c8521761..9539c3deef 100644 --- a/src/cpu/amd/agesa/family14/model_14_init.c +++ b/src/cpu/amd/agesa/family14/model_14_init.c @@ -15,9 +15,7 @@ static void model_14_init(struct device *dev) { - u8 i; msr_t msr; - unsigned int num_banks; int msrno; #if CONFIG(LOGICAL_CPUS) u32 siblings; @@ -59,11 +57,7 @@ static void model_14_init(struct device *dev) x86_enable_cache(); /* zero the machine check error status registers */ - num_banks = mca_get_bank_count(); - msr.lo = 0; - msr.hi = 0; - for (i = 0; i < num_banks; i++) - wrmsr(IA32_MC_STATUS(i), msr); + mca_clear_status(); /* Enable the local CPU APICs */ setup_lapic(); -- cgit v1.2.3