[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/skylake: Remove redundant mca_configure() in ramstage

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32553 )

Change subject: soc/intel/skylake: Remove redundant mca_configure() in ramstage
......................................................................

soc/intel/skylake: Remove redundant mca_configure() in ramstage

This patch removes redundant mca_configure() function call
from ramstage to clear machine check exception. First time it's
getting called from soc_core_init() function inside cpu.c file.

TEST=Build and boot SKL/KBL/AML platform without any machine-check
exception.

Change-Id: I7e54fd07816c6317588ab6db06365937c4300ccd
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32553
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
Reviewed-by: Nick Vaccaro 
---
M src/soc/intel/skylake/finalize.c
1 file changed, 0 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, approved
  Nick Vaccaro: Looks good to me, but someone else must approve

diff --git a/src/soc/intel/skylake/finalize.c b/src/soc/intel/skylake/finalize.c
index 5d7e1e0..34738f2 100644
--- a/src/soc/intel/skylake/finalize.c
+++ b/src/soc/intel/skylake/finalize.c
@@ -20,7 +20,6 @@
 #include <bootstate.h>
 #include <console/console.h>
 #include <console/post_codes.h>
-#include <cpu/x86/mp.h>
 #include <cpu/x86/smm.h>
 #include <device/pci.h>
 #include <intelblocks/cpulib.h>
@@ -142,9 +141,6 @@

pch_finalize_script(dev);

-	printk(BIOS_DEBUG, "Clearing MCA.\n");
-	mp_run_on_all_cpus(mca_configure, NULL, 17 * USECS_PER_SEC);
-
    soc_lockdown(dev);

printk(BIOS_DEBUG, "Finalizing SMM.\n");