summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBill Hoffa <wghoffa@us.ibm.com>2019-11-07 15:47:00 -0600
committerDaniel M Crowell <dcrowell@us.ibm.com>2019-11-08 10:58:07 -0600
commitc283187cd8301a0c00259908dd950d31b2f82f67 (patch)
tree8e4ab2e63088b52c27f3a6cb8c7943ea4fd19454
parentc3689a69137b76f117ac9524a8f7ec189e133b04 (diff)
downloadtalos-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.C14
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
OpenPOWER on IntegriCloud