Furquan Shaikh 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 1:
(1 comment)
https://review.coreboot.org/#/c/32553/1/src/soc/intel/skylake/finalize.c File src/soc/intel/skylake/finalize.c:
https://review.coreboot.org/#/c/32553/1/src/soc/intel/skylake/finalize.c@a14... PS1, Line 146: Do we know why this change was added here? It seems like we should revert both these CLs:
https://review.coreboot.org/c/coreboot/+/26392/ https://review.coreboot.org/c/coreboot/+/26391/