SBE compile issue fix for p9_pm_pfet_control.C

Change-Id: I62ac8fb5e119a41e174f5d48e167587129c6170a Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30829 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Gregory S. Still <stillgs@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36101 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
open-power · Feb 12, 2017 · ea1efc2 · ea1efc2
1 parent 75c1736
commit ea1efc2
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_pm_pfet_control.C b/src/import/chips/p9/procedures/hwp/pm/p9_pm_pfet_control.C
@@ -139,13 +139,13 @@ fapi2::ReturnCode p9_pm_pfet_control_eq(
     const PM_PFET_TYPE_C::pfet_force_t i_op)
 {
     fapi2::current_err     = fapi2::FAPI2_RC_SUCCESS;
-    uint8_t l_unit_pos     = 0;
+    uint32_t l_unit_pos     = 0;
     bool core_target_found = false;
 
     FAPI_INF("p9_pm_pfet_control_eq: Entering...");
 
     // Get chiplet position
-    FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, i_target, l_unit_pos));
+    l_unit_pos = i_target.getChipletNumber();
     FAPI_INF("pfet control for EQ chiplet %d", l_unit_pos);
 
     // When i_op == OFF all functional cores first followed by EQ
@@ -160,7 +160,7 @@ fapi2::ReturnCode p9_pm_pfet_control_eq(
     for (auto l_core_target : i_target.getChildren<fapi2::TARGET_TYPE_CORE>
          (fapi2::TARGET_STATE_FUNCTIONAL))
     {
-        FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, l_core_target, l_unit_pos));
+        l_unit_pos = l_core_target.getChipletNumber();
         FAPI_INF("Core chiplet %d in EQ", l_unit_pos);
         FAPI_TRY(pfet_ctrl<fapi2::TargetType::TARGET_TYPE_CORE>(l_core_target,
                  i_rail, i_op), "Error: pfet_ctrl for core!!");
@@ -193,20 +193,20 @@ fapi2::ReturnCode p9_pm_pfet_control_ex(
     const PM_PFET_TYPE_C::pfet_force_t i_op)
 {
     fapi2::current_err     = fapi2::FAPI2_RC_SUCCESS;
-    uint8_t l_unit_pos     = 0;
+    uint32_t l_unit_pos     = 0;
     bool core_target_found = false;
 
     FAPI_INF("p9_pm_pfet_control_ex: Entering...");
 
     // Get chiplet position
-    FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, i_target, l_unit_pos));
+    l_unit_pos = i_target.getChipletNumber();
     FAPI_INF("pfet control for EX chiplet %d", l_unit_pos);
 
     // Check for all core chiplets in EX and power on/off targets accordingly
     for (auto l_core_target : i_target.getChildren<fapi2::TARGET_TYPE_CORE>
          (fapi2::TARGET_STATE_FUNCTIONAL))
     {
-        FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, l_core_target, l_unit_pos));
+        l_unit_pos = l_core_target.getChipletNumber();
         FAPI_INF("Core chiplet %d in EX", l_unit_pos);
         FAPI_TRY(pfet_ctrl<fapi2::TargetType::TARGET_TYPE_CORE>(l_core_target,
                  i_rail, i_op), "Error: pfet_ctrl for core!!");
@@ -257,12 +257,12 @@ fapi2::ReturnCode pfet_ctrl(
     const PM_PFET_TYPE_C::pfet_force_t i_op)
 {
     fapi2::current_err = fapi2::FAPI2_RC_SUCCESS;
-    uint8_t l_unit_pos     = 0;
+    uint32_t l_unit_pos     = 0;
 
     FAPI_INF("pfet_ctrl: Entering...");
 
     // Get chiplet position
-    FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_UNIT_POS, i_target, l_unit_pos));
+    l_unit_pos = i_target.getChipletNumber();
 
     // Check for target passed
     if(i_target.getType() & fapi2::TARGET_TYPE_CORE)
@@ -279,7 +279,7 @@ fapi2::ReturnCode pfet_ctrl(
         FAPI_ASSERT(false,
                     fapi2::PFET_CTRL_INVALID_CHIPLET_ERROR()
                     .set_TARGET(i_target),
-                    "ERROR: Invalid chiplet selected. Target:%x", i_target);
+                    "ERROR: Invalid chiplet selected");
     }
 
     switch(i_op)