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

[Pratikkumar V Prajapati (Code Review)]

Pratikkumar V Prajapati has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32553 )

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

Patch Set 2:
> Patch Set 2:
> > Patch Set 2:
> >
> > (1 comment)
>
> On AML SOCs (for Nocturne), we have seen MCEs being dumped after booting to kernel. After debugging i found MCAs were not cleared before booting the kernel so i pushed those 2 patches and with that kernel was not reporting MCEs.
>
> (I am trying to find the crosbug but i am not able to access them, seems like i have some access issues)
>
> @Subrata, Have you checked on SKL/KBL/AML platforms that MCEs are not dumped if you dont clear them before booting?
>
> Clearing from cpu.c might be too early, i think. in my understanding, Clearing MCEs at that point is needed only if we need to enable SGX, as when we reload microcode for SGX we need have MCA clean.

https://partnerissuetracker.corp.google.com/issues/79783714 is the tracker.