> From: Sevincer, Abdullah [mailto:abdullah.sevin...@intel.com]
> Sent: Thursday, 9 February 2023 19.50
> 
> Acked: by abdullah.sevin...@intel.com

Thanks.

Patchwork didn't catch it due to formatting, but the point is obvious:

Acked-by: Abdullah Sevincer <abdullah.sevin...@intel.com>

> 
> -----Original Message-----
> From: Morten Brørup <m...@smartsharesystems.com>
> Sent: Thursday, February 9, 2023 8:51 AM
> To: dev@dpdk.org; roret...@linux.microsoft.com; rm...@marvell.com;
> McDaniel, Timothy <timothy.mcdan...@intel.com>; ma...@nvidia.com;
> viachesl...@nvidia.com
> Cc: ruifeng.w...@arm.com; zhou...@loongson.cn; d...@linux.vnet.ibm.com;
> k...@semihalf.com; Richardson, Bruce <bruce.richard...@intel.com>;
> konstantin.v.anan...@yandex.ru
> Subject: RE: [PATCH v7 2/4] event/dlb2: remove superfluous rte_memcpy
> 
> PING dlb2 maintainers. Care to review this, so it can be included in
> 23.03?
> 
> > From: Morten Brørup [mailto:m...@smartsharesystems.com]
> > Sent: Monday, 16 January 2023 14.07
> >
> > Copying with the same src and dst address has no effect; removed to
> > avoid compiler warning with decorated rte_memcpy.
> >
> > Fixes: e7c9971a857a ("event/dlb2: add probe-time hardware init")
> > Cc: timothy.mcdan...@intel.com
> >
> > Signed-off-by: Morten Brørup <m...@smartsharesystems.com>
> > Acked-by: Stephen Hemminger <step...@networkplumber.org>
> > ---
> > v7:
> > * No changes.
> > v6:
> > * Add Fixes to patch description. (Stephen)
> > v5:
> > * First patch in series.
> > ---
> >  drivers/event/dlb2/dlb2.c | 3 ---
> >  1 file changed, 3 deletions(-)
> >
> > diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c
> > index 60c5cd4804..03d32c779f 100644
> > --- a/drivers/event/dlb2/dlb2.c
> > +++ b/drivers/event/dlb2/dlb2.c
> > @@ -215,7 +215,6 @@ static int
> >  dlb2_hw_query_resources(struct dlb2_eventdev *dlb2)  {
> >     struct dlb2_hw_dev *handle = &dlb2->qm_instance;
> > -   struct dlb2_hw_resource_info *dlb2_info = &handle->info;
> >     int num_ldb_ports;
> >     int ret;
> >
> > @@ -277,8 +276,6 @@ dlb2_hw_query_resources(struct dlb2_eventdev
> *dlb2)
> >     handle->info.hw_rsrc_max.reorder_window_size =
> >             dlb2->hw_rsrc_query_results.num_hist_list_entries;
> >
> > -   rte_memcpy(dlb2_info, &handle->info.hw_rsrc_max,
> > sizeof(*dlb2_info));
> > -
> >     return 0;
> >  }
> >
> > --
> > 2.17.1

Reply via email to