[dpdk-dev] [PATCH v2 2/7] net/txgbe: add device arguments
Ferruh Yigit
ferruh.yigit at intel.com
Wed Mar 24 12:01:29 CET 2021
On 3/24/2021 10:59 AM, Ferruh Yigit wrote:
> On 3/16/2021 10:51 AM, Jiawen Wu wrote:
>> Add device arguments to support runtime options. These arguments are
>> only set for backplane NICs.
>>
>> Signed-off-by: Jiawen Wu <jiawenwu at trustnetic.com>
>
> <...>
>
>> +
>> +#define TXGBE_DEVARG_BP_AUTO "auto"
>> +#define TXGBE_DEVARG_KR_POLL "poll"
>> +#define TXGBE_DEVARG_KR_PRESENT "present"
>> +#define TXGBE_DEVARG_KX_SGMII "sgmii"
>> +#define TXGBE_DEVARG_FFE_SET "ffe_set"
>> +#define TXGBE_DEVARG_FFE_MAIN "ffe_main"
>> +#define TXGBE_DEVARG_FFE_PRE "ffe_pre"
>> +#define TXGBE_DEVARG_FFE_POST "ffe_post"
>> +
>
> I just recognized that this patch is adding devargs without using them, and
> later patches are using them. This separates them from the full context.
>
> Can you please add them gradually as they are used, in the same patch they are
> used?
>
> Meanwhile you can add the release notes update in next version.
>
> Thanks,
> ferruh
And I think 'auto' can be 'auto_neg', since 'auto' is too generic, but up to you.
More information about the dev
mailing list