> -----Original Message-----
> From: Tomasz Duszynski [mailto:[email protected]]
> Sent: Friday, April 27, 2018 8:15 AM
> To: [email protected]
> Cc: De Lara Guarch, Pablo <[email protected]>;
> [email protected]; Tomasz Duszynski <[email protected]>; Natalie Samsonov
> <[email protected]>
> 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 <[email protected]>
> Signed-off-by: Tomasz Duszynski <[email protected]>
> ---

...

> --- 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