RE: [PATCH v8 07/12] vdpa/ifc: change internal function name

[Pei, Andy]

Hi Maxime,

Thanks for your efforts.

> -----Original Message-----
> From: Maxime Coquelin <maxime.coque...@redhat.com>
> Sent: Tuesday, October 18, 2022 9:45 PM
> To: Pei, Andy <andy....@intel.com>; dev@dpdk.org
> Cc: Xia, Chenbo <chenbo....@intel.com>; Xu, Rosen <rosen...@intel.com>;
> Huang, Wei <wei.hu...@intel.com>; Cao, Gang <gang....@intel.com>
> Subject: Re: [PATCH v8 07/12] vdpa/ifc: change internal function name
> 
> 
> 
> On 10/18/22 14:07, Andy Pei wrote:
> > Change internal function name "find_internal_resource_by_dev"
> > to "find_internal_resource_by_pci_dev".
> >
> > Signed-off-by: Andy Pei <andy....@intel.com>
> > ---
> >   drivers/vdpa/ifc/ifcvf_vdpa.c | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/vdpa/ifc/ifcvf_vdpa.c
> > b/drivers/vdpa/ifc/ifcvf_vdpa.c index 9c49f9c..73d04ed 100644
> > --- a/drivers/vdpa/ifc/ifcvf_vdpa.c
> > +++ b/drivers/vdpa/ifc/ifcvf_vdpa.c
> > @@ -121,7 +121,7 @@ struct rte_vdpa_dev_info {
> >   }
> >
> >   static struct internal_list *
> > -find_internal_resource_by_dev(struct rte_pci_device *pdev)
> > +find_internal_resource_by_pci_dev(struct rte_pci_device *pdev)
> >   {
> >     int found = 0;
> >     struct internal_list *list;
> > @@ -1746,7 +1746,7 @@ struct rte_vdpa_dev_info dev_info[] = {
> >     if (rte_eal_process_type() != RTE_PROC_PRIMARY)
> >             return 0;
> >
> > -   list = find_internal_resource_by_dev(pci_dev);
> > +   list = find_internal_resource_by_pci_dev(pci_dev);
> >     if (list == NULL) {
> >             DRV_LOG(ERR, "Invalid device: %s", pci_dev->name);
> >             return -1;
> 
> Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com>
> 
> Thanks,
> Maxime