Stopclocks procedure to stop PLL region also

Change-Id: Ia3eb8355e4609a1f3328f8eff00621fab594d0a3 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41059 Reviewed-by: SRINIVAS V. POLISETTY <srinivan@in.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Dev-Ready: Soma Bhanutej <soma.bhanu@in.ibm.com> Reviewed-by: Daniel Kiss <kiss@de.ibm.com> Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41088 Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
open-power · May 31, 2017 · 3a909a6 · 3a909a6
1 parent 4f594cf
commit 3a909a6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_cplt_stopclocks.C b/src/import/chips/p9/procedures/hwp/perv/p9_cplt_stopclocks.C
@@ -54,7 +54,7 @@ enum P9_CPLT_STOPCLOCKS_Private_Constants
     CLOCK_TYPES = 0x7,
     DONT_STARTMASTER = 0x0,
     DONT_STARTSLAVE = 0x0,
-    REGIONS_ALL_EXCEPT_PLL = 0x7FE,
+    REGIONS_ALL_INCLUDING_PLL = 0x7FF,
     STARTMASTER = 0x1,
     STARTSLAVE = 0x1
 };
@@ -110,8 +110,7 @@ fapi2::ReturnCode p9_cplt_stopclocks(const
         FAPI_TRY(p9_common_stopclocks_raise_fence(l_trgt_chplt));
 
         FAPI_DBG("Region setup");
-        FAPI_TRY(p9_perv_sbe_cmn_regions_setup_64(l_trgt_chplt, REGIONS_ALL_EXCEPT_PLL,
-                 l_regions));
+        FAPI_TRY(p9_perv_sbe_cmn_regions_setup_64(l_trgt_chplt, REGIONS_ALL_INCLUDING_PLL, l_regions));
         FAPI_DBG("Regions value: %#018lX", l_regions);
 
         FAPI_DBG("Call module clock start stop for xbus, obus, pcie, mc chiplets");

diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_tp_stopclocks.C b/src/import/chips/p9/procedures/hwp/perv/p9_tp_stopclocks.C
@@ -52,8 +52,8 @@ enum P9_TP_STOPCLOCKS_Private_Constants
     DONT_STARTMASTER = 0x0,
     DONT_STARTSLAVE = 0x0,
     REGIONS_ONLY_PIB_NET = 0x300,
-    REGIONS_ALL_EXCEPT_PIB_NET_PLL = 0x4FE,
-    REGIONS_ALL_EXCEPT_PLL = 0x7FE,
+    REGIONS_ALL_EXCEPT_PIB_NET = 0x4FF,
+    REGIONS_ALL_INCLUDING_PLL = 0x7FF,
     STARTMASTER = 0x1,
     STARTSLAVE = 0x1
 };
@@ -109,18 +109,18 @@ fapi2::ReturnCode p9_tp_stopclocks(const fapi2::Target<fapi2::TARGET_TYPE_PROC_C
 
     if(i_stop_tp_clks && i_stop_pib_clks)
     {
-        FAPI_DBG("p9_tp_stopclocks: TP regions selected is REGIONS_ALL_EXCEPT_PLL");
+        FAPI_DBG("p9_tp_stopclocks: TP regions selected is REGIONS_ALL_INCLUDING_PLL");
         FAPI_TRY(p9_perv_sbe_cmn_regions_setup_64(
                      i_target_chip.getChildren<fapi2::TARGET_TYPE_PERV>(fapi2::TARGET_FILTER_TP,
-                             fapi2::TARGET_STATE_FUNCTIONAL)[0], REGIONS_ALL_EXCEPT_PLL, l_clock_regions));
+                             fapi2::TARGET_STATE_FUNCTIONAL)[0], REGIONS_ALL_INCLUDING_PLL, l_clock_regions));
         FAPI_DBG("p9_tp_stopclocks: Regions value: %#018lX", l_clock_regions);
     }
     else if(i_stop_tp_clks)
     {
-        FAPI_DBG("p9_tp_stopclocks: TP regions selected is REGIONS_ALL_EXCEPT_PIB_NET_PLL");
+        FAPI_DBG("p9_tp_stopclocks: TP regions selected is REGIONS_ALL_EXCEPT_PIB_NET");
         FAPI_TRY(p9_perv_sbe_cmn_regions_setup_64(
                      i_target_chip.getChildren<fapi2::TARGET_TYPE_PERV>(fapi2::TARGET_FILTER_TP,
-                             fapi2::TARGET_STATE_FUNCTIONAL)[0], REGIONS_ALL_EXCEPT_PIB_NET_PLL, l_clock_regions));
+                             fapi2::TARGET_STATE_FUNCTIONAL)[0], REGIONS_ALL_EXCEPT_PIB_NET, l_clock_regions));
         FAPI_DBG("p9_tp_stopclocks: Regions value: %#018lX", l_clock_regions);
     }