diff options
author | Zane Shelley <zshelle@us.ibm.com> | 2018-05-11 20:38:29 -0500 |
---|---|---|
committer | Zane C. Shelley <zshelle@us.ibm.com> | 2018-05-18 10:39:31 -0400 |
commit | 5370984de38992719a693391a8c19444eea80f0b (patch) | |
tree | ce9bb3ee97689cd2d672eef8c3159491d8e62ae5 /src | |
parent | 4e78db3c12dfe447bbb2536aa5e2cf5519b49770 (diff) | |
download | talos-hostboot-5370984de38992719a693391a8c19444eea80f0b.tar.gz talos-hostboot-5370984de38992719a693391a8c19444eea80f0b.zip |
PRD: Remove ForceRead() from MemAddr utils
After manually stopping background scrubbing, we refresh all the
required registers. There is no need to do the ForceRead() now in the
MemAddr utility functions anymore.
Change-Id: I7c2cd53790c1d042bfdeff2c746d6b16a3db36e8
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/58809
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com>
Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com>
Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59010
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/usr/diag/prdf/common/plat/mem/prdfMemAddress.C | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/src/usr/diag/prdf/common/plat/mem/prdfMemAddress.C b/src/usr/diag/prdf/common/plat/mem/prdfMemAddress.C index d4e5a99d3..5893d6dc9 100644 --- a/src/usr/diag/prdf/common/plat/mem/prdfMemAddress.C +++ b/src/usr/diag/prdf/common/plat/mem/prdfMemAddress.C @@ -245,10 +245,8 @@ uint32_t getMemMaintAddr<TYPE_MCBIST>( ExtensibleChip * i_chip, PRDF_ASSERT( TYPE_MCBIST == i_chip->getType() ); // Read the address register - // We need to use ForceRead() here because it is possible we needed to stop - // the maintenance command after capturing the initial register data. SCAN_COMM_REGISTER_CLASS * reg = i_chip->getRegister( "MCBMCAT" ); - uint32_t o_rc = reg->ForceRead(); + uint32_t o_rc = reg->Read(); if ( SUCCESS != o_rc ) { PRDF_ERR( PRDF_FUNC "Read() failed on MCBMCAT: i_chip=0x%08x", @@ -296,13 +294,11 @@ uint32_t getMemMaintAddr<TYPE_MBA>( ExtensibleChip * i_chip, MemAddr & o_addr ) PRDF_ASSERT( TYPE_MBA == i_chip->getType() ); // Read the address register - // We need to use ForceRead() here because it is possible we needed to stop - // the maintenance command after capturing the initial register data. SCAN_COMM_REGISTER_CLASS * reg = i_chip->getRegister( "MBMACA" ); - uint32_t o_rc = reg->ForceRead(); + uint32_t o_rc = reg->Read(); if ( SUCCESS != o_rc ) { - PRDF_ERR( PRDF_FUNC "Read() failed on MCBMCAT: i_chip=0x%08x", + PRDF_ERR( PRDF_FUNC "Read() failed on MBMACA: i_chip=0x%08x", i_chip->getHuid() ); } else @@ -334,17 +330,13 @@ uint32_t getMcbistMaintPort( ExtensibleChip * i_mcbChip, o_mcaList.clear(); - // Note that we will need to use ForceRead() on these registers because it is - // possible we needed to stop the maintenance command after capturing the - // initial register data. - SCAN_COMM_REGISTER_CLASS * mcbagra = i_mcbChip->getRegister( "MCBAGRA" ); SCAN_COMM_REGISTER_CLASS * mcbmcat = i_mcbChip->getRegister( "MCBMCAT" ); SCAN_COMM_REGISTER_CLASS * mcb_cntl = i_mcbChip->getRegister( "MCB_CNTL" ); do { - o_rc = mcbagra->ForceRead(); + o_rc = mcbagra->Read(); if ( SUCCESS != o_rc ) { PRDF_ERR( PRDF_FUNC "Read() failed on MCBAGRA: i_mcbChip=0x%08x", @@ -358,7 +350,7 @@ uint32_t getMcbistMaintPort( ExtensibleChip * i_mcbChip, uint8_t portMask = 0; if ( !mcbagra->IsBitSet(10) || mcbagra->IsBitSet(11) ) // broadcast mode { - o_rc = mcb_cntl->ForceRead(); + o_rc = mcb_cntl->Read(); if ( SUCCESS != o_rc ) { PRDF_ERR( PRDF_FUNC "Read() failed on MCB_CNTL: " @@ -370,7 +362,7 @@ uint32_t getMcbistMaintPort( ExtensibleChip * i_mcbChip, } else // non-broadcast mode { - o_rc = mcbmcat->ForceRead(); + o_rc = mcbmcat->Read(); if ( SUCCESS != o_rc ) { PRDF_ERR( PRDF_FUNC "Read() failed on MCBMCAT: " |