From f40c41f816bce84633670cd9ff963d2924652c5d Mon Sep 17 00:00:00 2001
From: Louis Stermole <stermole@us.ibm.com>
Date: Tue, 4 Feb 2020 08:03:31 -0500
Subject: [PATCH] Increase exp_draminit polling timeout for MDS

Change-Id: Iaa58b3f9ffdb830c22f2f8fd7b498b27fd1c98f4
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90831
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Dev-Ready: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90852
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>
---
 .../procedures/hwp/memory/lib/inband/exp_inband.C        | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/inband/exp_inband.C b/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/inband/exp_inband.C
index b871c96828e..0b9a565229b 100644
--- a/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/inband/exp_inband.C
+++ b/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/inband/exp_inband.C
@@ -5,7 +5,7 @@
 /*                                                                        */
 /* OpenPOWER HostBoot Project                                             */
 /*                                                                        */
-/* Contributors Listed Below - COPYRIGHT 2019                             */
+/* Contributors Listed Below - COPYRIGHT 2019,2020                        */
 /* [+] International Business Machines Corp.                              */
 /*                                                                        */
 /*                                                                        */
@@ -517,9 +517,10 @@ fapi2::ReturnCode poll_for_response_ready(const fapi2::Target<fapi2::TARGET_TYPE
 {
     // NUM_LOOPS is based on EXP_FW_DDR_PHY_INIT command, which completes in ~370ms in HW.
     // We initially delay 8ms, so we should only need to poll for ~360ms here.
-    // We're waiting 20ms between polls, so we should only need about 18 loops here,
-    // but we make it 200 to be safe. Max timeout is (200 x 20ms) = 4 seconds
-    constexpr uint64_t NUM_LOOPS = 200;
+    // Update: for MDS parts training takes on the order of 23 seconds with UART attached.
+    // We're waiting 20ms between polls so we poll 1300 times.
+    // Max timeout is (1300 x 20ms) = 26 seconds
+    constexpr uint64_t NUM_LOOPS = 1300;
 
     // So, why aren't we using the memory team's polling API?
     // This is a base function that will be utilized by the platform code