diff options
author | Dan Crowell <dcrowell@us.ibm.com> | 2018-02-08 13:37:25 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-02-12 10:11:47 -0500 |
commit | c80dab12a8551cc1d7b22c1c142f6bbf2bf3da8e (patch) | |
tree | 21d87cf4ba13866e4010314f2914d7b23047d401 /src/usr/initservice/istepdispatcher | |
parent | 27bddb0e962f0b13135cfcbaaa76e5120a99724f (diff) | |
download | talos-hostboot-c80dab12a8551cc1d7b22c1c142f6bbf2bf3da8e.tar.gz talos-hostboot-c80dab12a8551cc1d7b22c1c142f6bbf2bf3da8e.zip |
Switch polarity of return data for perst commands
HWSV and Hostboot had backwards interpretations of fail/success.
Changing Hostboot to return zero for success.
Change-Id: Ia21b2b7c78eb9c64f06a690a50740357441b23c1
CQ: SW415796
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53653
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>
Reviewed-by: Brian E. Bakke <bbakke@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/initservice/istepdispatcher')
-rw-r--r-- | src/usr/initservice/istepdispatcher/istepdispatcher.C | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/usr/initservice/istepdispatcher/istepdispatcher.C b/src/usr/initservice/istepdispatcher/istepdispatcher.C index a59a70399..2e27ac6bd 100644 --- a/src/usr/initservice/istepdispatcher/istepdispatcher.C +++ b/src/usr/initservice/istepdispatcher/istepdispatcher.C @@ -2195,8 +2195,8 @@ void IStepDispatcher::handlePerstMsg(msg_t * & io_pMsg) { TRACFCOMP(g_trac_initsvc, ENTER_MRK"IStepDispatcher::handlePerstMsg"); - // assume the HWP will succeed - io_pMsg->data[1] = true; + // assume the HWP will succeed (0=success) + io_pMsg->data[1] = 0; errlHndl_t l_errl = NULL; @@ -2215,7 +2215,7 @@ void IStepDispatcher::handlePerstMsg(msg_t * & io_pMsg) "PLID = 0x%x", l_errl->plid() ); - io_pMsg->data[1] = false; + io_pMsg->data[1] = ERRL_GETRC_SAFE(l_errl); errlCommit( l_errl, INITSVC_COMP_ID ); break; } @@ -2253,7 +2253,7 @@ void IStepDispatcher::handlePerstMsg(msg_t * & io_pMsg) l_errl->collectTrace("FAPI",256); errlCommit(l_errl, HWPF_COMP_ID); - io_pMsg->data[1] = false; + io_pMsg->data[1] = ERRL_GETRC_SAFE(l_errl); } } while(0); |