[dpdk-dev] [PATCH v4 10/20] net/bnxt: remove devargs for stats accumulation

[Ferruh Yigit]

On 11/3/2021 12:52 AM, Ajit Khaparde wrote:
> From: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
> 
> The accumulation of flow counters is not determined by the
> application device arguments anymore. Instead it is now dictated by
> the platform capabilities whether to do software based accumulation or not.
> 
> Signed-off-by: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
> Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
> Reviewed-by: Mike Baucom <michael.baucom at broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khaparde at broadcom.com>

<...>

> @@ -87,7 +87,6 @@ static const struct rte_pci_id bnxt_pci_id_map[] = {
>   	{ .vendor_id = 0, /* sentinel */ },
>   };
>   
> -#define	BNXT_DEVARG_ACCUM_STATS	"accum-stats"
>   #define BNXT_DEVARG_FLOW_XSTAT	"flow-xstat"
>   #define BNXT_DEVARG_MAX_NUM_KFLOWS  "max-num-kflows"
>   #define BNXT_DEVARG_REPRESENTOR	"representor"

Hi Ajit,

Not for this patch, but I recognized that runtime devargs for the driver
is not documented at all.

Can you please have a separate patch to document them in bnxt.rst?