> -----Original Message----- > From: McDaniel, Timothy <timothy.mcdan...@intel.com> > Sent: Friday, September 11, 2020 2:19 PM > Cc: dev@dpdk.org; Carrillo, Erik G <erik.g.carri...@intel.com>; Eads, Gage > <gage.e...@intel.com>; Van Haaren, Harry <harry.van.haa...@intel.com>; > jer...@marvell.com > Subject: [PATCH v4 14/22] event/dlb: add eventdev start > > Add support for the eventdev start entry point. > > Signed-off-by: Timothy McDaniel <timothy.mcdan...@intel.com> > --- > drivers/event/dlb/dlb.c | 225 > +++++++++++++++++++++++++------ > drivers/event/dlb/dlb_iface.c | 3 + > drivers/event/dlb/dlb_iface.h | 3 + > drivers/event/dlb/pf/base/dlb_resource.c | 142 +++++++++++++++++++ > drivers/event/dlb/pf/dlb_pf.c | 23 ++++ > 5 files changed, 352 insertions(+), 44 deletions(-) > > diff --git a/drivers/event/dlb/dlb.c b/drivers/event/dlb/dlb.c > index 4f56869..5f2a7fa 100644 > --- a/drivers/event/dlb/dlb.c > +++ b/drivers/event/dlb/dlb.c > @@ -1443,6 +1443,7 @@ dlb_eventdev_ldb_queue_setup(struct rte_eventdev > *dev, > return 0; > } > > + ^^^ Extra newline Same with the dlb2 patch, I think this warrants a detailed commit message. The code looks fine though. With that: Reviewed-by: Gage Eads <gage.e...@intel.com> Thanks, Gage
