summaryrefslogtreecommitdiffstats
path: root/src/usr/hwas/common
diff options
context:
space:
mode:
authorMatt Derksen <v2cibmd@us.ibm.com>2016-06-02 16:44:52 -0500
committerDaniel M. Crowell <dcrowell@us.ibm.com>2016-06-08 22:31:14 -0400
commit447b6ab0fec150916949e2333d88cf8f5c56791c (patch)
tree25dbdfd2fbc33f10c3ae291f6f0a169af24a73fe /src/usr/hwas/common
parentb5871645c924dfbf4fa71d7edcf71c6d1ceb1ab2 (diff)
downloadtalos-hostboot-447b6ab0fec150916949e2333d88cf8f5c56791c.tar.gz
talos-hostboot-447b6ab0fec150916949e2333d88cf8f5c56791c.zip
Include DIMMs in checkMinimumHardware
Change-Id: I0d9d819cccc25785b0c535b4e568a75e0d79924d RTC:149770 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25371 Tested-by: Jenkins Server Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/hwas/common')
-rw-r--r--src/usr/hwas/common/hwas.C9
1 files changed, 1 insertions, 8 deletions
diff --git a/src/usr/hwas/common/hwas.C b/src/usr/hwas/common/hwas.C
index 2b7f584c2..dfe446576 100644
--- a/src/usr/hwas/common/hwas.C
+++ b/src/usr/hwas/common/hwas.C
@@ -1464,9 +1464,6 @@ errlHndl_t checkMinimumHardware(const TARGETING::ConstTargetHandle_t i_nodeOrSys
HWAS_DBG( "checkMinimumHardware: %d functional dimms",
l_dimms.size());
-// @todo RTC:149770 Once ZZ supports DIMMs, add DIMMs back in min hdw check.
-// It works for stand alone.
-#if (0)
if (l_dimms.empty())
{
HWAS_ERR( "Insufficient hardware to continue IPL (func DIMM)");
@@ -1517,7 +1514,7 @@ errlHndl_t checkMinimumHardware(const TARGETING::ConstTargetHandle_t i_nodeOrSys
errlCommit(l_errl, HWAS_COMP_ID);
// errl is now NULL
} // if no dimms
-#endif
+
// There needs to be either functional MCS/MCAs (NIMBUS) or MCS/MBAs
// (CUMULUS). Check for MCAs first.
PredicateCTM l_mca(CLASS_UNIT, TYPE_MCA);
@@ -2079,9 +2076,6 @@ void presentByAssoc(TargetInfoVector& io_funcTargets,
case TYPE_MCA:
{
-// @todo RTC:149770 Once ZZ supports DIMMs, add DIMMs back in min hdw check.
-// It works for stand alone.
-#if (0)
// No Child DIMMs
// If next is not a DIMM sharing the same MCA, deconfig MCS
if ( (l_nextTargetInfo == NULL) ||
@@ -2095,7 +2089,6 @@ void presentByAssoc(TargetInfoVector& io_funcTargets,
// No Parent MCS
// If MCA doesn't share the same MCS as MCSIndex, deconfig MCA
else
-#endif // the new line can be removed
if ( (l_MCSIndex == __INT_MAX__) ||
!isSameSubPath(l_curTargetInfo, io_funcTargets[l_MCSIndex]))
{
OpenPOWER on IntegriCloud