From ebd72d043f2358485d2a1ad293c2ffbfc3369dba Mon Sep 17 00:00:00 2001
From: Thi Tran <thi@us.ibm.com>
Date: Fri, 19 Aug 2016 10:53:30 -0500
Subject: Change ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING name

Change to ATTR_MRW_HW_MIRRORING_ENABLE and invert its meaning to
make it more straight forward.

Change-Id: I9c4fbd30d7832d2b4a11e0a5420d52f79cfb41a6
Original-Change-Id: I0e63d52285a10bb1e8c75d4ff2598f0c50eab6de
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28536
Tested-by: Jenkins Server <pfd-jenkins+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>
Reviewed-by: CHRISTINA L. GRAVES <clgraves@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Matt K. Light <mklight@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/72544
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
---
 src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'src')

diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C b/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C
index 1b3f00701..e384055cd 100644
--- a/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C
+++ b/src/import/chips/p9/procedures/hwp/nest/p9_query_mssinfo.C
@@ -74,7 +74,7 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET
     uint64_t occ_base;
     uint64_t occ_size;
     uint8_t  mirror_policy;
-    uint8_t  no_mirror;
+    uint8_t  l_mirrorEnabled;
     char     chipid[200];
     char l_target_string[fapi2::MAX_ECMD_STRING_LEN];
 
@@ -196,14 +196,14 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET
                  (uint64_t)fapi2::current_err);
 
         //Mirrored memory attributes read if enabled
-        FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING,
-                               i_vect_pu_targets[i].getParent<fapi2::TARGET_TYPE_SYSTEM>(), no_mirror),
-                 "Error reading ATTR_MRW_ENHANCED_GROUPING_NO_MIRRORING, l_rc 0x%.8X",
+        FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_MRW_HW_MIRRORING_ENABLE,
+                               i_vect_pu_targets[i].getParent<fapi2::TARGET_TYPE_SYSTEM>(), l_mirrorEnabled),
+                 "Error reading ATTR_MRW_HW_MIRRORING_ENABLE, l_rc 0x%.8X",
                  (uint64_t)fapi2::current_err);
 
-        FAPI_DBG("p9_query_mssinfo: Current no_mirror=%u!\n", no_mirror);
+        FAPI_DBG("p9_query_mssinfo: Current l_mirrorEnabled=%u!\n", l_mirrorEnabled);
 
-        if (no_mirror == 0)
+        if (l_mirrorEnabled == 1)
         {
             // ATTR_PROC_MIRROR_SIZES
             FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_PROC_MIRROR_SIZES, i_vect_pu_targets[i], mirror_sizes),
@@ -293,7 +293,7 @@ fapi2::ReturnCode p9_query_mssinfo(const std::vector<fapi2::Target<fapi2::TARGET
         // Print out memory reservations
         //------------------------------------------------------------------------------------------------------------------------
         if(((mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_NORMAL ) ||
-            (mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_FLIPPED)    ) && (no_mirror == 0) )
+            (mirror_policy == fapi2::ENUM_ATTR_MEM_MIRROR_PLACEMENT_POLICY_FLIPPED)    ) && (l_mirrorEnabled == 1) )
         {
             // ATTR_PROC_NHTM_BAR_BASE_ADDR: Get Nest Hardware Trace Macro (NHTM) bar base addr
             FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_PROC_NHTM_BAR_BASE_ADDR, i_vect_pu_targets[i], nhtm_base),
-- 
cgit v1.2.1