From eb3edaac057e7c44a2e5649a89f2a6bf72fd4a37 Mon Sep 17 00:00:00 2001 From: Joachim Fenkes Date: Thu, 9 Mar 2017 14:57:12 -0600 Subject: stopclocks: Don't set ABSTCLK_MUXSEL on clock stop In chiplets with dual-domain arrays, setting the mux to 1 will enable ABISTing those arrays but break scanning some *_abst chains in neighboring chiplets. Keep the select at 0 to keep scanning intact after a clock stop; ABIST related tools flip that mux anyway. Change-Id: Ic49a6357dd8766fc554125b4bf3e30cb53a2585f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37754 Tested-by: Jenkins Server Tested-by: PPE CI Reviewed-by: Soma Bhanutej Reviewed-by: Joseph J. McGill Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37760 Reviewed-by: Hostboot Team Tested-by: FSP CI Jenkins Reviewed-by: Sachin Gupta --- src/import/chips/p9/procedures/hwp/perv/p9_common_stopclocks.C | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'src/import/chips') diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_common_stopclocks.C b/src/import/chips/p9/procedures/hwp/perv/p9_common_stopclocks.C index feb19796..a62278e9 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_common_stopclocks.C +++ b/src/import/chips/p9/procedures/hwp/perv/p9_common_stopclocks.C @@ -83,11 +83,9 @@ fapi2::ReturnCode p9_common_stopclocks_cplt_ctrl_action_function( l_data64.insertFromRight<4, 11>(l_cplt_ctrl_init); FAPI_TRY(fapi2::putScom(i_target_chiplet, PERV_CPLT_CTRL1_OR, l_data64)); - FAPI_DBG("set abistclk_muxsel and syncclk_muxsel"); + FAPI_DBG("set syncclk_muxsel"); //Setting CPLT_CTRL0 register value l_data64.flush<0>(); - //CPLT_CTRL0.CTRL_CC_ABSTCLK_MUXSEL_DC = 1 - l_data64.setBit(); //CPLT_CTRL0.TC_UNIT_SYNCCLK_MUXSEL_DC = 1 l_data64.setBit(); FAPI_TRY(fapi2::putScom(i_target_chiplet, PERV_CPLT_CTRL0_OR, l_data64)); -- cgit v1.2.1