diff options
| author | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2016-04-06 13:24:21 -0500 |
|---|---|---|
| committer | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2016-04-08 16:43:35 +0200 |
| commit | 49e917bffdb4d43caa418b9712235d174272a664 (patch) | |
| tree | e7e9076fcfdc63efca80129b5ce5495a4632b4f0 /src/northbridge/amd/amdfam10 | |
| parent | 056e454da343b9e7bf0d512b521acc8f1697284b (diff) | |
nb/amd/amdfam10: Only flag machine check exception if valid bit is set
Change-Id: I42d901ae9445943a863fb3ba9bda5a915f255e02
Signed-off-by: Timothy Pearson <tpearson@raptorengineeringinc.com>
Reviewed-on: https://review.coreboot.org/14264
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/northbridge/amd/amdfam10')
| -rw-r--r-- | src/northbridge/amd/amdfam10/misc_control.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/northbridge/amd/amdfam10/misc_control.c b/src/northbridge/amd/amdfam10/misc_control.c index de9f34233f..eee5c465fb 100644 --- a/src/northbridge/amd/amdfam10/misc_control.c +++ b/src/northbridge/amd/amdfam10/misc_control.c @@ -178,7 +178,7 @@ static void misc_control_init(struct device *dev) if (dimm_present) { uint32_t mc4_status_high = pci_read_config32(dev, 0x4c); uint32_t mc4_status_low = pci_read_config32(dev, 0x48); - if (mc4_status_high != 0) { + if ((mc4_status_high & (0x1 << 31)) && (mc4_status_high != 0xffffffff)) { printk(BIOS_WARNING, "\nWARNING: MC4 Machine Check Exception detected on node %d!\n" "Signature: %08x%08x\n", node, mc4_status_high, mc4_status_low); } |