diff options
author | Bill Hoffa <wghoffa@us.ibm.com> | 2019-11-07 15:47:00 -0600 |
---|---|---|
committer | Daniel M Crowell <dcrowell@us.ibm.com> | 2019-11-08 10:58:07 -0600 |
commit | c283187cd8301a0c00259908dd950d31b2f82f67 (patch) | |
tree | 8e4ab2e63088b52c27f3a6cb8c7943ea4fd19454 | |
parent | c3689a69137b76f117ac9524a8f7ec189e133b04 (diff) | |
download | talos-hostboot-c283187cd8301a0c00259908dd950d31b2f82f67.tar.gz talos-hostboot-c283187cd8301a0c00259908dd950d31b2f82f67.zip |
Modify INTRP Initialization Order to route LSI interrupts over PSHIHB
- Currently the code sets up routing over PSIHIB and then a second
function call overwrites that setting which forces things back
on the local wire
Change-Id: I7d3b44d48c66f994e8a8758eb75568038492fdaf
CQ: SW464174
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86669
Tested-by: Jenkins Server <pfd-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>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
-rw-r--r-- | src/usr/intr/intrrp.C | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/usr/intr/intrrp.C b/src/usr/intr/intrrp.C index d198610af..afc8cefc7 100644 --- a/src/usr/intr/intrrp.C +++ b/src/usr/intr/intrrp.C @@ -547,13 +547,6 @@ errlHndl_t IntrRp::_init() break; } - //Route LSI interrupt events over PSIHB instead of local wire - // This is a HW Bug Workaround for slaves using the PSIHB and - // the master using the local wire - routeLSIInterrupts(l_procIntrHdlr); - - enableLsiInterrupts(); - TRACFCOMP(g_trac_intr, "IntrRp::_init() Enabling PSIHB Interrupts"); //Enable PSIHB Interrupts l_err = enableInterrupts(l_procIntrHdlr); @@ -562,6 +555,13 @@ errlHndl_t IntrRp::_init() TRACFCOMP(g_trac_intr, "IntrRp::_init() Error enabling Interrupts"); break; } + + //Route LSI interrupt events over PSIHB instead of local wire + // This is a HW Bug Workaround for slaves using the PSIHB and + // the master using the local wire + routeLSIInterrupts(l_procIntrHdlr); + + enableLsiInterrupts(); } // Build up list of unregistered LSI sources, at this point no sourced |