Hernan, 

Please be more explicit on the commit description in the next patch version, 
suggesting below:

The computation of the HARQ output length should not include the filler bits on 
the N3000 HW implementation as these are already taken out by the 
deratematching step.
The change is only an optimization (no functional impact), no change required 
on stable branches.

Thanks
Nic

> -----Original Message-----
> From: Akhil Goyal <gak...@marvell.com>
> Sent: Monday, May 16, 2022 11:07 AM
> To: Vargas, Hernan <hernan.var...@intel.com>; dev@dpdk.org;
> t...@redhat.com
> Cc: Chautru, Nicolas <nicolas.chau...@intel.com>; Zhang, Qi Z
> <qi.z.zh...@intel.com>
> Subject: RE: [EXT] [PATCH v1 5/5] baseband/fpga_5gnr_fec: remove filler
> from HARQ
> 
> > Removed dec->n_filler from harq_out_length calculation.
> >
> Please provide a better description describing the reason to remove it.

Reply via email to