On Wed, 3 Aug 2022 19:30:54 +0800 Junfeng Guo <junfeng....@intel.com> wrote:
> +
> +static int
> +idpf_dev_configure(struct rte_eth_dev *dev)
> +{
> + struct idpf_vport *vport =
> + (struct idpf_vport *)dev->data->dev_private;
Cast of void pointer dev_private is unnecessary in C (only in C++ is it needed).
> + int ret = 0;
Useless initialization, you are setting in next line.
> +
> + ret = idpf_init_vport_req_info(dev);
> + if (ret) {
