[dpdk-dev] [PATCH 1/2] baseband/fpga_5gnr_fec: fix API naming

Maxime Coquelin maxime.coquelin at redhat.com
Mon Oct 5 09:28:06 CEST 2020



On 10/5/20 9:11 AM, Maxime Coquelin wrote:
>>> diff --git a/drivers/baseband/fpga_5gnr_fec/rte_pmd_fpga_5gnr_fec.h b/drivers/baseband/fpga_5gnr_fec/rte_pmd_fpga_5gnr_fec.h
>>> index 70a4acf0b7..c2752fbd52 100644
>>> --- a/drivers/baseband/fpga_5gnr_fec/rte_pmd_fpga_5gnr_fec.h
>>> +++ b/drivers/baseband/fpga_5gnr_fec/rte_pmd_fpga_5gnr_fec.h
>>> @@ -30,7 +30,7 @@ extern "C" {
>>>  /**
>>>   * Structure to pass FPGA 4G FEC configuration.
>> 4G ?
>>
>> There are a number of 4G references in this file's comments.
>>
>> Consider a future patch to clean this up.
> Agree, I noticed that.
> It has to be fixed in another series.
> 

Another thing that we might consider fixing is the naming of the
drivers.

fpga_5gnr & fpga_lte are too generic, it should refer to an Intel
product/family.

Regards,
Maxime



More information about the dev mailing list