Hi Reshma, > -----Original Message----- > From: Pattan, Reshma > Sent: Wednesday, July 4, 2018 10:54 AM > To: De Lara Guarch, Pablo <[email protected]>; [email protected] > Cc: De Lara Guarch, Pablo <[email protected]> > Subject: RE: [dpdk-dev] [PATCH] crypto/snow3g: add dynamic logging > > Hi Pablo, > > Also need to remove CONFIG_RTE_LIBRTE_PMD_SNOW3G_DEBUG flag.
I haven't removed it, because it is still used in the data path, so it wasn't used just for logging. Thanks for the review. Pablo > > Thanks, > Reshma > > > -----Original Message----- > > From: Pattan, Reshma > > Sent: Wednesday, July 4, 2018 10:32 AM > > To: 'Pablo de Lara' <[email protected]>; [email protected] > > Cc: De Lara Guarch, Pablo <[email protected]> > > Subject: RE: [dpdk-dev] [PATCH] crypto/snow3g: add dynamic logging > > > > Hi Pablo, > > > > > -----Original Message----- > > > From: dev [mailto:[email protected]] On Behalf Of Pablo de Lara > > > Sent: Tuesday, July 3, 2018 5:37 AM > > > To: [email protected] > > > Cc: De Lara Guarch, Pablo <[email protected]> > > > Subject: [dpdk-dev] [PATCH] crypto/snow3g: add dynamic logging diff > > > --git a/drivers/crypto/snow3g/rte_snow3g_pmd.c > > > b/drivers/crypto/snow3g/rte_snow3g_pmd.c > > > index 72751e35e..bb990a07e 100644 > > > --- a/drivers/crypto/snow3g/rte_snow3g_pmd.c > > > > > +++ b/drivers/crypto/snow3g/rte_snow3g_pmd.c > > > b/drivers/crypto/snow3g/rte_snow3g_pmd_ops.c > > > > Update year in license header, otherwise > > > > Acked-by: Reshma Pattan <[email protected]>

