Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson. Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/56280 )
Change subject: soc/amd/cezanne/mca: add empty mca_check_all_banks function ......................................................................
soc/amd/cezanne/mca: add empty mca_check_all_banks function
This will allow factoring out and moving check_mca() to soc/amd/common.
Change-Id: I92c7657baef17c248a5aef1eda268e9647502837 Signed-off-by: Felix Held felix-coreboot@felixheld.de --- M src/soc/amd/cezanne/mca.c 1 file changed, 6 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/56280/1
diff --git a/src/soc/amd/cezanne/mca.c b/src/soc/amd/cezanne/mca.c index 24a7788..a8d9702 100644 --- a/src/soc/amd/cezanne/mca.c +++ b/src/soc/amd/cezanne/mca.c @@ -3,11 +3,15 @@ #include <amdblocks/mca.h> #include <cpu/x86/msr.h>
+static void mca_check_all_banks(void) +{ + /* TODO: Implement MCAX register checking and BERT table generation. */ +} + /* Check the Machine Check Architecture Extension registers */ void check_mca(void) { - /* TODO: Implement MCAX register checking and BERT table generation. */ - + mca_check_all_banks(); /* mca_clear_status uses the MCA registers and not the MCAX ones. Since they are aliases, we can use either set of registers. */ mca_clear_status();