PSTATE_PARAMETER_BLOCK: FIx VFRT trace logic

Change-Id: I1caa4fa1dc3d676ca4a718e97539f2c686d216ba
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/47058
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Prem Shanker Jha <premjha2@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/47062
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 12 insertions, 3 deletions

diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C
index 3dc27116a..a9a73ead8 100644
--- a/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C
+++ b/src/import/chips/p9/procedures/hwp/pm/p9_pstate_parameter_block.C
@@ -3193,7 +3193,7 @@ iddq_print(IddqTable* i_iddqt)
 
     FAPI_INF("%s", l_line_str);
 
-    // get average temperatur}e measurements with all cores and caches OFF
+    // get average temperature measurements with all cores and caches OFF
     IDDQ_TRACE ("  Average temp all cores OFF, caches OFF:", IDDQ_DESC_SIZE);
 
     for (i = 0; i < IDDQ_MEASUREMENTS; i++)
@@ -3818,13 +3818,14 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb,
 
     bool b_fratio_set = true;
 
+    bool b_first_vratio_set = true;
+
     //Initialize VFRT data part
     for (l_index_0 = 0; l_index_0 < VFRT_FRATIO_SIZE; ++l_index_0)
     {
         strcpy(l_buffer_str, "");
         strcpy(l_line_str, "    ");
 
-        bool b_first_vratio_set = true;
 
         for (l_index_1 = 0; l_index_1 < VFRT_VRATIO_SIZE; ++l_index_1)
         {
@@ -3852,7 +3853,7 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb,
             // is correct without overfilling the HB trace buffer.
             if (!((l_index_1 + 1) % 8) && b_first_vratio_set && b_fratio_set)
             {
-                FAPI_INF("%s", l_line_str);
+                FAPI_DBG("%s ", l_line_str);
                 strcpy(l_buffer_str, "");
                 strcpy(l_line_str, "    ");
                 b_first_vratio_set = false;
@@ -3863,7 +3864,15 @@ void p9_pstate_update_vfrt(const GlobalPstateParmBlock* i_gppb,
 
         // If fratio is not enabled, don't trace the remaining, duplicate entries.
         if (!l_enable_fratio)
+        {
             b_fratio_set = false;
+            b_first_vratio_set = false;
+        }
+        else
+        {
+            b_fratio_set = true;
+            b_first_vratio_set = true;
+        }
 
     }