diff options
author | Dan Crowell <dcrowell@us.ibm.com> | 2019-12-19 12:15:05 -0600 |
---|---|---|
committer | William G Hoffa <wghoffa@us.ibm.com> | 2020-01-10 10:11:23 -0600 |
commit | 6ceeb473dc9e84d3af17a940e6b37fac19036233 (patch) | |
tree | eca4a5bdf2110e35e785fad4badd322259c42f33 | |
parent | 7b91b9596065935f14d87d2eef2dd980039823e2 (diff) | |
download | talos-hostboot-6ceeb473dc9e84d3af17a940e6b37fac19036233.tar.gz talos-hostboot-6ceeb473dc9e84d3af17a940e6b37fac19036233.zip |
Move p9a_omi_setup_bars from 12.4 to 12.3
The code wasn't matching the IPL Spec. There is really no
change to the order things ran, but now it matches the doc.
Change-Id: I35e3186d30bc946bf882e4657900a200ee1aa1b9
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88949
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: Corey V Swenson <cswenson@us.ibm.com>
Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
-rw-r--r-- | src/usr/isteps/istep12/call_cen_dmi_scominit.C | 48 | ||||
-rw-r--r-- | src/usr/isteps/istep12/call_proc_dmi_scominit.C | 53 |
2 files changed, 52 insertions, 49 deletions
diff --git a/src/usr/isteps/istep12/call_cen_dmi_scominit.C b/src/usr/isteps/istep12/call_cen_dmi_scominit.C index 8e7a9feef..868f1da2a 100644 --- a/src/usr/isteps/istep12/call_cen_dmi_scominit.C +++ b/src/usr/isteps/istep12/call_cen_dmi_scominit.C @@ -46,10 +46,6 @@ // HWP #include <p9_io_cen_scominit.H> -#ifdef CONFIG_AXONE -#include <p9a_omi_setup_bars.H> -#endif - using namespace ISTEP; using namespace ISTEP_ERROR; using namespace ERRORLOG; @@ -107,50 +103,6 @@ void* call_cen_dmi_scominit (void *io_pArgs) } - #ifdef CONFIG_AXONE - TARGETING::TargetHandleList l_procTargetList; - getAllChips(l_procTargetList, TYPE_PROC); - - TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, "call_cen_dmi_scominit: %d procs found", - l_procTargetList.size()); - - for (const auto & l_proc_target : l_procTargetList) - { - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "p9a_omi_setup_bars HWP target HUID %.8x", - TARGETING::get_huid(l_proc_target)); - - // call the HWP with each target - fapi2::Target <fapi2::TARGET_TYPE_PROC_CHIP> l_fapi_proc_target - (l_proc_target); - - FAPI_INVOKE_HWP(l_err, p9a_omi_setup_bars, l_fapi_proc_target); - - // process return code. - if ( l_err ) - { - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "ERROR 0x%.8X: p9a_omi_setup_bars HWP on target HUID %.8x", - l_err->reasonCode(), TARGETING::get_huid(l_proc_target) ); - - // capture the target data in the elog - ErrlUserDetailsTarget(l_proc_target).addToLog( l_err ); - - // Create IStep error log and cross reference to error that occurred - l_StepError.addErrorDetails( l_err ); - - // Commit Error - errlCommit( l_err, ISTEP_COMP_ID ); - } - else - { - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, - "SUCCESS : p9a_omi_setup_bars HWP"); - } - - } - #endif // CONFIG_AXONE - TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, "call_cen_dmi_scominit exit" ); // end task, returning any errorlogs to IStepDisp diff --git a/src/usr/isteps/istep12/call_proc_dmi_scominit.C b/src/usr/isteps/istep12/call_proc_dmi_scominit.C index b875322b4..e0c452206 100644 --- a/src/usr/isteps/istep12/call_proc_dmi_scominit.C +++ b/src/usr/isteps/istep12/call_proc_dmi_scominit.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2018 */ +/* Contributors Listed Below - COPYRIGHT 2015,2019 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -46,6 +46,11 @@ // HWP #include <p9_io_dmi_scominit.H> +#ifdef CONFIG_AXONE +#include <p9a_omi_setup_bars.H> +#endif + + #include <mmio/mmio.H> using namespace ISTEP; @@ -63,6 +68,7 @@ void* call_proc_dmi_scominit (void *io_pArgs) TRACDCOMP( ISTEPS_TRACE::g_trac_isteps_trace, "call_proc_dmi_scominit entry" ); +#ifndef CONFIG_AXONE TARGETING::TargetHandleList l_dmiTargetList; getAllChiplets(l_dmiTargetList, TYPE_DMI); @@ -105,6 +111,51 @@ void* call_proc_dmi_scominit (void *io_pArgs) } +#else // CONFIG_AXONE + + TARGETING::TargetHandleList l_procTargetList; + getAllChips(l_procTargetList, TYPE_PROC); + + TRACFCOMP(ISTEPS_TRACE::g_trac_isteps_trace, "call_proc_dmi_scominit: %d procs found", + l_procTargetList.size()); + + for (const auto & l_proc_target : l_procTargetList) + { + TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, + "p9a_omi_setup_bars HWP target HUID %.8x", + TARGETING::get_huid(l_proc_target)); + + // call the HWP with each target + fapi2::Target <fapi2::TARGET_TYPE_PROC_CHIP> l_fapi_proc_target + (l_proc_target); + + FAPI_INVOKE_HWP(l_err, p9a_omi_setup_bars, l_fapi_proc_target); + + // process return code. + if ( l_err ) + { + TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, + "ERROR 0x%.8X: p9a_omi_setup_bars HWP on target HUID %.8x", + l_err->reasonCode(), TARGETING::get_huid(l_proc_target) ); + + // capture the target data in the elog + ErrlUserDetailsTarget(l_proc_target).addToLog( l_err ); + + // Create IStep error log and cross reference to error that occurred + l_StepError.addErrorDetails( l_err ); + + // Commit Error + errlCommit( l_err, ISTEP_COMP_ID ); + } + else + { + TRACFCOMP( ISTEPS_TRACE::g_trac_isteps_trace, + "SUCCESS : p9a_omi_setup_bars HWP"); + } + + } +#endif // CONFIG_AXONE + // map OCMBs into Hostboot memory l_err = MMIO::mmioSetup(); if ( l_err ) |