p9_quad_power_off.C fix for chip unit pos

Change-Id: I9bc9d17923e2b68092bde9274f9c4b0087969859
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31124
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31247
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>

1 files changed, 7 insertions, 9 deletions

diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_quad_power_off.C b/src/import/chips/p9/procedures/hwp/pm/p9_quad_power_off.C
index e5bd8117..e93dd70a 100644
--- a/src/import/chips/p9/procedures/hwp/pm/p9_quad_power_off.C
+++ b/src/import/chips/p9/procedures/hwp/pm/p9_quad_power_off.C
@@ -56,23 +56,21 @@
 fapi2::ReturnCode p9_quad_power_off(
     const fapi2::Target<fapi2::TARGET_TYPE_EQ>& i_target)
 {
-    fapi2::ReturnCode rc   = fapi2::FAPI2_RC_SUCCESS;
-    uint8_t l_unit_pos     = 0;
+    fapi2::ReturnCode rc = fapi2::FAPI2_RC_SUCCESS;
+    uint32_t l_unit_pos = 0;
 
     FAPI_INF("p9_quad_power_off: 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("Quad power off chiplet no.%d", l_unit_pos);
 
     // Call the procedure
-    p9_pm_pfet_control_eq(i_target,
-                          PM_PFET_TYPE_C::BOTH,
-                          PM_PFET_TYPE_C::OFF);
-
-
-    FAPI_INF("p9_quad_power_off: ...Exiting");
+    FAPI_EXEC_HWP(rc, p9_pm_pfet_control_eq, i_target,
+                  PM_PFET_TYPE_C::BOTH,PM_PFET_TYPE_C::OFF);
+    FAPI_TRY(rc);
 
 fapi_try_exit:
+    FAPI_INF("p9_quad_power_off: ...Exiting");
     return fapi2::current_err;
 }