On 13.07.2017 14:14, Bill Fischofer wrote: > On Thu, Jul 13, 2017 at 5:55 AM, Savolainen, Petri (Nokia - FI/Espoo) > <petri.savolai...@nokia.com> wrote: >> >> >>> -----Original Message----- >>> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >>> Github ODP bot >>> Sent: Wednesday, July 12, 2017 5:00 PM >>> To: lng-odp@lists.linaro.org >>> Subject: [lng-odp] [PATCH API-NEXT v2 4/4] api: crypto: revert deprecation >>> of crypto completion API >>> >>> From: Dmitry Eremin-Solenikov <dmitry.ereminsoleni...@linaro.org> >>> >>> It was decided that it would be benefitable to live with both API types >>> at this point, as odp_crypto_compl_t was available for some time. So >>> undeprecate odp_crypto_compl_t and related functionality. Validation >>> tests also provide necessary tests for pref_mode and for completion >>> event. >>> >>> Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsoleni...@linaro.org> >>> --- >>> /** Email created from pull request 74 (lumag:crypto-packet) >>> ** https://github.com/Linaro/odp/pull/74 >>> ** Patch: https://github.com/Linaro/odp/pull/74.patch >>> ** Base sha: ee5be324411a7520528a367967c28fc529d3bc2e >>> ** Merge commit sha: 5411462e6545fa2d6a286a40c2057db97714ee74 >>> **/ >>> include/odp/api/spec/crypto.h | 38 +++++++---------- >>> --- >>> include/odp/arch/default/api/abi/crypto.h | 4 +-- >>> include/odp/arch/default/api/abi/event.h | 4 +-- >>> .../include/odp/api/plat/crypto_types.h | 3 +- >>> .../include/odp/api/plat/event_types.h | 3 +- >>> platform/linux-generic/odp_crypto.c | 4 --- >>> platform/linux-generic/odp_event.c | 2 -- >>> test/common_plat/performance/odp_crypto.c | 1 + >>> test/common_plat/validation/api/crypto/crypto.c | 2 ++ >>> .../validation/api/crypto/odp_crypto_test_inp.c | 41 >>> +++++++++++++++++++++- >>> .../validation/api/crypto/odp_crypto_test_inp.h | 2 ++ >>> 11 files changed, 61 insertions(+), 43 deletions(-) >>> >>> diff --git a/include/odp/api/spec/crypto.h b/include/odp/api/spec/crypto.h >>> index 3e47f3ef..6736214b 100644 >>> --- a/include/odp/api/spec/crypto.h >>> +++ b/include/odp/api/spec/crypto.h >>> @@ -271,11 +271,8 @@ typedef struct odp_crypto_session_param_t { >>> */ >>> odp_bool_t auth_cipher_text; >>> >>> - /** Preferred sync vs. async >>> - * >>> - * @deprecated no-op now, odp_crypto_operation() will always >>> process >>> - * data in non-posted mode */ >>> - odp_crypto_op_mode_t ODP_DEPRECATE(pref_mode); >>> + /** Preferred sync vs. async for odp_crypto_operation() */ >>> + odp_crypto_op_mode_t pref_mode; >> >> Maybe it makes still sense to leave these @deprecated doxygen tags into >> documentation. So, that we (and user) have some means to follow where goes >> to line between new and old API. Old API documentation should not change, >> but just tag those that are part of the old API and will be removed in >> future. >> >> As agreed, we'll leave out ODP_DEPRECATE() macros in this first phase. Add >> those in next phase, and remove in the last phase. > > +1. We should also update the crypto section of the User Guide to > explain the new crypto APIs and how the intent is to phase out the old > ones. Dmitry: I can post an update patch for this if you'd prefer to > now focus your time on the IPsec development work.
Sure, thanks. I'm not that good in writing documentation. -- With best wishes Dmitry