diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2017-08-03 22:50:50 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-08-18 00:31:44 -0400 |
commit | b0506146ce9a416ed84c788d94c3ee8886290468 (patch) | |
tree | 23483fd3e5e7210a49f6272c3132a270e78c60e5 /src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H | |
parent | f6df91cc6f5beee527aae6acb180ba20ad9c4269 (diff) | |
download | talos-hostboot-b0506146ce9a416ed84c788d94c3ee8886290468.tar.gz talos-hostboot-b0506146ce9a416ed84c788d94c3ee8886290468.zip |
L3 review and update to p9_mvpd_ring_funcs_errors.xml and
associated files.
Included in this commit:
- Greatly improved FFDC capturing and renaming of RC variables.
- Addition of fapiTarget as parm to several functions.
- Minor cleanup of APIs.
Change-Id: Icb2a2677ddb5c7faa56092542a127d8415066765
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44200
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44201
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H b/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H index f2045936d..b8fae9fd5 100644 --- a/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H +++ b/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.H @@ -47,10 +47,10 @@ enum mvpdRingFuncOp }; typedef ReturnCode (*mvpdRingFuncs_FP_t) ( + const Target<TARGET_TYPE_PROC_CHIP>&, mvpdRingFuncOp, MvpdRecord, MvpdKeyword, - const Target<TARGET_TYPE_PROC_CHIP>&, const uint8_t, const uint64_t, const uint8_t, @@ -67,28 +67,28 @@ extern "C" * detailed comments on get and set are in the * getMvpdRing.H and setMvpdRing.H and apply here as well. * + * @param i_fapiTarget - cpu target * @param i_mvpdRingFuncOp - indicate get or set via enum mvpdRingFuncOp * @param i_record - Record enumerator * @param i_keyword - Keyword enumerator - * @param i_fapiTarget - cpu target * @param i_chipletId - Chiplet ID * @param i_evenOddMask - Even (0) or odd (1) EX. Disregarded for other chiplets. * @param i_ringId - Ring ID - * @param i_pRingBuf - The buffer to receive or send the ring + * @param o_pRingBuf - The buffer to receive or send the ring * @param io_rRingBufsize - Size of ring / ring buffer * * @return fapi2::ReturnCode - FAPI_RC_SUCCESS if success, * relevant error code for failure. */ - ReturnCode mvpdRingFunc( mvpdRingFuncOp i_mvpdRingFuncOp, + ReturnCode mvpdRingFunc( const Target<TARGET_TYPE_PROC_CHIP>& + i_fapiTarget, + mvpdRingFuncOp i_mvpdRingFuncOp, MvpdRecord i_record, MvpdKeyword i_keyword, - const Target<TARGET_TYPE_PROC_CHIP>& - i_fapiTarget, const uint8_t i_chipletId, const uint8_t i_evenOdd, const uint8_t i_ringId, - uint8_t* i_pRingBuf, + uint8_t* o_pRingBuf, uint32_t& io_rRingBufsize ); } // extern "C" |