diff options
author | Dan Crowell <dcrowell@us.ibm.com> | 2018-02-12 10:44:40 -0600 |
---|---|---|
committer | William G. Hoffa <wghoffa@us.ibm.com> | 2018-02-14 14:29:59 -0500 |
commit | 328e7f781d6d90858c52b24da1289e4f9f455e3e (patch) | |
tree | 1c3cf3e264ea38ad5eeb02693b885ca7dff54b1b /src/usr/scom | |
parent | b6f70467b7203502ac1cb92eb921620086c17775 (diff) | |
download | talos-hostboot-328e7f781d6d90858c52b24da1289e4f9f455e3e.tar.gz talos-hostboot-328e7f781d6d90858c52b24da1289e4f9f455e3e.zip |
Remove extraneous error messages for multicast scoms
Fixed two bugs in the code that was added to avoid extra
error logging in the multicast workaround code:
- Fixed bad double-negative check for MC sync mode
- Fixed incorrect EC chiplet range
Change-Id: I8a6747ee66206e0df519ad9b58cdc0a37a4aa7d6
CQ: SW417057
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53857
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/usr/scom')
-rw-r--r-- | src/usr/scom/scom.C | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/usr/scom/scom.C b/src/usr/scom/scom.C index 5c35dfdd2..ef9bdf8e6 100644 --- a/src/usr/scom/scom.C +++ b/src/usr/scom/scom.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2011,2017 */ +/* Contributors Listed Below - COPYRIGHT 2011,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -1219,7 +1219,7 @@ errlHndl_t doMulticastWorkaround( DeviceFW::OperationType i_opType, constexpr uint64_t P9_FIRST_EQ = 0x10; constexpr uint64_t P9_LAST_EQ = 0x1F; constexpr uint64_t P9_FIRST_EC = 0x20; - constexpr uint64_t P9_LAST_EC = 0x2F; + constexpr uint64_t P9_LAST_EC = 0x3F; #endif // Skip calls to the SENTINEL since we don't have the @@ -1292,7 +1292,7 @@ errlHndl_t doMulticastWorkaround( DeviceFW::OperationType i_opType, // because we don't start clocks until later on in that case auto l_syncMode = i_target->getAttr<TARGETING::ATTR_MC_SYNC_MODE>(); - if( l_syncMode ) + if( l_syncMode == TARGETING::MC_SYNC_MODE_NOT_IN_SYNC ) { continue; } |