summaryrefslogtreecommitdiffstats
path: root/src/import/chips/p9/procedures/hwp/memory
diff options
context:
space:
mode:
authorAndre Marin <aamarin@us.ibm.com>2016-07-11 23:37:22 -0500
committerDaniel M. Crowell <dcrowell@us.ibm.com>2016-08-06 22:23:48 -0400
commit886838febf22a917245e51e44f0e39abba9c2e01 (patch)
treedc6d27f9f763aa7e70deb553ecf58c7b8319ceab /src/import/chips/p9/procedures/hwp/memory
parent76de804f0cc4d43165666e30f58f548777ebff76 (diff)
downloadtalos-hostboot-886838febf22a917245e51e44f0e39abba9c2e01.tar.gz
talos-hostboot-886838febf22a917245e51e44f0e39abba9c2e01.zip
Fix mss_utils_ut for multiple processors
Change-Id: Icbc29513a0edd9a5d3915ae3e625e2cf4194db1d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26895 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27635 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/import/chips/p9/procedures/hwp/memory')
-rw-r--r--src/import/chips/p9/procedures/hwp/memory/lib/utils/pos.H18
1 files changed, 11 insertions, 7 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/utils/pos.H b/src/import/chips/p9/procedures/hwp/memory/lib/utils/pos.H
index 9e065f718..3989e13b2 100644
--- a/src/import/chips/p9/procedures/hwp/memory/lib/utils/pos.H
+++ b/src/import/chips/p9/procedures/hwp/memory/lib/utils/pos.H
@@ -98,7 +98,12 @@ inline typename TT::pos_type pos(const fapi2::Target<T>& i_target)
fapi_try_exit:
// If we can't get our unit position, we're in other trouble
FAPI_ERR("can't get our chip unit position");
+
+ // Cronusflex doesn't support Assert.
+ // Enable once move to dev complete - AAM
+#if MOVED_AWAY_FROM_CRONUSFLEX
fapi2::Assert(false);
+#endif
return 0;
}
@@ -122,7 +127,9 @@ inline posTraits<fapi2::TARGET_TYPE_DIMM>::pos_type pos(const fapi2::Target<fapi
fapi_try_exit:
// If we can't get our position, we're in other trouble
FAPI_ERR("can't get our position");
+#if MOVED_AWAY_FROM_CRONUSFLEX
fapi2::Assert(false);
+#endif
return 0;
}
@@ -148,7 +155,9 @@ inline posTraits<fapi2::TARGET_TYPE_PROC_CHIP>::pos_type pos(
fapi_try_exit:
// If we can't get our position, we're in other trouble
FAPI_ERR("can't get our position");
+#if MOVED_AWAY_FROM_CRONUSFLEX
fapi2::Assert(false);
+#endif
return 0;
}
@@ -181,10 +190,6 @@ inline uint8_t relative_pos<fapi2::TARGET_TYPE_MCBIST>(const fapi2::Target<fapi2
template< fapi2::TargetType T, typename TT = posTraits<T> >
inline uint32_t fapi_pos(const fapi2::Target<T>& i_target)
{
-
-// Doesn't look like ATTR_FAPI_POS is supported in Cronus yet, so we'll
-// use our chip-relative position for the time being.
-#ifdef ATTR_FAPI_POS_SUPPORTED
uint32_t i_pos = 0;
// Don't use FAPI_TRY as you'll mess up fapi2::current_err which
@@ -199,11 +204,10 @@ inline uint32_t fapi_pos(const fapi2::Target<T>& i_target)
fapi_try_exit:
// If we can't get our fapi position, we're in other trouble
FAPI_ERR("can't get our fapi position");
+#if MOVED_AWAY_FROM_CRONUSFLEX
fapi2::Assert(false);
- return 0;
-#else
- return pos(i_target);
#endif
+ return 0;
}
OpenPOWER on IntegriCloud