diff options
| author | Christian Geddes <crgeddes@us.ibm.com> | 2018-02-26 13:59:03 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-03-07 14:46:23 -0500 |
| commit | 74156401d2fb37e05e5ae0fc50906e321756a354 (patch) | |
| tree | ec6ff3b6a02946c58e2583901875f972c3c79ec9 /src/usr/isteps | |
| parent | c142eb8503804c432410459012d4f6e62cd00ab0 (diff) | |
| download | talos-hostboot-74156401d2fb37e05e5ae0fc50906e321756a354.tar.gz talos-hostboot-74156401d2fb37e05e5ae0fc50906e321756a354.zip | |
Don't include duplicate connections when looking up xbus mapping
Previously in P8 when running the xbus scominit hwp we needed to
include duplicate connections IE xbus0->xbus1 and xbus1->xbus0 when
we look up what to run the xbus scominit hwp on. In p9 the HWP will
initialize both ends of the xbus connection when we pass both ends
into the xbus scominit hardware procedure
Change-Id: I5a0a356e2086b15ee5d25cf350c9a0559ec2084b
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54726
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>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/isteps')
| -rw-r--r-- | src/usr/isteps/istep08/call_proc_xbus_scominit.C | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/usr/isteps/istep08/call_proc_xbus_scominit.C b/src/usr/isteps/istep08/call_proc_xbus_scominit.C index faa203d4a..b446f7b06 100644 --- a/src/usr/isteps/istep08/call_proc_xbus_scominit.C +++ b/src/usr/isteps/istep08/call_proc_xbus_scominit.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2016 */ +/* Contributors Listed Below - COPYRIGHT 2015,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -89,14 +89,14 @@ void* call_proc_xbus_scominit( void *io_pArgs ) { EDI_EI_INITIALIZATION::TargetPairs_t l_XbusConnections; // Note: - // i_noDuplicate parameter must be set to false because - // two separate calls would be needed: + // i_noDuplicate parameter must be set to true because + // one call to p9_io_xbus_scominit will handle both // X0 <--> X1 // X1 <--> X0 - // only the first target is used to issue SCOMs + // both the xbus and the connected target are used to issue SCOMs l_err = EDI_EI_INITIALIZATION::PbusLinkSvc::getTheInstance().getPbusConnections( - l_XbusConnections, TYPE_XBUS, false); + l_XbusConnections, TYPE_XBUS, true); if (l_err) { TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, |
