summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2021-07-13 02:00:55 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-07-14 02:25:43 +0000
commitc75c8c15b99b5045caa77b2447cfff69b725576c (patch)
tree956a34c1128cadc1c86f733c5888eb97535933dd /src/soc
parentbad21a4bd204e4fd22fb683f389dfde21000e22d (diff)
soc/intel/common/block/cpu/cpulib: use mca_get_bank_count()
Use the common mca_get_bank_count function instead of open-coding the functionality to get the MCA bank number. Also re-type the num_banks variable from signed in to unsigned int, since the number of MCA bank is always positive, and make it constant. Change-Id: I449c74629ff16057c4559d7fd3620208230560f5 Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/56245 Reviewed-by: Raul Rangel <rrangel@chromium.org> Reviewed-by: Patrick Rudolph <siro@das-labor.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/common/block/cpu/cpulib.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/src/soc/intel/common/block/cpu/cpulib.c b/src/soc/intel/common/block/cpu/cpulib.c
index 9e95b0f9f4..a245824821 100644
--- a/src/soc/intel/common/block/cpu/cpulib.c
+++ b/src/soc/intel/common/block/cpu/cpulib.c
@@ -338,12 +338,10 @@ void mca_configure(void)
{
msr_t msr;
int i;
- int num_banks;
+ const unsigned int num_banks = mca_get_bank_count();
printk(BIOS_DEBUG, "Clearing out pending MCEs\n");
- msr = rdmsr(IA32_MCG_CAP);
- num_banks = msr.lo & 0xff;
msr.lo = msr.hi = 0;
for (i = 0; i < num_banks; i++) {