diff options
author | Stephen Glancy <sglancy@us.ibm.com> | 2016-09-14 11:59:44 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2016-09-15 13:32:52 -0400 |
commit | 553d3ea5adc5307d68fac9b5627abe222de90c68 (patch) | |
tree | 350a31adf6926dea9fc639346425db18df04f64c /src/import/chips/p9 | |
parent | 43bcffa950cebd21e4dee283b5455aedfe1df051 (diff) | |
download | talos-hostboot-553d3ea5adc5307d68fac9b5627abe222de90c68.tar.gz talos-hostboot-553d3ea5adc5307d68fac9b5627abe222de90c68.zip |
Fixed no DIMM configuration bug in xlate code
Change-Id: Icbeeff44c925b48d771bd5ac0ed869c0b077742d
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29681
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29683
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C b/src/import/chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C index 7e7aa77f9..c6ab549a5 100644 --- a/src/import/chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C +++ b/src/import/chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C @@ -38,6 +38,8 @@ #include <p9_mss_draminit_mc.H> #include <lib/fir/memdiags_fir.H> +#include <lib/utils/find.H> +#include <lib/utils/count_dimm.H> using fapi2::TARGET_TYPE_MCBIST; using fapi2::TARGET_TYPE_MCA; @@ -55,8 +57,15 @@ extern "C" FAPI_INF("Start draminit MC"); // No need to check to see if we have ports - this loop will just be skipped - for (const auto& p : i_target.getChildren<TARGET_TYPE_MCA>()) + for (const auto& p : mss::find_targets<fapi2::TARGET_TYPE_MCA>(i_target)) { + //skip this MCA if we have no DIMM's configured + if(mss::count_dimm(p) == 0) + { + FAPI_INF("No DIMM's configured on %s. Skipping this MCA.", mss::c_str(p)); + continue; + } + // Don't do this yet - leverage the sim inits for the moment #if 0 // All the scominit for this MCA |