16/09/2021 16:01, Bruce Richardson: > On Thu, Sep 16, 2021 at 02:49:57PM +0100, Ferruh Yigit 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? > > Completely agree. Can this be made a runtime option, perhaps passed in as a > devarg if necessary?
+1