[dpdk-dev] [PATCH v3 04/13] net/bnxt: add Thor SRAM mgr model

Ferruh Yigit ferruh.yigit at intel.com
Thu Sep 16 18:30:54 CEST 2021


On 9/16/2021 5:29 PM, Venkat Duvvuru wrote:
> On Thu, Sep 16, 2021 at 7:20 PM Ferruh Yigit <ferruh.yigit at intel.com> wrote:
>>
>> On 9/11/2021 4:30 PM, Venkat Duvvuru wrote:
>>> diff --git a/meson_options.txt b/meson_options.txt
>>> index 0e92734c49..f686e6d92a 100644
>>> --- a/meson_options.txt
>>> +++ b/meson_options.txt
>>> @@ -46,3 +46,5 @@ option('tests', type: 'boolean', value: true, description:
>>>         'build unit tests')
>>>  option('use_hpet', type: 'boolean', value: false, description:
>>>         'use HPET timer in EAL')
>>> +option('bnxt_tf_wc_slices', type: 'integer', min: 1, max: 4, value: 2,
>>> +     description: 'Number of slices per WC TCAM entry')
>>> --
>>
>>
>> Are we sure to add a PMD specific meson option?
>>
>> I believe if we let to go this way, soon the meson options can be as messy as
>> Makefile config options. @Thomas, @Bruce, what do you think?
> 
> Apologies for the confusion.
> It is actually taken care of in the code and this meson_options.txt
> change is not needed.
> Will submit a patch to revert it.
> 

Great, thanks.


More information about the dev mailing list