diff options
author | Prachi Gupta <pragupta@us.ibm.com> | 2015-06-17 09:22:10 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2015-07-21 09:42:07 -0500 |
commit | b5adff9f2c3862cb1b6ef9add4a6bbeee0824f77 (patch) | |
tree | a6da0d27588d755a4eac0d9a83cf373e1a126abd /src/usr/hwpf/hwp/mc_config | |
parent | 8de824a15fe9d97fc169b262c884e09f6e59912a (diff) | |
download | talos-hostboot-b5adff9f2c3862cb1b6ef9add4a6bbeee0824f77.tar.gz talos-hostboot-b5adff9f2c3862cb1b6ef9add4a6bbeee0824f77.zip |
SW311560: HWP error prevent IPL on habanero
Change-Id: Ibd19f1986c768e08a97acdbd8b39c174f51bb399
CQ:SW311560
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18555
Reviewed-by: Michael D. Pardeik <pardeik@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: PRACHI GUPTA <pragupta@us.ibm.com>
Tested-by: PRACHI GUPTA <pragupta@us.ibm.com>
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18556
Tested-by: Jenkins Server
Tested-by: Jenkins OP Build CI
Tested-by: Jenkins OP HW
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/usr/hwpf/hwp/mc_config')
-rw-r--r-- | src/usr/hwpf/hwp/mc_config/mss_eff_config/mss_eff_config_thermal.C | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/src/usr/hwpf/hwp/mc_config/mss_eff_config/mss_eff_config_thermal.C b/src/usr/hwpf/hwp/mc_config/mss_eff_config/mss_eff_config_thermal.C index 1f1bb7e63..996a1e0f6 100644 --- a/src/usr/hwpf/hwp/mc_config/mss_eff_config/mss_eff_config_thermal.C +++ b/src/usr/hwpf/hwp/mc_config/mss_eff_config/mss_eff_config_thermal.C @@ -22,7 +22,7 @@ /* permissions and limitations under the License. */ /* */ /* IBM_PROLOG_END_TAG */ -// $Id: mss_eff_config_thermal.C,v 1.31 2015/04/06 22:33:11 pardeik Exp $ +// $Id: mss_eff_config_thermal.C,v 1.32 2015/06/16 21:57:30 pardeik Exp $ // $Source: /afs/awd/projects/eclipz/KnowledgeBase/.cvsroot/eclipz/chips/ // centaur/working/procedures/ipl/fapi/mss_eff_config_thermal.C,v $ //------------------------------------------------------------------------------ @@ -55,6 +55,9 @@ //------------------------------------------------------------------------------ // Version:| Author: | Date: | Comment: //---------|----------|---------|----------------------------------------------- +// 1.32 | pardeik | 06/16/15| fix for ISDIMM systems to prevent a zero +// | | | ATTR_MSS_MEM_WATT_TARGET value +// | | | Removed unneeded TODO commented section // 1.31 | pardeik | 04/06/15 | attribute name changed for adjustment enable // 1.30 | pardeik |12-FEB-15| CDIMM DDR4 throttle updates (set Nmba to Nchip) // | | | Support for vmem regulator power adjustment @@ -131,10 +134,6 @@ // | | | case. // 1.1 | pardeik |01-NOV-11| First Draft. -/* -TODO ITEMS: -1. Update ISDIMM power table after hardware measurements are done (GA3) -*/ //------------------------------------------------------------------------------ // My Includes @@ -583,6 +582,11 @@ extern "C" { //------------------------------------------------------------------------------ // adjust the regulator power limit per dimm if enabled and use this if less than the thermal limit +// If reg power limit is zero, then set to thermal limit - needed for ISDIMM systems since some of these MRW attributes are not defined + if (l_dimm_reg_power_limit_per_dimm == 0) + { + l_dimm_reg_power_limit_per_dimm = dimm_thermal_power_limit; + } l_dimm_reg_power_limit_per_dimm_adj = l_dimm_reg_power_limit_per_dimm; if (l_dimm_reg_power_limit_adj_enable == fapi::ENUM_ATTR_MRW_VMEM_REGULATOR_POWER_LIMIT_PER_DIMM_ADJ_ENABLE_TRUE) { |