summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCHRISTINA L. GRAVES <clgraves@us.ibm.com>2016-09-01 11:05:58 -0500
committerSachin Gupta <sgupta2m@in.ibm.com>2016-09-07 00:20:54 -0400
commitebdc059e5a24ee3d0555b3a47c467c4032c4f3ad (patch)
treea459daf57c1e380379b294f4d4bdf956d5d16cdf
parent9769074bce53085855928a50d96912e1b6497438 (diff)
downloadtalos-sbe-ebdc059e5a24ee3d0555b3a47c467c4032c4f3ad.tar.gz
talos-sbe-ebdc059e5a24ee3d0555b3a47c467c4032c4f3ad.zip
Adding in reset at the beginning of pba_setup
Change-Id: I03b9fd8a3dbd8a55cafd8a6b7adbc1ee3b819dc0 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29101 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29158 Reviewed-by: Hostboot Team <hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
-rw-r--r--src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C11
1 files changed, 6 insertions, 5 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C b/src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C
index 32df239e..ad47890d 100644
--- a/src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C
+++ b/src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C
@@ -1,7 +1,7 @@
/* IBM_PROLOG_BEGIN_TAG */
/* This is an automatically generated prolog. */
/* */
-/* $Source: import/chips/p9/procedures/hwp/nest/p9_pba_setup.C $ */
+/* $Source: src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C $ */
/* */
/* OpenPOWER sbe Project */
/* */
@@ -68,10 +68,11 @@ extern "C"
FAPI_TRY(p9_pba_coherent_utils_check_fbc_state(i_target),
"Error from p9_pba_coherent_utils_check_fbc_state");
- //reset the ADU - cleanup just calls reset
- //TODO Joe had made a comment on this about resetting the PBA as part of the setup process - I need to test
- //this more before I'm willing to actually put this in - I got some errors when I had it in earlier.
- //FAPI_TRY(p9_pba_coherent_cleanup_pba(i_target), "p9_pba_setup: Error from p9_pba_cleanup_pba");
+ //reset the PBA - cleanup just calls reset
+ FAPI_TRY(p9_pba_coherent_cleanup_pba(i_target), "Error from p9_pba_cleanup_pba");
+
+ //Check to make sure that it's ready for another write
+ FAPI_TRY(p9_pba_coherent_status_check(i_target), "Error from p9_pba_coherent_status_check");
//The PBA Bar and PBA Bar Mask need to be setup before getting the number of granules because how they get setup affects the number of granules that can be read/written
//setup the PBA Bar
OpenPOWER on IntegriCloud