From ebdc059e5a24ee3d0555b3a47c467c4032c4f3ad Mon Sep 17 00:00:00 2001 From: "CHRISTINA L. GRAVES" Date: Thu, 1 Sep 2016 11:05:58 -0500 Subject: 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 Tested-by: PPE CI Reviewed-by: RAJA DAS Reviewed-by: Thi N. Tran Reviewed-by: Joseph J. McGill Reviewed-by: Jennifer A. Stofer Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29158 Reviewed-by: Hostboot Team Tested-by: FSP CI Jenkins Reviewed-by: Sachin Gupta --- src/import/chips/p9/procedures/hwp/nest/p9_pba_setup.C | 11 ++++++----- 1 file 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 -- cgit v1.2.1