summaryrefslogtreecommitdiffstats
path: root/src/import/chips
diff options
context:
space:
mode:
authorBen Gass <bgass@us.ibm.com>2015-11-18 17:20:05 -0600
committerSachin Gupta <sgupta2m@in.ibm.com>2016-10-13 12:52:15 -0400
commit75f63f22ac6f502894c02f91599431d9bb3894bd (patch)
tree5d8398d8989b9ce1521d96fd44fb7f92c464a721 /src/import/chips
parent262bd6f02d372c7ea0e6747d4b30eef46793bb6c (diff)
downloadtalos-sbe-75f63f22ac6f502894c02f91599431d9bb3894bd.tar.gz
talos-sbe-75f63f22ac6f502894c02f91599431d9bb3894bd.zip
Add wrapper fix bug
The check for core or ex target was not working for ex targets. I think == should work for it. Only accept EX targets Change-Id: I2670f6a8971380dbbda6d79aa54c97170e254221 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/22189 Tested-by: Jenkins Server Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31145 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/import/chips')
-rwxr-xr-xsrc/import/chips/p9/procedures/hwp/nest/p9_l2_flush.C44
-rwxr-xr-xsrc/import/chips/p9/procedures/hwp/nest/p9_l2_flush.H9
2 files changed, 7 insertions, 46 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.C b/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.C
index 9d73471d..086e7a3d 100755
--- a/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.C
+++ b/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.C
@@ -80,20 +80,6 @@ enum
//------------------------------------------------------------------------------
///
-/// @brief l2_flush: Utility subroutine to start flush and check the status
-/// @param[in] i_target Ex target
-/// @param[in] i_purgeData Structure having values for MEM, CGC, BANK
-/// passed by the user
-/// @return FAPI2_RC_SUCCESS if purge operation was started,
-/// RC_P9_L2_FLUSH_PURGE_REQ_OUTSTANDING if a prior purge
-/// operation has not yet completed
-/// else FAPI getscom/putscom return code for failing operation
-//------------------------------------------------------------------------------
-fapi2::ReturnCode l2_flush(const fapi2::Target<fapi2::TARGET_TYPE_EX>& i_target,
- const p9core::purgeData_t& i_purgeData);
-
-//------------------------------------------------------------------------------
-///
/// @brief l2_flush_start: Utility subroutine to initiate L2 cache flush
/// via purge engine
/// @param[in] i_target Ex target
@@ -235,8 +221,9 @@ fapi_try_exit:
//------------------------------------------------------------------------------
// Hardware Procedure
//------------------------------------------------------------------------------
-fapi2::ReturnCode l2_flush(const fapi2::Target<fapi2::TARGET_TYPE_EX>& i_target,
- const p9core::purgeData_t& i_purgeData)
+fapi2::ReturnCode p9_l2_flush(const fapi2::Target < fapi2::TARGET_TYPE_EX >
+ & i_target,
+ const p9core::purgeData_t& i_purgeData)
{
FAPI_DBG("i_purgeData.iv_cmdType: 0x%x", i_purgeData.iv_cmdType);
FAPI_DBG("i_purgeData.iv_cmdMem : 0x%x", i_purgeData.iv_cmdMem);
@@ -278,28 +265,3 @@ fapi_try_exit:
return fapi2::current_err;
}
-fapi2::ReturnCode p9_l2_flush(const fapi2::Target < fapi2::TARGET_TYPE_CORE
- | fapi2::TARGET_TYPE_EX >
- &i_target,
- const p9core::purgeData_t& i_purgeData)
-{
- FAPI_IMP("p9_l2_flush, Core|Ex 0x%lx: Enter", i_target.get());
-
- fapi2::Target<fapi2::TARGET_TYPE_EX> l_exTarget;
-
- if(i_target.getType() & fapi2::TARGET_TYPE_CORE)
- {
- l_exTarget = i_target.getParent<fapi2::TARGET_TYPE_EX>();
- }
- else
- {
- // We got an Ex target.
- l_exTarget = i_target.get();
- }
-
- FAPI_TRY(l2_flush(l_exTarget, i_purgeData));
-
-fapi_try_exit:
- FAPI_IMP("p9_l2_flush, Core: Exit");
- return fapi2::current_err;
-}
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.H b/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.H
index dbb6a099..35738cc6 100755
--- a/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.H
+++ b/src/import/chips/p9/procedures/hwp/nest/p9_l2_flush.H
@@ -92,7 +92,7 @@ struct purgeData_t
//------------------------------------------------------------------------------
typedef fapi2::ReturnCode (*p9_l2_flush_FP_t)
-(const fapi2::Target < fapi2::TARGET_TYPE_CORE | fapi2::TARGET_TYPE_EX > &i_target,
+(const fapi2::Target < fapi2::TARGET_TYPE_EX >& i_target,
const p9core::purgeData_t& i_purgeData);
extern "C"
@@ -100,7 +100,7 @@ extern "C"
///
/// @brief p9_l2_flush HWP to flush entire content of L2 cache via purge engine
-/// @param[in] i_target Core target or Ex target
+/// @param[in] i_target Ex target
/// @param[in] i_purgeData Specifies a particular purge type
/// @return: FAPI2_RC_SUCCESS if purge operation completes successfully,
/// RC_P9_L2_FLUSH_UNKNOWN_PLATFORM
@@ -113,9 +113,8 @@ extern "C"
/// if purge operation reports error,
/// else FAPI getscom/putscom return code for failing operation
///
- fapi2::ReturnCode p9_l2_flush(const fapi2::Target < fapi2::TARGET_TYPE_CORE
- | fapi2::TARGET_TYPE_EX >
- &i_target,
+ fapi2::ReturnCode p9_l2_flush(const fapi2::Target < fapi2::TARGET_TYPE_EX >
+ & i_target,
const p9core::purgeData_t& i_purgeData);
} // end of extern C
OpenPOWER on IntegriCloud