From 5fc37c5e06243691eb64d2909393828d5b9d1473 Mon Sep 17 00:00:00 2001 From: Stephen Glancy Date: Fri, 20 Jan 2017 13:19:22 -0600 Subject: [PATCH] Fixed periodic cal bug causing data fails A quarter-cycle adjustment signal (blue waterfall) in the PHY was updating erroneously when periodic cal was being run. This update sets a bit that disables the blue waterfall from updating. Change-Id: I82d2ee89f8e0b1628d1640e10a89f39ad353fb1d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35170 Reviewed-by: JACOB L. HARVEY Reviewed-by: Louis Stermole Reviewed-by: Brian R. Silver Tested-by: Jenkins Server Tested-by: Hostboot CI Reviewed-by: Jennifer A. Stofer Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35185 Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell --- .../procedures/hwp/memory/lib/workarounds/dp16_workarounds.C | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/workarounds/dp16_workarounds.C b/src/import/chips/p9/procedures/hwp/memory/lib/workarounds/dp16_workarounds.C index 90a17d239c3..67a15254743 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/workarounds/dp16_workarounds.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/workarounds/dp16_workarounds.C @@ -105,7 +105,9 @@ fapi_try_exit: fapi2::ReturnCode rd_dia_config5( const fapi2::Target& i_target ) { // Config provided by S. Wyatt 8/16 - constexpr uint64_t rd_dia_config = 0x0010; + // 8 corresponds to bit 52 and is a workaround for periodic calibration related fails. + // Currently, it looks like this workaround will be required for all versions of Nimbus, so just hard coding it + constexpr uint64_t rd_dia_config = 0x0810; // Not checking fo EC level as this isn't an EC feature workaround, it's a incorrect documentation workaround. static const std::vector l_addrs =