diff options
| author | Benjamin Weisenbeck <bweisenb@us.ibm.com> | 2017-12-04 14:58:52 -0600 |
|---|---|---|
| committer | Zane C. Shelley <zshelle@us.ibm.com> | 2017-12-11 17:21:47 -0500 |
| commit | 9b826df04888393a5417b9a790ad748f0462dca9 (patch) | |
| tree | c2cd0d99c49ae4bacb1fa7f6f17f49d7b2aac235 /src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C | |
| parent | f74d2c56712b6a2ccfce256885145b6867826059 (diff) | |
| download | talos-hostboot-9b826df04888393a5417b9a790ad748f0462dca9.tar.gz talos-hostboot-9b826df04888393a5417b9a790ad748f0462dca9.zip | |
PRD: Fix missing clear of pll errors
Change-Id: Ie3bebc8dc3c315dc4dd148649d5786cb4534a26b
CQ: SW409500
Backport: release-fips910
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50643
Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50678
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/50675
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>
Diffstat (limited to 'src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C')
| -rw-r--r-- | src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C b/src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C index 472eaae1f..2d0dd36f8 100644 --- a/src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C +++ b/src/usr/diag/prdf/common/plat/p9/prdfP9PllDomain.C @@ -189,9 +189,12 @@ int32_t PllDomain::Analyze(STEP_CODE_DATA_STRUCT & serviceData, } // end for each chip in domain // Remove all non-functional chips. - for ( auto i : nfchips ) + if ( CHECK_STOP != serviceData.service_data->getPrimaryAttnType() ) { - systemPtr->RemoveStoppedChips( i->getTrgt() ); + for ( auto i : nfchips ) + { + systemPtr->RemoveStoppedChips( i->getTrgt() ); + } } // TODO: RTC 155673 - use attributes to callout active clock sources @@ -311,6 +314,11 @@ int32_t PllDomain::Analyze(STEP_CODE_DATA_STRUCT & serviceData, (serviceData, mskErrType)); } + // Clear PLLs from this domain. + ExtensibleDomainFunction * l_clear = getExtensibleFunction("ClearPll"); + (*l_clear)(this, + PluginDef::bindParm<STEP_CODE_DATA_STRUCT&>(serviceData)); + // Run PLL Post Analysis on any analyzed chips in this domain. for(auto l_chip : sysRefList) { |
