diff options
author | Andre Marin <aamarin@us.ibm.com> | 2016-04-19 20:15:17 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-05-12 11:10:06 -0400 |
commit | 26753f805150fca2ada15f8527b4433a7a4edaa0 (patch) | |
tree | 9cbec57bd369c16a21f4deee8223ccb203d0663b /src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H | |
parent | effcd0c0d34e5e4032324e57a4d6d8d881167e32 (diff) | |
download | talos-hostboot-26753f805150fca2ada15f8527b4433a7a4edaa0.tar.gz talos-hostboot-26753f805150fca2ada15f8527b4433a7a4edaa0.zip |
Fix throttle procedure & MSS attribute clean up
Change-Id: Iccebe2f5bbad9d4fed2de914e0b65dba0b0b4662
Original-Change-Id: I7b545b65aaf9cdfea08ab2c5142898f5c971a74b
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/23486
Tested-by: Jenkins Server
Tested-by: Hostboot CI
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: JACOB L. HARVEY <jlharvey@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Brian R. Silver <bsilver@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/24407
Tested-by: FSP CI Jenkins
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H | 43 |
1 files changed, 0 insertions, 43 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H b/src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H index 028c272d3..90db0ff9c 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H +++ b/src/import/chips/p9/procedures/hwp/memory/lib/utils/checker.H @@ -79,49 +79,6 @@ fapi_try_exit: }// dram_type - -/// -/// @brief Checks nominal voltage is correct for all DIMMs -/// @param[in] i_target the fapi2 target -/// @param[in] i_dimm_state (i.e. functional or non-functional) -/// @return ReturnCode -/// -inline fapi2::ReturnCode module_nominal_voltage(const fapi2::Target<fapi2::TARGET_TYPE_MCS>& i_target, - fapi2::TargetState i_dimm_state) -{ - - uint64_t l_module_nom_voltage[mss::PORTS_PER_MCS][mss::MAX_DIMM_PER_PORT] = {0}; - - FAPI_TRY(mss::spd_module_nominal_voltage(i_target, &l_module_nom_voltage[0][0])); - - for (const auto& l_mca : i_target.getChildren<fapi2::TARGET_TYPE_MCA>()) - { - const auto l_port_num = mss::index(l_mca); - - for (const auto& l_dimm : l_mca.getChildren<fapi2::TARGET_TYPE_DIMM>(i_dimm_state)) - { - const auto l_dimm_num = mss::index(l_dimm); - - FAPI_INF("%s DRAM nominal voltage (in milliseconds) is %d", - mss::c_str(l_dimm), - l_module_nom_voltage[l_port_num][l_dimm_num]); - - // All DIMMs have to be operable at 1.2 V, else don't IPL (according to Warren) - FAPI_ASSERT( l_module_nom_voltage[l_port_num][l_dimm_num] == fapi2::ENUM_ATTR_SPD_MODULE_NOMINAL_VOLTAGE_OPERABLE, - fapi2::MSS_VOLT_DDR_TYPE_REQUIRED_VOLTAGE(). - set_DIMM_VOLTAGE(l_module_nom_voltage[l_port_num][l_dimm_num]), - "%s.Failed conditional, data returned:o %d.", - mss::c_str(l_dimm), - l_module_nom_voltage[l_port_num][l_dimm_num] ); - } - } - -fapi_try_exit: - return fapi2::current_err; - -}// module_nominal_voltage - - /// /// @brief Checks conditional and implements traces & exits if it fails /// @param[in] i_target the dimm target |