Skip to content

Commit

Permalink
Increase exp_draminit polling timeout for MDS
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
stermole authored and dcrowell77 committed Mar 9, 2020
1 parent 3a26734 commit f40c41f
Showing 1 changed file with 5 additions and 4 deletions.
Expand Up @@ -5,7 +5,7 @@
/* */
/* OpenPOWER HostBoot Project */
/* */
/* Contributors Listed Below - COPYRIGHT 2019 */
/* Contributors Listed Below - COPYRIGHT 2019,2020 */
/* [+] International Business Machines Corp. */
/* */
/* */
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit f40c41f

Please sign in to comment.