Re: [PATCH v2 5/7] bus: factorize devargs lookup

Kevin Traynor
2026-03-26T11:51:13Z

On 3/26/26 10:24 AM, David Marchand wrote:
> Each bus reimplements some similar devargs lookup code.
>
> The differences are in how some bus (PCI, VMBUS etc...) normalizes the
> device names. We can't use the .parse existing handler from outside the
> bus code itself, as the size of the bus specific device location address
> is unknown.
> Introduce a bus specific helper to compare two device names and
> hide this ugly detail.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> Tested-by: Maxime Leroy <maxime@leroys.fr>
> ---
> Changes since v1:
> - fixed copy/paste descriptions in doxygen,
>
> Changes since RFC v3:
> - fixed doxygen,
>
> ---
> drivers/bus/auxiliary/auxiliary_common.c | 16 ++-------
> drivers/bus/cdx/cdx.c | 14 +-------
> drivers/bus/dpaa/dpaa_bus.c | 41 +++++++++++++-----------
> drivers/bus/fslmc/fslmc_bus.c | 34 ++++++++++----------
> drivers/bus/pci/pci_common.c | 38 +++++++++++-----------
> drivers/bus/platform/platform.c | 17 ++--------
> drivers/bus/uacce/uacce.c | 19 ++---------
> drivers/bus/vmbus/linux/vmbus_bus.c | 2 +-
> drivers/bus/vmbus/private.h | 3 --
> drivers/bus/vmbus/vmbus_common.c | 30 ++++++++---------
> drivers/dma/idxd/idxd_bus.c | 14 ++------
> lib/eal/common/eal_common_bus.c | 20 ++++++++++++
> lib/eal/include/bus_driver.h | 32 ++++++++++++++++++
> 13 files changed, 133 insertions(+), 147 deletions(-)

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