diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2018-02-15 10:41:31 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-02-16 20:22:29 -0500 |
commit | 35165b3476254ee9a6f95786b93e6425ed6482a2 (patch) | |
tree | b660147a42574839bc32cdf8dcd4f99249a991a4 /src/usr/isteps/istep21 | |
parent | 2d6ba11ae2fc6dd2aca923be277cb94ef7e95c92 (diff) | |
download | talos-hostboot-35165b3476254ee9a6f95786b93e6425ed6482a2.tar.gz talos-hostboot-35165b3476254ee9a6f95786b93e6425ed6482a2.zip |
Remove all explicit calls to special_wakeup from hostboot code
The only time that special wakeup should be called on the hostboot
platform is when it is called by other HWPs. PM_RESET will only
enable special wakeup if the SKIP_WAKEUP attribute is set to 0
Change-Id: Ib9939eae857fa5113a876a717f8169e150b78318
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54148
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: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/isteps/istep21')
-rw-r--r-- | src/usr/isteps/istep21/call_host_start_payload.C | 61 |
1 files changed, 0 insertions, 61 deletions
diff --git a/src/usr/isteps/istep21/call_host_start_payload.C b/src/usr/isteps/istep21/call_host_start_payload.C index 7024fd98c..eba79e0f5 100644 --- a/src/usr/isteps/istep21/call_host_start_payload.C +++ b/src/usr/isteps/istep21/call_host_start_payload.C @@ -49,7 +49,6 @@ #include <isteps/hwpf_reasoncodes.H> #include <fapi2/target.H> #include <fapi2/plat_hwp_invoker.H> -#include <p9_cpu_special_wakeup.H> #include <p9n2_quad_scom_addresses_fld.H> #include <ipmi/ipmiwatchdog.H> #include <config.h> @@ -67,7 +66,6 @@ using namespace ERRORLOG; using namespace ISTEP; using namespace ISTEP_ERROR; using namespace TARGETING; -using namespace p9specialWakeup; namespace ISTEP_21 { @@ -92,13 +90,6 @@ errlHndl_t callShutdown ( uint64_t i_hbInstance, bool i_masterIntance ); */ errlHndl_t broadcastShutdown ( uint64_t i_hbInstance ); -/** - * @brief This function disables the special wakeup that allows scom - * operations on napped cores - * - * @return errlHndl_t error handle - */ -errlHndl_t disableSpecialWakeup(); /** * @brief Re-enables the local core checkstop function @@ -366,12 +357,6 @@ errlHndl_t callShutdown ( uint64_t i_masterInstance, do { - err = disableSpecialWakeup(); - if(err) - { - break; - } - // Tell SBE to Close All Unsecure Memory Regions err = SBEIO::closeAllUnsecureMemRegions(); if (err) @@ -654,52 +639,6 @@ errlHndl_t broadcastShutdown ( uint64_t i_hbInstance ) } -errlHndl_t disableSpecialWakeup() -{ - errlHndl_t l_errl = nullptr; - - TargetHandleList l_cores; - getAllChiplets(l_cores, TYPE_CORE); - uint32_t l_numCores = 0; - - for( const auto & l_core: l_cores ) - { - l_numCores += 1; - - const fapi2::Target<fapi2::TARGET_TYPE_CORE> - l_fapi2_coreTarget(l_core); - - TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "Running p9_cpu_special_wakeup_core (DISABLE) target HUID %.8X", - TARGETING::get_huid(l_core)); - - // Disable special wakeup on core - FAPI_INVOKE_HWP( l_errl, - p9_cpu_special_wakeup_core, - l_fapi2_coreTarget, - SPCWKUP_DISABLE, - HOST); - if( l_errl ) - { - TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, - "ERROR : disable p9_cpu_special_wakeup_core, " - "PLID=0x%x", l_errl->plid() ); - - ErrlUserDetailsTarget(l_core).addToLog( l_errl ); - - break; - } - else - { - TRACDCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "SUCCESS: disable p9_cpu_special_wakeup_core"); - } - } - - return l_errl; -} - - /** * @brief Re-enables the local core checkstop function */ |