Hi Naga/Talluri,

> -----Original Message-----
> From: Somarowthu, Naga SureshX
> Sent: Tuesday, June 12, 2018 7:51 AM
> To: [email protected]
> Cc: De Lara Guarch, Pablo <[email protected]>; Pattan, Reshma
> <[email protected]>; Chaitanya Babu, TalluriX
> <[email protected]>
> Subject: [PATCH] crypto/aesni_gcm: add dynamic logging to aesni_gcm
> 
> From: tchaitax <[email protected]>
> 
> 1.added new logtype for aesni_gcm driver.
> 2.registered new logtype.
> 3.GCM_LOG_ERR and CDEV_LOG_ERR are replaced with
>   new logtype name AESNI_GCM_PMD_LOG.
> 
> Signed-off-by: Talluri Chaitanya <[email protected]>
> Reviewed-by: Reshma Pattan <[email protected]>

Could you make the following changes?

- Change the macro to AESNI_GCM_LOG, to make it shorter.
- Change title to: crypto/aesni_gcm: add dynamic logging
- Change the author name to "Talluri Chaitanya"
- Remove unused "CONFIG_RTE_LIBRTE_PMD_AESNI_GCM_DEBUG=n"
from config file.
- Set log level of the new type to INFO (see isal_compress_pmd.c)

Thanks,
Pablo

Reply via email to