diff options
author | Caleb Palmer <cnpalmer@us.ibm.com> | 2018-01-25 14:52:08 -0600 |
---|---|---|
committer | Zane C. Shelley <zshelle@us.ibm.com> | 2018-02-07 13:41:07 -0500 |
commit | 4a33c3221672202c2eba7c8b781c70dcb2b247d7 (patch) | |
tree | a9ac4063c97c2c1637b1598bf6f058d63c9b69cc | |
parent | 0b8f08a0e4f7b3a496a5a459b7c82a8f9efebedc (diff) | |
download | talos-hostboot-4a33c3221672202c2eba7c8b781c70dcb2b247d7.tar.gz talos-hostboot-4a33c3221672202c2eba7c8b781c70dcb2b247d7.zip |
PRD: Restore repairs fix bad dimm mask check
Change-Id: Icce73a781910d01cef9f6dd156a212d860cabec7
CQ: SW411781
Backport: release-op910
Backport: release-fips910
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52662
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com>
Reviewed-by: Benjamin J Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Zane C Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53492
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
-rw-r--r-- | src/usr/diag/prdf/plat/mem/prdfRestoreDramRepairs.C | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/usr/diag/prdf/plat/mem/prdfRestoreDramRepairs.C b/src/usr/diag/prdf/plat/mem/prdfRestoreDramRepairs.C index 5e7c6bffb..fbac10c7a 100644 --- a/src/usr/diag/prdf/plat/mem/prdfRestoreDramRepairs.C +++ b/src/usr/diag/prdf/plat/mem/prdfRestoreDramRepairs.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2012,2017 */ +/* Contributors Listed Below - COPYRIGHT 2012,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -362,8 +362,10 @@ bool processBadDimms<TYPE_MCA>( TargetHandle_t i_trgt, uint8_t i_badDimmMask ) uint8_t dimm = getTargetPosition( i ) % MAX_DIMM_PER_PORT; // i_badDimmMask is defined as a 2-bit mask where a bit set means that - // DIMM had more bad bits than could be repaired. - uint8_t mask = 0x2 >> dimm; + // DIMM had more bad bits than could be repaired. Note: the value is + // actually a 4-bit field for use with Centaur, but we only use the + // first 2 bits of that field here. + uint8_t mask = 0x80 >> dimm; if ( 0 != (i_badDimmMask & mask) ) { |