Thanks Pablo for the fix. I am ok with this change Acked-by: Vipin Varghese <vipin.vargh...@intel.com>
> -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Pablo de Lara > Sent: Friday, May 18, 2018 6:54 PM > To: Doherty, Declan <declan.dohe...@intel.com> > Cc: dev@dpdk.org; De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; > sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] cryptodev: add missing feature string > > Security protocol flag string was not added when the actual flag was added. > > Fixes: eadb4fa1e1fe ("cryptodev: support security APIs") > Cc: sta...@dpdk.org > > Signed-off-by: Pablo de Lara <pablo.de.lara.gua...@intel.com> > --- > lib/librte_cryptodev/rte_cryptodev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/lib/librte_cryptodev/rte_cryptodev.c > b/lib/librte_cryptodev/rte_cryptodev.c > index 2a95a351f..3712caf73 100644 > --- a/lib/librte_cryptodev/rte_cryptodev.c > +++ b/lib/librte_cryptodev/rte_cryptodev.c > @@ -346,6 +346,8 @@ rte_cryptodev_get_feature_name(uint64_t flag) > return "CPU_NEON"; > case RTE_CRYPTODEV_FF_CPU_ARM_CE: > return "CPU_ARM_CE"; > + case RTE_CRYPTODEV_FF_SECURITY: > + return "SECURITY_PROTOCOL"; > default: > return NULL; > } > -- > 2.17.0
