[PATCH v4 1/5] bus/platform: match device by devicetree compatible string

[David Marchand]

On Wed, 12 Aug 2026 at 11:30, Gagandeep Singh <g.singh at nxp.com> wrote:
> diff --git a/drivers/bus/platform/bus_platform_driver.h b/drivers/bus/platform/bus_platform_driver.h
> index e4dcbacf5e..607fd0bd1d 100644
> --- a/drivers/bus/platform/bus_platform_driver.h
> +++ b/drivers/bus/platform/bus_platform_driver.h
> @@ -121,12 +121,19 @@ static const char *pdrvinit_ ## nm ## _alias; \
>  RTE_INIT(pdrvinitfn_ ##nm) \
>  { \
>         (platform_drv).driver.name = RTE_STR(nm); \
> -       (platform_drv).driver.alias = pdrvinit_ ## nm ## _alias; \
> +       if (pdrvinit_ ## nm ## _alias != NULL) \
> +               (platform_drv).driver.alias = pdrvinit_ ## nm ## _alias; \
>         rte_platform_register(&(platform_drv)); \

This alias stuff in the platform bus header looks wrong in the first place.
We have a conflict between vdev and platform bus both defining
RTE_PMD_REGISTER_ALIAS.

You could simply declare:

RTE_PMD_REGISTER_PLATFORM(dma_imx_edma5, imx_edma5_pmd_drv);
static const char *pdrvinit_dma_imx_edma5_alias = IMX_EDMA5_COMPAT;

But, since we have no in-tree platform driver using the alias stuff so
far, I am for removing the RTE_PMD_REGISTER_ALIAS macro from
bus/platform (plus the pdrvinit_ ## nm ## _alias variable), and let
the driver value be the reference.

>  } \
>  RTE_PMD_EXPORT_NAME(nm)
>
> -/** Helper for setting platform driver alias. */
> +/**
> + * Helper for setting platform driver alias.
> + *
> + * Note: this macro uses RTE_STR() to stringify the alias argument, so it
> + * cannot express strings that contain a comma. For such aliases,
> + * set .driver.alias directly in the rte_platform_driver initializer instead.
> + */
>  #define RTE_PMD_REGISTER_ALIAS(nm, alias) \
>  static const char *pdrvinit_ ## nm ## _alias = RTE_STR(alias)
>


-- 
David Marchand