diff options
author | Dzuy Nguyen <dzuy@us.ibm.com> | 2016-09-27 14:27:21 -0500 |
---|---|---|
committer | Matthew A. Ploetz <maploetz@us.ibm.com> | 2016-10-10 15:22:31 -0400 |
commit | 6ec92d726613ef8a744333add060d942200e1475 (patch) | |
tree | c30e877ad90aa80720c708e8c50a7b5e36b205af /src/usr/isteps/istep08 | |
parent | ea0b739975be946d8221391aada95c941898da8e (diff) | |
download | talos-hostboot-6ec92d726613ef8a744333add060d942200e1475.tar.gz talos-hostboot-6ec92d726613ef8a744333add060d942200e1475.zip |
Add p9_getecid into call_proc_check_slave_sbe_seeprom_complete.C
Change-Id: I2b000fe21dbc3feb040959501be344bc82ee2ba9
RTC:149518
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30378
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
Diffstat (limited to 'src/usr/isteps/istep08')
-rw-r--r-- | src/usr/isteps/istep08/call_proc_check_slave_sbe_seeprom_complete.C | 24 | ||||
-rw-r--r-- | src/usr/isteps/istep08/makefile | 1 |
2 files changed, 12 insertions, 13 deletions
diff --git a/src/usr/isteps/istep08/call_proc_check_slave_sbe_seeprom_complete.C b/src/usr/isteps/istep08/call_proc_check_slave_sbe_seeprom_complete.C index 4c820cd5d..efac0b2a4 100644 --- a/src/usr/isteps/istep08/call_proc_check_slave_sbe_seeprom_complete.C +++ b/src/usr/isteps/istep08/call_proc_check_slave_sbe_seeprom_complete.C @@ -60,6 +60,7 @@ #include <p9_extract_sbe_rc.H> #include <p9_get_sbe_msg_register.H> +#include <p9_getecid.H> using namespace ISTEP; using namespace ISTEP_ERROR; @@ -261,27 +262,25 @@ void* call_proc_check_slave_sbe_seeprom_complete( void *io_pArgs ) const_cast<TARGETING::Target*> (l_cpu_target)); TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "Running proc_getecid HWP" + "Running p9_getecid HWP" " on processor target %.8X", TARGETING::get_huid(l_cpu_target) ); -// TODO-RTC:149518 -/* - // proc_getecid should set the fuse string to 112 bits long. - ecmdDataBufferBase l_fuseString; + // p9_getecid should set the fuse string to 112 bytes long. + fapi2::variable_buffer l_fuseString(112); // Invoke the HWP FAPI_INVOKE_HWP(l_errl, - proc_getecid, + p9_getecid, l_fapi2ProcTarget, l_fuseString ); -*/ + if (l_errl) { if (l_cpu_target->getAttr<ATTR_HWAS_STATE>().functional) { TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "ERROR : proc_getecid", + "ERROR : p9_getecid", " failed, returning errorlog" ); // capture the target data in the elog @@ -297,7 +296,7 @@ void* call_proc_check_slave_sbe_seeprom_complete( void *io_pArgs ) else // Not functional, proc deconfigured, don't report error { TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "ERROR : proc_getecid", + "ERROR : p9_getecid", " failed, proc target deconfigured" ); delete l_errl; l_errl = NULL; @@ -305,10 +304,9 @@ void* call_proc_check_slave_sbe_seeprom_complete( void *io_pArgs ) } else { - // TODO-RTC:149518 - //TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - // "SUCCESS : proc_getecid", - // " completed ok"); + TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, + "SUCCESS : proc_getecid", + " completed ok"); } } // end of going through all processors diff --git a/src/usr/isteps/istep08/makefile b/src/usr/isteps/istep08/makefile index 061b22dfd..a57494504 100644 --- a/src/usr/isteps/istep08/makefile +++ b/src/usr/isteps/istep08/makefile @@ -82,6 +82,7 @@ include ${PROCEDURES_PATH}/hwp/perv/p9_start_cbs.mk include ${PROCEDURES_PATH}/hwp/perv/p9_check_slave_sbe_seeprom_complete.mk include ${PROCEDURES_PATH}/hwp/perv/p9_extract_sbe_rc.mk include ${PROCEDURES_PATH}/hwp/sbe/p9_get_sbe_msg_register.mk +include ${PROCEDURES_PATH}/hwp/perv/p9_getecid.mk # proc_cen_ref_clk_enable # Cummulus only -- p9_cen_ref_clk_enable.mk not defined yet |