> -----Original Message-----
> From: Chengwen Feng <fengcheng...@huawei.com>
> Sent: Wednesday, October 9, 2024 12:51 PM
> To: tho...@monjalon.net; ferruh.yi...@amd.com; Chaoyong He
> <chaoyong...@corigine.com>
> Cc: dev@dpdk.org; step...@networkplumber.org
> Subject: [PATCH v6 4/4] common/nfp: use new API to parse kvargs
>
> The nfp_parse_class_options() function could handle both key=value and
> only-key, so it should use rte_kvargs_process_opt() instead of
> rte_kvargs_process() to parse.
>
> Signed-off-by: Chengwen Feng <fengcheng...@huawei.com>
> ---
> drivers/common/nfp/nfp_common_pci.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/common/nfp/nfp_common_pci.c
> b/drivers/common/nfp/nfp_common_pci.c
> index 723035d0f7..5c36052f9d 100644
> --- a/drivers/common/nfp/nfp_common_pci.c
> +++ b/drivers/common/nfp/nfp_common_pci.c
> @@ -170,10 +170,8 @@ nfp_parse_class_options(const struct rte_devargs
> *devargs)
> if (kvargs == NULL)
> return dev_class;
>
> - if (rte_kvargs_count(kvargs, RTE_DEVARGS_KEY_CLASS) != 0) {
> - rte_kvargs_process(kvargs, RTE_DEVARGS_KEY_CLASS,
> - nfp_kvarg_dev_class_handler, &dev_class);
> - }
> + rte_kvargs_process_opt(kvargs, RTE_DEVARGS_KEY_CLASS,
> + nfp_kvarg_dev_class_handler, &dev_class);
>
> rte_kvargs_free(kvargs);
>
> --
> 2.17.1
Looks good to me.
Acked-by: Chaoyong He <chaoyong...@corigine.com>
