From 553d3ea5adc5307d68fac9b5627abe222de90c68 Mon Sep 17 00:00:00 2001 From: Stephen Glancy Date: Wed, 14 Sep 2016 11:59:44 -0500 Subject: 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 Tested-by: Jenkins Server Reviewed-by: JACOB L. HARVEY Tested-by: Hostboot CI Reviewed-by: Brian R. Silver Reviewed-by: Richard J. Knight Reviewed-by: Jennifer A. Stofer Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29683 Reviewed-by: Hostboot Team Tested-by: FSP CI Jenkins Reviewed-by: Christian R. Geddes --- .../chips/p9/procedures/hwp/memory/p9_mss_draminit_mc.C | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) (limited to 'src') 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 #include +#include +#include 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()) + for (const auto& p : mss::find_targets(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 -- cgit v1.2.1