[PATCH 2/2] drivers: remove bogus DM_FLAG_PROBE_AFTER_BIND flags

Simon Glass sjg at chromium.org
Sat Jan 18 05:32:52 CET 2025


On Fri, 17 Jan 2025 at 00:29, Caleb Connolly <caleb.connolly at linaro.org>
wrote:
>
> Some drivers set DM_FLAG_PROBE_AFTER_BIND, this does nothing since it's
> only every applied on a per-device basis.
>
> Remove the flags.
>
> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
> ---
>  drivers/mailbox/zynqmp-ipi.c  | 1 -
>  drivers/watchdog/da9063-wdt.c | 1 -
>  2 files changed, 2 deletions(-)
>
> diff --git a/drivers/mailbox/zynqmp-ipi.c b/drivers/mailbox/zynqmp-ipi.c
> index 713d93a200c4..851aa737c03e 100644
> --- a/drivers/mailbox/zynqmp-ipi.c
> +++ b/drivers/mailbox/zynqmp-ipi.c
> @@ -269,6 +269,5 @@ U_BOOT_DRIVER(zynqmp_ipi) = {
>         .name = "zynqmp_ipi",
>         .id = UCLASS_NOP,
>         .of_match = zynqmp_ipi_ids,
>         .probe = zynqmp_ipi_probe,
> -       .flags = DM_FLAG_PROBE_AFTER_BIND,
>  };
> diff --git a/drivers/watchdog/da9063-wdt.c b/drivers/watchdog/da9063-wdt.c
> index b7216b578630..ec9bc0330114 100644
> --- a/drivers/watchdog/da9063-wdt.c
> +++ b/drivers/watchdog/da9063-wdt.c
> @@ -144,6 +144,5 @@ U_BOOT_DRIVER(da9063_wdt) = {
>         .name = "da9063-wdt",
>         .id = UCLASS_WDT,
>         .of_match = da9063_wdt_ids,
>         .ops = &da9063_wdt_ops,
> -       .flags = DM_FLAG_PROBE_AFTER_BIND,
>  };
>
> --
> 2.48.0
>

Reviewed-by: Simon Glass <sjg at chromium.org>

It is actually a bit unclear, since the comments for struct driver mention
DM_FLAG_... for the flags.


More information about the U-Boot mailing list