> -----Original Message-----
> From: Tomasz Duszynski [mailto:t...@semihalf.com]
> Sent: Friday, April 27, 2018 8:15 AM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>;
> d...@marvell.com; Tomasz Duszynski <t...@semihalf.com>; Natalie Samsonov
> <nsams...@marvell.com>
> Subject: [PATCH v2] crypto/mrvl: rename PMD to mvsam
> 
> Picking a company stock ticker for a PMD name might not be a best approach in
> a long run since name is too generic.
> 
> This patch addresses that and renames mrvl to mvsam.
> 
> Signed-off-by: Natalie Samsonov <nsams...@marvell.com>
> Signed-off-by: Tomasz Duszynski <t...@semihalf.com>
> ---
...

> --- a/test/test/test_cryptodev.h
> +++ b/test/test/test_cryptodev.h
> @@ -61,7 +61,7 @@
>  #define CRYPTODEV_NAME_DPAA_SEC_PMD  crypto_dpaa_sec
>  #define CRYPTODEV_NAME_DPAA2_SEC_PMD crypto_dpaa2_sec
>  #define CRYPTODEV_NAME_SCHEDULER_PMD crypto_scheduler
> -#define CRYPTODEV_NAME_MRVL_PMD              crypto_mrvl
> +#define CRYPTODEV_NAME_MRVL_PMD              crypto_mvsam

I think it would be better if you modify the macro names in the test app, to be 
MVSAM.

>  #define CRYPTODEV_NAME_CCP_PMD               crypto_ccp
>  #define CRYPTODEV_NAME_VIRTIO_PMD    crypto_virtio
> 
> --
> 2.7.4

Reply via email to