summaryrefslogtreecommitdiffstats
path: root/src/import/chips/p9/utils
diff options
context:
space:
mode:
authorClaus Michael Olsen <cmolsen@us.ibm.com>2018-02-22 11:16:19 -0600
committerSachin Gupta <sgupta2m@in.ibm.com>2018-02-28 22:38:49 -0500
commit54f0bc5c31d34c47ef9a3f2f3a2ee42398617669 (patch)
treefa9c29ee2b0a08cd6582311db47643619fc8d469 /src/import/chips/p9/utils
parent20c449a70cde1ca055568bcb03536cd02888a640 (diff)
downloadtalos-sbe-54f0bc5c31d34c47ef9a3f2f3a2ee42398617669.tar.gz
talos-sbe-54f0bc5c31d34c47ef9a3f2f3a2ee42398617669.zip
Update to putRingUtils to proper scanning of perv_pll_bndy_flt rings
I've updated p9_putRingUtils.C for SBE to make sure we do a 1-bit boundary scanning for the perv_pll_bndy_flt rings since they are override rings by nature. Change-Id: I1c8a63708c571f67be5359b1a0e4a9b050a8275b Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54575 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54587 Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/import/chips/p9/utils')
-rw-r--r--src/import/chips/p9/utils/imageProcs/p9_ring_id.h16
-rw-r--r--src/import/chips/p9/utils/p9_putRingUtils.C5
2 files changed, 15 insertions, 6 deletions
diff --git a/src/import/chips/p9/utils/imageProcs/p9_ring_id.h b/src/import/chips/p9/utils/imageProcs/p9_ring_id.h
index 338c48c9..d4fb93e9 100644
--- a/src/import/chips/p9/utils/imageProcs/p9_ring_id.h
+++ b/src/import/chips/p9/utils/imageProcs/p9_ring_id.h
@@ -5,7 +5,7 @@
/* */
/* OpenPOWER sbe Project */
/* */
-/* Contributors Listed Below - COPYRIGHT 2016,2017 */
+/* Contributors Listed Below - COPYRIGHT 2016,2018 */
/* [+] International Business Machines Corp. */
/* */
/* */
@@ -29,9 +29,17 @@
/// @enum RingID
/// @brief Enumeration of Ring ID values. These values are used to traverse
/// an image having Ring Containers.
-// NOTE: Do not change the numbering, the sequence or add new constants to
-// the below enum, unless you know the effect it has on the traversing
-// of the image for Ring Containers.
+
+// *****************
+// *** IMPORTANT ***
+// *****************
+// Do *NOT* change the numbering, the sequence or add new constants to
+// the below enum, unless you know the effect it has on the traversing
+// of the image for Ring Containers:
+// - There are hard coded assumptions in SBE and SGPE PPE codes about the
+// below enums.
+// - A change to the below values is likely to cause a co-req situation.
+//
enum RingID
{
//*****************************
diff --git a/src/import/chips/p9/utils/p9_putRingUtils.C b/src/import/chips/p9/utils/p9_putRingUtils.C
index 8f8b6689..bdefd2d9 100644
--- a/src/import/chips/p9/utils/p9_putRingUtils.C
+++ b/src/import/chips/p9/utils/p9_putRingUtils.C
@@ -5,7 +5,7 @@
/* */
/* OpenPOWER sbe Project */
/* */
-/* Contributors Listed Below - COPYRIGHT 2016,2017 */
+/* Contributors Listed Below - COPYRIGHT 2016,2018 */
/* [+] International Business Machines Corp. */
/* */
/* */
@@ -1149,7 +1149,8 @@ fapi2::ReturnCode rs4DecompressionSvc(
//This is a special case for eq_ana_bndy bucket rings
if ( ( (l_ringId >= eq_ana_bndy_bucket_0) && (l_ringId <= eq_ana_bndy_bucket_25) ) ||
( l_ringId == eq_ana_bndy_bucket_l3dcc ) ||
- ( (l_ringId >= eq_ana_bndy_bucket_26) && (l_ringId <= eq_ana_bndy_bucket_41) ) )
+ ( (l_ringId >= eq_ana_bndy_bucket_26) && (l_ringId <= eq_ana_bndy_bucket_41) ) ||
+ ( (l_ringId >= perv_pll_bndy_flt_1) && (l_ringId <= perv_pll_bndy_flt_4) ) )
{
i_applyOverride = true;
}
OpenPOWER on IntegriCloud