diff options
| author | Jacob Harvey <jlharvey@us.ibm.com> | 2017-10-12 13:02:26 -0500 |
|---|---|---|
| committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2017-11-02 10:38:07 -0400 |
| commit | cf5d76cbc8911c5ac71fd539b2f5d2c7470d49a0 (patch) | |
| tree | 269575274b8abc45c78e50340e6c3720984b6001 /src/import/chips/p9/procedures/hwp/memory/lib/phy | |
| parent | 7ac94ae277b9a96e921384c93864531c84b53364 (diff) | |
| download | talos-hostboot-cf5d76cbc8911c5ac71fd539b2f5d2c7470d49a0.tar.gz talos-hostboot-cf5d76cbc8911c5ac71fd539b2f5d2c7470d49a0.zip | |
Set blue waterfall range to 1-4 for all freqs
If red waterfall workaround was ran
Change-Id: I7ae2433e1e5aaa7f0f0f77455a78f411c1768ef3
CQ: SW404923
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48313
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Dev-Ready: JACOB L. HARVEY <jlharvey@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@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: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48327
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: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/lib/phy')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/phy/dp16.C | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/phy/dp16.C b/src/import/chips/p9/procedures/hwp/memory/lib/phy/dp16.C index e41461bb1..89bf21328 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/phy/dp16.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/phy/dp16.C @@ -2495,6 +2495,10 @@ fapi2::ReturnCode reset_drift_limits( const fapi2::Target<TARGET_TYPE_MCA>& i_ta blue_waterfall_range::TWO_TO_FIVE : blue_waterfall_range::ONE_TO_FOUR; + // So if the red waterfall workaround was/ is going to run, we have to change the blue waterfall range + // This is freq independent + mss::workarounds::dp16::update_blue_waterfall_extend_range( i_target, l_value); + //From John Bialas : the value is good for hardware, but not so much in sim, in sim they should be 0s - bgass - 10/6/2017 l_value = (l_sim == 1) ? blue_waterfall_range::ZERO_TO_THREE : l_value; |
