From 328e7f781d6d90858c52b24da1289e4f9f455e3e Mon Sep 17 00:00:00 2001 From: Dan Crowell Date: Mon, 12 Feb 2018 10:44:40 -0600 Subject: 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 Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Reviewed-by: Corey V. Swenson Tested-by: Jenkins OP HW Reviewed-by: Prachi Gupta Reviewed-by: Christian R. Geddes Reviewed-by: William G. Hoffa --- src/usr/scom/scom.C | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/usr/scom') 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(); - if( l_syncMode ) + if( l_syncMode == TARGETING::MC_SYNC_MODE_NOT_IN_SYNC ) { continue; } -- cgit v1.2.1