diff options
| author | Chris Steffen <cwsteffen@us.ibm.com> | 2018-08-13 15:38:51 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-08-20 10:36:15 -0500 |
| commit | 70e60e2e03e7e21b5be76493f312644f50983496 (patch) | |
| tree | 8b43645b0c72fc56f2a22066095c1b9faa4146f5 /src/import | |
| parent | 8e74571d1bfbba9377de4ecbef568c80735395cf (diff) | |
| download | talos-hostboot-70e60e2e03e7e21b5be76493f312644f50983496.tar.gz talos-hostboot-70e60e2e03e7e21b5be76493f312644f50983496.zip | |
Set SMP Abus Rx AC Coupled at Dccal
- Setting rx ac coupled in dccal so that
the calibrated settings are valid.
Change-Id: I693a9c8422c9d35e68fcf2db03de850ed8be3ca4
CQ: SW442174
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64408
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: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Gary A. Peterson <garyp@us.ibm.com>
Reviewed-by: Megan P. Nguyen <pmegan@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64418
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>
Diffstat (limited to 'src/import')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/io/p9_io_obus_dccal.C | 25 | ||||
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/io/p9_io_obus_linktrain.C | 5 |
2 files changed, 25 insertions, 5 deletions
diff --git a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_dccal.C b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_dccal.C index 9e2b3ff8a..225b6fb4c 100644 --- a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_dccal.C +++ b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_dccal.C @@ -644,6 +644,27 @@ fapi_try_exit: return fapi2::current_err; } +/** + * @brief A I/O Set RX AC Coupled + * @param[in] i_tgt FAPI2 Target + * @param[in] i_data Data to be set + * @retval ReturnCode + */ +fapi2::ReturnCode set_obus_rx_ac_coupled( const OBUS_TGT i_tgt, const uint8_t i_data ) +{ + FAPI_IMP( "set_obus_rx_ac_coupled: I/O Obus Entering" ); + const uint8_t GRP0 = 0; + const uint8_t LN0 = 0; + + // Per Group Register + FAPI_TRY( io::rmw( OPT_RX_AC_COUPLED, i_tgt, GRP0, LN0, i_data ) ); + +fapi_try_exit: + FAPI_IMP( "set_obus_rx_ac_coupled: I/O Obus Exiting" ); + return fapi2::current_err; +} + + } // end namespace P9_IO_OBUS_DCCAL using namespace P9_IO_OBUS_DCCAL; @@ -677,6 +698,10 @@ fapi2::ReturnCode p9_io_obus_dccal( const OBUS_TGT i_tgt, const uint32_t i_lane_ // Power up Clock Distribution & Lanes FAPI_TRY( obus_powerup( i_tgt, i_lane_vector ) ); + // SW442174 :: Set RX_AC_COUPLED = 1 + // - This must be done before dccal to get accurate dccal values + FAPI_TRY( set_obus_rx_ac_coupled( i_tgt, 1 ) ); + // Run Tx Zcal State Machine FAPI_TRY( tx_run_zcal( i_tgt ), "I/O Obus Tx Run Z-Cal Failed" ); FAPI_DBG( "I/O Obus Tx Zcal State Machine Successful." ); diff --git a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_linktrain.C b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_linktrain.C index 49f68e57e..3643f4c37 100644 --- a/src/import/chips/p9/procedures/hwp/io/p9_io_obus_linktrain.C +++ b/src/import/chips/p9/procedures/hwp/io/p9_io_obus_linktrain.C @@ -133,11 +133,6 @@ fapi2::ReturnCode p9_io_obus_linktrain(const OBUS_TGT& i_tgt) FAPI_TRY(io::write(OPT_TX_MODE2_PL, i_tgt, GRP0, lane, l_data)); } - // Set RX_AC_COUPLED - FAPI_TRY(io::read( OPT_RX_CTL_MODE2_O_PG, i_tgt, GRP0, 0, l_data)); - io::set(OPT_RX_AC_COUPLED, 1, l_data); - FAPI_TRY(io::write( OPT_RX_CTL_MODE2_O_PG, i_tgt, GRP0, 0, l_data)); - FAPI_TRY(FAPI_ATTR_GET(fapi2::ATTR_CHIP_EC_FEATURE_HW419022, i_chip_target, l_hw419022), |
