diff options
author | Brian Stegmiller <bjs@us.ibm.com> | 2019-01-16 10:39:19 -0600 |
---|---|---|
committer | Zane C. Shelley <zshelle@us.ibm.com> | 2019-01-22 09:38:53 -0600 |
commit | 9c50b31d9770a16a13a89f23075ae45c077400c2 (patch) | |
tree | bbf199d7e6278e878d2afaf4da813a2f47316ece /src/usr/diag/prdf | |
parent | 17ba81ec0a525707be8539d12ec0e2050227c354 (diff) | |
download | talos-hostboot-9c50b31d9770a16a13a89f23075ae45c077400c2.tar.gz talos-hostboot-9c50b31d9770a16a13a89f23075ae45c077400c2.zip |
PRD: Allow UCS and HOST broadcast reads for hostboot
Change-Id: I1bb092ee10b6b044e9bfa3066c8bf20ff06e8681
CQ:SW441374
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70541
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com>
Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70703
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')
-rw-r--r-- | src/usr/diag/prdf/common/plat/p9/prdfP9Proc.C | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/src/usr/diag/prdf/common/plat/p9/prdfP9Proc.C b/src/usr/diag/prdf/common/plat/p9/prdfP9Proc.C index 31afc89df..00a2ae78a 100644 --- a/src/usr/diag/prdf/common/plat/p9/prdfP9Proc.C +++ b/src/usr/diag/prdf/common/plat/p9/prdfP9Proc.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2016,2018 */ +/* Contributors Listed Below - COPYRIGHT 2016,2019 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -309,20 +309,14 @@ PRDF_PLUGIN_DEFINE_NS( nimbus_proc, Proc, checkNotNimbusDD10 ); int32_t isHostAttnFirAccessible(ExtensibleChip * i_chip, bool & o_isOkToAccess) { - o_isOkToAccess = false; - // Host Processor side can always access the 'host' attn reg // The FSP can not access it during IPL steps 15 thru 16.2 // Host attn is only needed for MS diag and runtime case. + o_isOkToAccess = atRuntime(); - if ( (true == atRuntime()) #ifdef __HOSTBOOT_MODULE - || (true == isInMdiaMode()) + o_isOkToAccess = true; #endif - ) - { - o_isOkToAccess = true; - } return SUCCESS; } @@ -338,7 +332,6 @@ int32_t isUcsFirAccessible(ExtensibleChip * i_chip, bool & o_isOkToAccess) // The FSP can not access it during IPL steps 15 thru 16.2 o_isOkToAccess = atRuntime(); - #ifdef CONFIG_ENABLE_CHECKSTOP_ANALYSIS if (false == o_isOkToAccess) { @@ -360,6 +353,14 @@ int32_t isUcsFirAccessible(ExtensibleChip * i_chip, bool & o_isOkToAccess) o_isOkToAccess = true; } } + +#else + +#ifdef __HOSTBOOT_MODULE + // Can read this reg at anytime from hostboot side + o_isOkToAccess = true; +#endif + #endif // CONFIG_ENABLE_CHECKSTOP_ANALYSIS |