diff options
author | Chris Steffen <cwsteffen@us.ibm.com> | 2017-07-27 15:59:49 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-09-01 10:55:06 -0400 |
commit | 90710bc0d34d6dbacca53155702500915a168df5 (patch) | |
tree | 04504f1dfa800ef7d1c2327babdadad911b5b377 | |
parent | e73c836ba207192ef12fe1005f50c19591c1799a (diff) | |
download | talos-hostboot-90710bc0d34d6dbacca53155702500915a168df5.tar.gz talos-hostboot-90710bc0d34d6dbacca53155702500915a168df5.zip |
Making zcal errors visible
Change-Id: Idf3c083aaa14c5f9db1cf4bc75b21b6c117786d2
Original-Change-Id: I6a4dff7d58441bb2783ecd206254c3ccf125c3b6
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43802
Reviewed-by: Gary A. Peterson <garyp@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45485
Tested-by: Jenkins OP Build CI <op-jenkins+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>
-rw-r--r-- | src/import/chips/centaur/procedures/hwp/io/p9_io_cen_dccal.C | 26 |
1 files changed, 14 insertions, 12 deletions
diff --git a/src/import/chips/centaur/procedures/hwp/io/p9_io_cen_dccal.C b/src/import/chips/centaur/procedures/hwp/io/p9_io_cen_dccal.C index 009b74fb0..da4803884 100644 --- a/src/import/chips/centaur/procedures/hwp/io/p9_io_cen_dccal.C +++ b/src/import/chips/centaur/procedures/hwp/io/p9_io_cen_dccal.C @@ -134,28 +134,28 @@ fapi2::ReturnCode txZcalVerifyResults(uint32_t& io_pvalx4, uint32_t& io_nvalx4) if(io_pvalx4 > X4_MAX) { - FAPI_INF("Warning: Cen Tx Zcal Pval(0x%X) > Max Allowed(0x%X): Code will override with 0x%X and continue", + FAPI_ERR("I/O EDI Cen Tx Zcal Pval(0x%X) > Max Allowed(0x%X): Code will override with 0x%X and continue", io_pvalx4, X4_MAX, X4_MAX); io_pvalx4 = X4_MAX; } if(io_nvalx4 > X4_MAX) { - FAPI_INF("Warning: Cen Tx Zcal Nval(0x%X) > Max Allowed(0x%X): Code will override with 0x%X and continue.", + FAPI_ERR("I/O EDI Cen Tx Zcal Nval(0x%X) > Max Allowed(0x%X): Code will override with 0x%X and continue.", io_nvalx4, X4_MAX, X4_MAX); io_nvalx4 = X4_MAX; } if(io_pvalx4 < X4_MIN) { - FAPI_INF("Warning: Cen Tx Zcal Pval(0x%X) < Min Allowed(0x%X): Code will override with 0x%X and continue.", + FAPI_ERR("I/O EDI Cen Tx Zcal Pval(0x%X) < Min Allowed(0x%X): Code will override with 0x%X and continue.", io_pvalx4, X4_MIN, X4_MIN); io_pvalx4 = X4_MIN; } if(io_nvalx4 < X4_MIN) { - FAPI_INF("Warning: Cen Tx Zcal Nval(0x%X) < Min Allowed(0x%X): Code will override with 0x%X and continue.", + FAPI_ERR("I/O EDI Cen Tx Zcal Nval(0x%X) < Min Allowed(0x%X): Code will override with 0x%X and continue.", io_nvalx4, X4_MIN, X4_MIN); io_nvalx4 = X4_MIN; } @@ -214,18 +214,22 @@ fapi2::ReturnCode txZcalRunStateMachinePoll(const CEN_TGT& i_tgt) FAPI_TRY(io::read(EDI_TX_IMPCAL_PB, i_tgt, GRP0, LN0, regData)); } - + // Check for Zcal Done if(io::get(EDI_TX_ZCAL_DONE, regData) == 1) { - FAPI_DBG("tx_zcal_run_sm: I/O EDI cen Tx Zcal Poll Completed(%d/%d).", count, TIMEOUT); + FAPI_DBG("I/O EDI cen Tx Zcal Poll Completed(%d/%d).", count, TIMEOUT); } - else if(io::get(EDI_TX_ZCAL_ERROR, regData) == 1) + + // Check for Zcal Error + if(io::get(EDI_TX_ZCAL_ERROR, regData) == 1) { - FAPI_ERR("tx_zcal_run_sm: WARNING: Tx Z Calibration Error"); + FAPI_ERR("I/O EDI Cen Tx Z Calibration Error"); } - else + + // Check for Zcal Timeout + if(count >= TIMEOUT) { - FAPI_ERR("tx_zcal_run_sm: WARNING: Tx Z Calibration Timeout: Loops(%d)", count); + FAPI_ERR("I/O EDI Cen Tx Z Calibration Timeout: Loops(%d)", count); } fapi_try_exit: @@ -564,5 +568,3 @@ fapi_try_exit: FAPI_IMP("rx_dccal: I/O EDI Cen Exiting"); return fapi2::current_err; } - - |