summaryrefslogtreecommitdiffstats
path: root/src/import/chips/p9/procedures
diff options
context:
space:
mode:
Diffstat (limited to 'src/import/chips/p9/procedures')
-rw-r--r--src/import/chips/p9/procedures/hwp/pm/p9_setup_evid.C13
1 files changed, 7 insertions, 6 deletions
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_setup_evid.C b/src/import/chips/p9/procedures/hwp/pm/p9_setup_evid.C
index 15af9ab31..d52409657 100644
--- a/src/import/chips/p9/procedures/hwp/pm/p9_setup_evid.C
+++ b/src/import/chips/p9/procedures/hwp/pm/p9_setup_evid.C
@@ -513,19 +513,19 @@ p9_setup_evid_voltageWrite(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_
while (!l_goodResponse);
// Compute the delta
- l_delta_mv = l_present_voltage_mv - i_voltage_mv;
+ l_delta_mv = (int32_t)l_present_voltage_mv - (int32_t)i_voltage_mv;
if (l_delta_mv > 0)
{
- FAPI_DBG("Decreasing voltage - delta = %d", l_delta_mv );
+ FAPI_INF("Decreasing voltage - delta = %d", l_delta_mv );
}
else if (l_delta_mv < 0)
{
- FAPI_DBG("Increasing voltage - delta = %d", l_delta_mv );
+ FAPI_INF("Increasing voltage - delta = %d", l_delta_mv );
}
else
{
- FAPI_DBG("Voltage to be set equals the initial voltage");
+ FAPI_INF("Voltage to be set equals the initial voltage");
}
// Break into steps limited by attr.attr_ext_vrm_step_size_mv
@@ -534,7 +534,7 @@ p9_setup_evid_voltageWrite(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_
// Hostboot doesn't support abs()
uint32_t l_abs_delta_mv = l_delta_mv < 0 ? -l_delta_mv : l_delta_mv;
- if (l_abs_delta_mv > i_ext_vrm_step_size_mv)
+ if (i_ext_vrm_step_size_mv > 0 && l_abs_delta_mv > i_ext_vrm_step_size_mv )
{
if (l_delta_mv > 0) // Decreasing
{
@@ -586,7 +586,8 @@ p9_setup_evid_voltageWrite(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_
while (!l_goodResponse);
l_present_voltage_mv = l_target_mv;
- l_delta_mv = l_present_voltage_mv - i_voltage_mv;
+ l_delta_mv = (int32_t)l_present_voltage_mv - (int32_t)i_voltage_mv;
+ FAPI_INF("New delta = %d", l_delta_mv );
}
fapi_try_exit:
OpenPOWER on IntegriCloud