crypto/mvsam: remove crypto end enumerators

Message ID 20200930080157.13759-1-michaelsh@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: akhil goyal
Headers
Series crypto/mvsam: remove crypto end enumerators |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/travis-robot success Travis build: passed

Commit Message

Michael Shamis Sept. 30, 2020, 8:01 a.m. UTC
  From: Michael Shamis <michaelsh@marvell.com>

Remove enumerators RTE_CRYPTO_CIPHER_LIST_END,
RTE_CRYPTO_AUTH_LIST_END, RTE_CRYPTO_AEAD_LIST_END to prevent
some problems that may arise when adding new crypto algorithms.

Signed-off-by: Michael Shamis <michaelsh@marvell.com>
---
 drivers/crypto/mvsam/rte_mrvl_pmd.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
  

Comments

Akhil Goyal Oct. 9, 2020, 7:51 p.m. UTC | #1
> From: Michael Shamis <michaelsh@marvell.com>
> 
> Remove enumerators RTE_CRYPTO_CIPHER_LIST_END,
> RTE_CRYPTO_AUTH_LIST_END, RTE_CRYPTO_AEAD_LIST_END to prevent
> some problems that may arise when adding new crypto algorithms.
> 
> Signed-off-by: Michael Shamis <michaelsh@marvell.com>
> ---
Applied to dpdk-next-crypto

Thanks.
  
Michael Shamis Oct. 9, 2020, 8:28 p.m. UTC | #2
Thanks!


---
Sent from Workspace ONE Boxer<https://whatisworkspaceone.com/boxer>

On 9 October 2020 at 22:51:52 GMT+3, Akhil Goyal <akhil.goyal@nxp.com> wrote:
External Email

----------------------------------------------------------------------
> From: Michael Shamis <michaelsh@marvell.com>
>
> Remove enumerators RTE_CRYPTO_CIPHER_LIST_END,
> RTE_CRYPTO_AUTH_LIST_END, RTE_CRYPTO_AEAD_LIST_END to prevent
> some problems that may arise when adding new crypto algorithms.
>
> Signed-off-by: Michael Shamis <michaelsh@marvell.com>
> ---
Applied to dpdk-next-crypto

Thanks.
  

Patch

diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd.c b/drivers/crypto/mvsam/rte_mrvl_pmd.c
index aaa40dced..bec51c9ff 100644
--- a/drivers/crypto/mvsam/rte_mrvl_pmd.c
+++ b/drivers/crypto/mvsam/rte_mrvl_pmd.c
@@ -67,7 +67,7 @@  __rte_aligned(32);
  * Map of supported cipher algorithms.
  */
 static const
-struct cipher_params_mapping cipher_map[RTE_CRYPTO_CIPHER_LIST_END] = {
+struct cipher_params_mapping cipher_map[] = {
 	[RTE_CRYPTO_CIPHER_NULL] = {
 		.supported = ALGO_SUPPORTED,
 		.cipher_alg = SAM_CIPHER_NONE },
@@ -107,7 +107,7 @@  struct cipher_params_mapping cipher_map[RTE_CRYPTO_CIPHER_LIST_END] = {
  * Map of supported auth algorithms.
  */
 static const
-struct auth_params_mapping auth_map[RTE_CRYPTO_AUTH_LIST_END] = {
+struct auth_params_mapping auth_map[] = {
 	[RTE_CRYPTO_AUTH_NULL] = {
 		.supported = ALGO_SUPPORTED,
 		.auth_alg = SAM_AUTH_NONE },
@@ -156,7 +156,7 @@  struct auth_params_mapping auth_map[RTE_CRYPTO_AUTH_LIST_END] = {
  * Map of supported aead algorithms.
  */
 static const
-struct cipher_params_mapping aead_map[RTE_CRYPTO_AEAD_LIST_END] = {
+struct cipher_params_mapping aead_map[] = {
 	[RTE_CRYPTO_AEAD_AES_GCM] = {
 		.supported = ALGO_SUPPORTED,
 		.cipher_alg = SAM_CIPHER_AES,