Skip to content

Commit

Permalink
Lengthen polling duration for Explorer outbound doorbell
Browse files Browse the repository at this point in the history
Change-Id: Ia8f009291481ccb2202be679d49f0652a56b81c5
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80221
Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com>
Reviewed-by: RYAN P. KING <rpking@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@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: STEPHEN GLANCY <sglancy@us.ibm.com>
Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80249
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 Jul 19, 2019
1 parent 66af054 commit dd6aa0f
Showing 1 changed file with 2 additions and 2 deletions.
Expand Up @@ -541,9 +541,9 @@ 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 ~10ms in HW.
// We initially delay 8ms, so we should only need to poll for ~2ms here.
// We're waiting 100us between polls, so we should only need about 20 loops here,
// but we make it 50 to be safe
// but we make it 200 to be safe
constexpr uint64_t NUM_LOOPS = 200;

constexpr uint64_t NUM_LOOPS = 50;
// 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
// As such, we don't want to pull in more libraries than we need to: it would cause extra dependencies
Expand Down

0 comments on commit dd6aa0f

Please sign in to comment.