Skip to content

Commit

Permalink
Modify INTRP Initialization Order to route LSI interrupts over PSHIHB
Browse files Browse the repository at this point in the history
  - 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>
  • Loading branch information
wghoffa authored and dcrowell77 committed Nov 8, 2019
1 parent c3689a6 commit c283187
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/usr/intr/intrrp.C
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand All @@ -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
Expand Down

0 comments on commit c283187

Please sign in to comment.