Hi Jananee/Pallantla, > -----Original Message----- > From: Parthasarathy, JananeeX M > Sent: Friday, June 15, 2018 11:44 AM > To: [email protected] > Cc: De Lara Guarch, Pablo <[email protected]>; Pattan, Reshma > <[email protected]>; Parthasarathy, JananeeX M > <[email protected]>; Poornima, PallantlaX > <[email protected]> > Subject: [PATCH] crypto/null: add dynamic logging to null driver > > 1.added new logtype for null driver. > 2.registered new logtype. > 3.NULL_CRYPTO_LOG_ERR, RTE_LOG_ERR and CDEV_LOG_ERR are replaced > with new logtype name NULL_PMD_LOG. > > Signed-off-by: Pallantla Poornima <[email protected]> > Reviewed-by: Reshma Pattan <[email protected]>
Could you make the following changes? - Change the macro to NULL_LOG, to make it shorter. - Change title to: crypto/null: add dynamic logging - There is a mismatch between the author name and the name in the Signed-off line. Could you fix it? - Set log level of the new type to INFO (see isal_compress_pmd.c) Thanks, Pablo
