diff options
| author | Tsung Yeung <tyeung@us.ibm.com> | 2019-09-23 07:14:39 -0500 |
|---|---|---|
| committer | Daniel M Crowell <dcrowell@us.ibm.com> | 2019-09-24 09:21:48 -0500 |
| commit | c7a3665d17be10a9e1188de39449eb3784ed8d13 (patch) | |
| tree | 02856e6124c50048dc235fb5a96510580525df14 | |
| parent | b7c4af200f89786106e986767ed9f293291f3bdf (diff) | |
| download | talos-hostboot-c7a3665d17be10a9e1188de39449eb3784ed8d13.tar.gz talos-hostboot-c7a3665d17be10a9e1188de39449eb3784ed8d13.zip | |
Double the NVDIMM_READY timeout to prevent erroneous error
NVDIMM controller POR or firmware induced reset time may vary
from run-to-run. Doubling the timeout here in case of edge
condition.
Change-Id: I717dfbef9b2903e8bb1e7aed92c28df1fab90033
CQ:SW476597
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84107
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V Swenson <cswenson@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
| -rw-r--r-- | src/usr/isteps/nvdimm/nvdimm.C | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/usr/isteps/nvdimm/nvdimm.C b/src/usr/isteps/nvdimm/nvdimm.C index f38e39ee9..54f631602 100644 --- a/src/usr/isteps/nvdimm/nvdimm.C +++ b/src/usr/isteps/nvdimm/nvdimm.C @@ -494,8 +494,8 @@ errlHndl_t nvdimmReady(Target *i_nvdimm) break; } - // Convert to ms for polling - uint32_t l_nvm_init_time_ms = l_nvm_init_time * MS_PER_SEC; + // Convert to ms for polling and double the value to avoid edge condition + uint32_t l_nvm_init_time_ms = l_nvm_init_time * MS_PER_SEC * 2; uint32_t l_poll = 0; do |
