diff options
author | CHRISTINA L. GRAVES <clgraves@us.ibm.com> | 2016-10-18 15:02:09 -0500 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2016-10-24 11:51:46 -0400 |
commit | 9a910ee6d940ef80d5d048f3fdd8ed7414a0904e (patch) | |
tree | 07cbdf77d2d642cf441bbbd8b85c785032b1717e /src/import | |
parent | 1c691b81d264a66d1800d992b436cff37a0cb5ef (diff) | |
download | talos-sbe-9a910ee6d940ef80d5d048f3fdd8ed7414a0904e.tar.gz talos-sbe-9a910ee6d940ef80d5d048f3fdd8ed7414a0904e.zip |
Removing adding extra FFDC data. Also removing phb includes.
Change-Id: I862c02033184e0ae5baf0a220e74d2d0ed47852e
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31545
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.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/31709
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/import')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_sbe_check_quiesce.C | 27 |
1 files changed, 2 insertions, 25 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_sbe_check_quiesce.C b/src/import/chips/p9/procedures/hwp/nest/p9_sbe_check_quiesce.C index db4a4172..e97ab652 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_sbe_check_quiesce.C +++ b/src/import/chips/p9/procedures/hwp/nest/p9_sbe_check_quiesce.C @@ -41,7 +41,6 @@ // Includes //-------------------------------------------------------------------------- #include <p9_sbe_check_quiesce.H> -#include <p9_phb_hv_access.H> #include <p9_quad_scom_addresses.H> #include <p9_misc_scom_addresses_fld.H> #include <p9_misc_scom_addresses.H> @@ -81,8 +80,6 @@ extern "C" { fapi_try_exit: fapi2::ReturnCode saveError = fapi2::current_err; fapi2::buffer<uint64_t> l_data(0); - uint64_t l_ffdc_data; - fapi2::ffdc_t CHECKSTOP_DATA; //If the quiesce fails then checkstop the system if (fapi2::current_err) @@ -93,29 +90,8 @@ extern "C" { if (rc) { - FAPI_INF("ERROR: There was an error doing the checkstop, it did not go through"); - //Set the extra FFDC data to DEADBEEFDEADBEEF which tells us there was a fail with the checkstop - l_ffdc_data = 0xDEADBEEFDEADBEEF; + FAPI_INF("ERROR: There was an error doing the checkstop, it may not have gone through"); } - else - { - //Add extra FFDC data so we can make sure the checkstop happened - rc = fapi2::getScom(i_target, 0x570F001C, l_data); - - if (rc) - { - FAPI_INF("ERROR: There was an error reading the checkstop data"); - l_ffdc_data = 0xBEEFDEADBEEFDEAD; - } - else - { - l_data.extractToRight(l_ffdc_data, 0, 64); - } - } - - CHECKSTOP_DATA.ptr() = static_cast<void*>(&l_ffdc_data); - CHECKSTOP_DATA.size() = sizeof(l_ffdc_data); - FAPI_ADD_INFO_TO_HWP_ERROR(saveError, RC_P9_CHECK_XSTOP); } FAPI_DBG("Exiting..."); @@ -805,6 +781,7 @@ extern "C" { FAPI_TRY(fapi2::putScom(i_target, PU_INT_CQ_RST_CTL, l_int_cq_rst_ctl_data), "Error writing INT_CQ_RST_CTL register to set reset"); + fapi_try_exit: FAPI_DBG("Exiting..."); return fapi2::current_err; |