On 18/01/2025 05:32, Simon Glass wrote:
> On Fri, 17 Jan 2025 at 00:29, Caleb Connolly <[email protected]
> <mailto:[email protected]>> 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 <[email protected]
> <mailto:[email protected]>>
>> ---
>> 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 <[email protected] <mailto:[email protected]>>
>
> It is actually a bit unclear, since the comments for struct driver
> mention DM_FLAG_... for the flags.
Yep, some of the flags (most even?) can be applied to drivers and there
is no proper distinction between which apply to drivers and which to
devices.
DRIVER_FLAG_... and DEV_FLAG_... or something maybe better, something to
revisit in the future for sure.
Thx for the review
--
// Caleb (they/them)
