Re: [dpdk-dev] [PATCH v7 6/6] devargs: parse bus info

[Stephen Hemminger]

On Wed,  5 Jul 2017 01:55:23 +0200
Gaetan Rivet <gaetan.ri...@6wind.com> wrote:

>       case RTE_DEVTYPE_WHITELISTED_PCI:
>       case RTE_DEVTYPE_BLACKLISTED_PCI:
>               /* try to parse pci identifier */
> -             if (eal_parse_pci_BDF(buf, &devargs->pci.addr) != 0 &&
> -                 eal_parse_pci_DomBDF(buf, &devargs->pci.addr) != 0)
> +             if (bus->parse(devname, &devargs->pci.addr) != 0)
>                       goto fail;

Shouldn't these go under bus args for PCI?
It would be good to get all the bus specific args out of generic code.