diff options
author | CHRISTINA L. GRAVES <clgraves@us.ibm.com> | 2017-01-12 13:14:39 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-01-16 10:48:25 -0500 |
commit | f287be6fdc7815cfb0c3b9e72eab3821cd112677 (patch) | |
tree | 4a5482b0320d3cbe26d3df3133831c9c380ff834 /src/import | |
parent | 88ec04ee471af36d28ac6d30c6cd47efe18a8ac3 (diff) | |
download | talos-hostboot-f287be6fdc7815cfb0c3b9e72eab3821cd112677.tar.gz talos-hostboot-f287be6fdc7815cfb0c3b9e72eab3821cd112677.zip |
Changing sync miss count max to 2 instead of 1
based on documentation this is what it should be
Also adding a delay for the slave that wasn't in there before but should be
Change-Id: Id05ab1c69048e02de18b1974b84eec2fd5a8118c
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34795
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34799
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_tod_setup.C | 8 | ||||
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_tod_utils.H | 4 |
2 files changed, 8 insertions, 4 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_tod_setup.C b/src/import/chips/p9/procedures/hwp/nest/p9_tod_setup.C index 3e1c89211..f9406d5ac 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_tod_setup.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_tod_setup.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2016 */ +/* Contributors Listed Below - COPYRIGHT 2015,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -318,6 +318,10 @@ extern "C" PERV_TOD_S_PATH_CTRL_REG_0_STEP_CHECK_CPS_DEVIATION_LEN>(STEP_CHECK_CPS_DEVIATION_75_00_PCENT); data.insertFromRight<PERV_TOD_S_PATH_CTRL_REG_0_STEP_CHECK_VALIDITY_COUNT, PERV_TOD_S_PATH_CTRL_REG_0_STEP_CHECK_VALIDITY_COUNT_LEN>(STEP_CHECK_VALIDITY_COUNT_8); + data.insertFromRight<PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_CHECK_CPS_DEVIATION, PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_CHECK_CPS_DEVIATION_LEN> + (STEP_CHECK_CPS_DEVIATION_75_00_PCENT); + data.insertFromRight<PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_CHECK_CPS_DEVIATION_FACTOR, PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_CHECK_CPS_DEVIATION_FACTOR_LEN> + (STEP_CHECK_CPS_DEVIATION_FACTOR_1); } else // (i_tod_sel==TOD_SECONDARY) { @@ -334,7 +338,7 @@ extern "C" //In either case set the S_PATH_REMOTE_SYNC_MISS_COUNT_MAX to 1 data.insertFromRight<PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_MISS_COUNT_MAX, PERV_TOD_S_PATH_CTRL_REG_REMOTE_SYNC_MISS_COUNT_MAX_LEN> - (S_PATH_REMOTE_SYNC_MISS_COUNT_1); + (S_PATH_REMOTE_SYNC_MISS_COUNT_2); FAPI_TRY(fapi2::putScom(*target, PERV_TOD_S_PATH_CTRL_REG, data), "fapiPutScom error for PERV_TOD_S_PATH_CTRL_REG SCOM."); diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_tod_utils.H b/src/import/chips/p9/procedures/hwp/nest/p9_tod_utils.H index 51a74169e..1a8bf74df 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_tod_utils.H +++ b/src/import/chips/p9/procedures/hwp/nest/p9_tod_utils.H @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2016 */ +/* Contributors Listed Below - COPYRIGHT 2015,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -267,7 +267,7 @@ const uint32_t PB_OLINK_RT_DELAY_CTL_SET_LINK_6 = 6; const uint32_t PB_OLINK_RT_DELAY_CTL_SET_LINK_7 = 7; //The number of syncs that we are ok with the slave path missing (this must be set to at least 1 or there will be a fail) -const uint32_t S_PATH_REMOTE_SYNC_MISS_COUNT_1 = 1; +const uint32_t S_PATH_REMOTE_SYNC_MISS_COUNT_2 = 5; //I don't see this one listed in the P9 documentation const uint32_t TOD_CHIP_CTRL_REG_LOW_ORDER_STEP_VAL_3F = 0x3F; |