Re: [dpdk-dev] [PATCH] drivers: register aliases for renamed cryptodev drivers

["De Lara Guarch, Pablo" <pablo.de.lara.guarch@intel.com>]

> -----Original Message-----
> From: Jan Blunck [mailto:jblunck@gmail.com] On Behalf Of Jan Blunck
> Sent: Friday, October 21, 2016 4:23 AM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo; thomas.monjalon@6wind.com;
> yuanhan.liu@linux.intel.com; Mcnamara, John
> Subject: [PATCH] drivers: register aliases for renamed cryptodev drivers
> 
> This registers the legacy names of the driver being renamed in commit
> 2f45703c17acb943aaded9f79676fd56a72542b2.
> 
> Signed-off-by: Jan Blunck <jblunck@infradead.org>
> ---
>  drivers/crypto/aesni_gcm/aesni_gcm_pmd.c   | 1 +
>  drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 1 +
>  drivers/crypto/kasumi/rte_kasumi_pmd.c     | 1 +
>  drivers/crypto/null/null_crypto_pmd.c      | 1 +
>  drivers/crypto/snow3g/rte_snow3g_pmd.c     | 1 +
>  5 files changed, 5 insertions(+)

Hi Jan,

Please, follow the convention to refer to other commits.
>From checkpatch:

ERROR: Please use git commit description style 'commit <12+ chars of sha1>
("<title line>")' - ie: 'commit 2f45703c17ac ("drivers: make driver names consistent")'

Also, this can probably be merged to the other patch that adds the aliases
for the network drivers, or maybe include it in the same patchset.
I have also tested this, and it works, so you can add:

Tested-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Thanks for the work!
Pablo