summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/sbefw/sbecmdiplcontrol.C17
-rw-r--r--src/sbefw/sbecmdmpipl.C14
2 files changed, 13 insertions, 18 deletions
diff --git a/src/sbefw/sbecmdiplcontrol.C b/src/sbefw/sbecmdiplcontrol.C
index 2a83b972..2acb5cad 100644
--- a/src/sbefw/sbecmdiplcontrol.C
+++ b/src/sbefw/sbecmdiplcontrol.C
@@ -182,7 +182,6 @@ ReturnCode istepWithExL3Flush( sbeIstepHwp_t i_hwp );
ReturnCode istepNoOpStartMpipl( sbeIstepHwp_t i_hwp );
ReturnCode istepWithProcSequenceDrtm( sbeIstepHwp_t i_hwp );
ReturnCode istepMpiplSetFunctionalState( sbeIstepHwp_t i_hwp );
-ReturnCode istepMpiplSetMPIPLMode( sbeIstepHwp_t i_hwp );
ReturnCode istepMpiplQuadPoweroff( sbeIstepHwp_t i_hwp );
ReturnCode istepStopClockMpipl( sbeIstepHwp_t i_hwp );
@@ -225,6 +224,7 @@ static istepMap_t g_istepMpiplStartPtrTbl[MPIPL_START_MAX_SUBSTEPS] =
{
#ifdef SEEPROM_IMAGE
// Place holder for StartMpipl Chip-op, State Change
+ // Set MPIPL mode in Sratch Reg 3
{ &istepNoOpStartMpipl, NULL },
// Find all the child cores within proc and call set block intr
{ &istepWithCoreSetBlock, { .coreBlockIntrHwp = &p9_block_wakeup_intr }},
@@ -252,8 +252,8 @@ static istepMap_t g_istepMpiplContinuePtrTbl[MPIPL_CONTINUE_MAX_SUBSTEPS] =
{ &istepMpiplSetFunctionalState, NULL},
// p9_quad_power_off
{ istepMpiplQuadPoweroff, { .eqHwp = &p9_quad_power_off} },
- // Set MPIPL mode in Sratch Reg 3
- { &istepMpiplSetMPIPLMode, NULL},
+ // No-op
+ { &istepNoOp, NULL},
#endif
};
@@ -1097,6 +1097,7 @@ ReturnCode istepNoOpStartMpipl( sbeIstepHwp_t i_hwp)
SBE_ENTER(SBE_FUNC);
(void)SbeRegAccess::theSbeRegAccess().stateTransition(
SBE_ENTER_MPIPL_EVENT);
+ // Set MPIPL mode bit in Scratch Reg 3
(void)SbeRegAccess::theSbeRegAccess().setMpIplMode(true);
SBE_EXIT(SBE_FUNC);
@@ -1159,16 +1160,6 @@ ReturnCode istepWithProcQuiesceLQASet( sbeIstepHwp_t i_hwp )
}
//----------------------------------------------------------------------------
-ReturnCode istepMpiplSetMPIPLMode( sbeIstepHwp_t i_hwp)
-{
- #define SBE_FUNC "istepMpiplSetMPIPLMode"
- // Set MPIPL mode bit in Scratch Reg 3
- (void)SbeRegAccess::theSbeRegAccess().setMpIplMode(true);
- return FAPI2_RC_SUCCESS;
- #undef SBE_FUNC
-}
-
-//----------------------------------------------------------------------------
ReturnCode istepMpiplSetFunctionalState( sbeIstepHwp_t i_hwp )
{
#define SBE_FUNC "istepMpiplSetFunctionalState"
diff --git a/src/sbefw/sbecmdmpipl.C b/src/sbefw/sbecmdmpipl.C
index b947603f..0b3330af 100644
--- a/src/sbefw/sbecmdmpipl.C
+++ b/src/sbefw/sbecmdmpipl.C
@@ -107,7 +107,6 @@ uint32_t sbeEnterMpipl(uint8_t *i_pArg)
#undef SBE_FUNC
}
-// TODO - RTC 133367
///////////////////////////////////////////////////////////////////////
// @brief sbeContinueMpipl Sbe Continue MPIPL function
//
@@ -119,7 +118,6 @@ uint32_t sbeContinueMpipl(uint8_t *i_pArg)
SBE_ENTER(SBE_FUNC);
uint32_t l_rc = SBE_SEC_OPERATION_SUCCESSFUL;
uint32_t len = 0;
- constexpr uint32_t ISTEP_SUBSTEP_22 = 22;
ReturnCode l_fapiRc = FAPI2_RC_SUCCESS;
sbeResponseFfdc_t l_ffdc;
@@ -132,16 +130,23 @@ uint32_t sbeContinueMpipl(uint8_t *i_pArg)
l_rc = sbeUpFifoDeq_mult (len, NULL);
CHECK_SBE_RC_AND_BREAK_IF_NOT_SUCCESS(l_rc);
+ sbeRole l_sbeRole = SbeRegAccess::theSbeRegAccess().isSbeSlave() ?
+ SBE_ROLE_SLAVE : SBE_ROLE_MASTER;
// Run isteps
- const uint8_t isteps[][4] = {
+ const uint8_t isteps[][3] = {
// Major Num, Minor Start, Minor End
{SBE_ISTEP_MPIPL_CONTINUE, ISTEP_MINOR_START, MPIPL_CONTINUE_MAX_SUBSTEPS},
- {SBE_ISTEP3, ISTEP_SUBSTEP_22, ISTEP_SUBSTEP_22},
{SBE_ISTEP4, ISTEP_MINOR_START, ISTEP4_MAX_SUBSTEPS},
{SBE_ISTEP5, ISTEP_MINOR_START, ISTEP5_MAX_SUBSTEPS}};
// Loop through isteps
for( auto istep : isteps)
{
+ // This is required here to skip the major istep 4/5 in slave
+ if((SBE_ROLE_SLAVE == l_sbeRole) &&
+ (istep[0] == SBE_ISTEP4 || istep[0] == SBE_ISTEP5))
+ {
+ continue;
+ }
for(uint8_t l_minor = istep[1]; l_minor <= istep[2]; l_minor++)
{
l_fapiRc = sbeExecuteIstep(istep[0], l_minor);
@@ -178,7 +183,6 @@ uint32_t sbeContinueMpipl(uint8_t *i_pArg)
// @brief sbeStopClocks Sbe Stop Clocks function
//
// @return RC from the underlying FIFO utility
-// RTC-161679 : Stop Clocks Chip-op to handle Proc Chip Target
///////////////////////////////////////////////////////////////////////
#define SBE_IS_EX0(chipletId) \
(((chipletId - EX_TARGET_OFFSET) & 0x0002) >> 1)
OpenPOWER on IntegriCloud