Re: [PATCH 4/8] drivers: cleanup devargs lookup in bus scan

Kevin Traynor
2026-03-24T14:13:12Z

On 3/23/26 10:52 AM, David Marchand wrote:
> Don't hardcode the bus names in the RTE_EAL_DEVARGS_FOREACH() calls.
> The bus name is set by code in EAL.
> Even if there is nothing broken, let's reuse the name from the bus object.
>
> And remove the now useless macros.
>
> Note: in the ifpga bus case, the call was using an incorrect macro
> (IFPGA_ARG_NAME instead of IFPGA_BUS_NAME), yet it was working fine
> as this macro is aligned with the ifpga bus name.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
> drivers/bus/auxiliary/auxiliary_common.c | 2 +-
> drivers/bus/auxiliary/bus_auxiliary_driver.h | 2 --
> drivers/bus/cdx/cdx.c | 3 +--
> drivers/bus/dpaa/dpaa_bus.c | 6 ++----
> drivers/bus/fslmc/fslmc_bus.c | 8 +++-----
> drivers/bus/ifpga/bus_ifpga_driver.h | 2 --
> drivers/bus/ifpga/ifpga_bus.c | 4 ++--
> drivers/bus/pci/pci_common.c | 2 +-
> drivers/bus/platform/platform.c | 2 +-
> drivers/bus/uacce/uacce.c | 2 +-
> drivers/bus/vdev/vdev.c | 2 +-
> drivers/bus/vmbus/vmbus_common.c | 2 +-
> drivers/raw/ifpga/ifpga_rawdev.c | 2 +-
> 13 files changed, 15 insertions(+), 24 deletions(-)
>

Acked-by: Kevin Traynor <ktraynor@redhat.com>