Skip to content

Commit

Permalink
Add 100ms wait to disable write protection for BPM
Browse files Browse the repository at this point in the history
There was an issue where without waiting after unlocking the BPM
registers we'd fail out of the BPM update. This commit adds a 100ms
delay after writing the unlock sequence before reading to verify that
the registers were unlocked.

Change-Id: Ic8cfeb8dba181d4071bb2d4dccaf367d88840cc2
CQ:SW480382
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86707
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
  • Loading branch information
MRaybuck authored and dcrowell77 committed Nov 11, 2019
1 parent 4e071c3 commit 7ddeb4d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/usr/isteps/nvdimm/bpm_update.C
Expand Up @@ -2302,6 +2302,8 @@ errlHndl_t Bpm::disableWriteProtection()
break;
}

nanosleep(0, 100 * NS_PER_MSEC);

// Make sure protection was removed
uint8_t data = 0;
errl = nvdimmReadReg(iv_nvdimm,
Expand Down

0 comments on commit 7ddeb4d

Please sign in to comment.