diff options
| author | spashabk-in <shakeebbk@in.ibm.com> | 2018-04-24 07:17:03 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-05-18 10:41:24 -0400 |
| commit | 764053f345194b23835de3bbfbbbda7143cd48f5 (patch) | |
| tree | 4884bedc92e8f73558931740150577afce8730f8 | |
| parent | 775a7639b7d3f9932156349549472996abdc8cea (diff) | |
| download | talos-hostboot-764053f345194b23835de3bbfbbbda7143cd48f5.tar.gz talos-hostboot-764053f345194b23835de3bbfbbbda7143cd48f5.zip | |
Remove clear(step 3) of reset vectors
Making the reset vector bits sticky to figure out hreset flow
Clear the reset vectors on start cbs
Change-Id: I78273cf14a3f623052e8c01618b2596323801207
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57732
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
Reviewed-by: SRINIVAS V. POLISETTY <srinivan@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57738
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>
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/perv/p9_sbe_hreset.C | 7 | ||||
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/perv/p9_start_cbs.C | 3 |
2 files changed, 2 insertions, 8 deletions
diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_sbe_hreset.C b/src/import/chips/p9/procedures/hwp/perv/p9_sbe_hreset.C index ea75e91ab..09c8723f0 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_sbe_hreset.C +++ b/src/import/chips/p9/procedures/hwp/perv/p9_sbe_hreset.C @@ -170,10 +170,6 @@ fapi2::ReturnCode p9_sbe_hreset( FAPI_TRY(l_data64.setBit(l_startVectorBit)); FAPI_TRY(fapi2::putScom(i_target, PERV_SB_CS_SCOM, l_data64), "Error from putScom to PERV_SB_CS_SCOM (2)"); - - FAPI_TRY(l_data64.clearBit(l_startVectorBit)); - FAPI_TRY(fapi2::putScom(i_target, PERV_SB_CS_SCOM, l_data64), - "Error from putScom to PERV_SB_CS_SCOM (3)"); } #ifndef __HOSTBOOT_RUNTIME @@ -203,9 +199,6 @@ fapi2::ReturnCode p9_sbe_hreset( FAPI_TRY(fapi2::putCfamRegister(i_target, PERV_SB_CS_FSI, l_data32), "Error from putCfamRegister to PERV_SB_CS_FSI (2)"); - FAPI_TRY(l_data32.clearBit(l_startVectorBit)); - FAPI_TRY(fapi2::putCfamRegister(i_target, PERV_SB_CS_FSI, l_data32), - "Error from putCfamRegister to PERV_SB_CS_FSI (3)"); } #endif diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_start_cbs.C b/src/import/chips/p9/procedures/hwp/perv/p9_start_cbs.C index ac3468162..d2c237d65 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_start_cbs.C +++ b/src/import/chips/p9/procedures/hwp/perv/p9_start_cbs.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2015,2017 */ +/* Contributors Listed Below - COPYRIGHT 2015,2018 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -95,6 +95,7 @@ fapi2::ReturnCode p9_start_cbs(const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP> FAPI_DBG("Setting up hreset to 0"); FAPI_TRY(fapi2::getCfamRegister(i_target_chip, PERV_SB_CS_FSI, l_data32)); l_data32.clearBit<PERV_SB_CS_START_RESTART_VECTOR0>(); + l_data32.clearBit<PERV_SB_CS_START_RESTART_VECTOR1>(); FAPI_TRY(fapi2::putCfamRegister(i_target_chip, PERV_SB_CS_FSI, l_data32)); FAPI_DBG("check for VDN_PGOOD"); |
