Re: [PATCH v4 05/10] platform: Add test managed platform_device/driver APIs

From: Stephen Boyd
Date: Tue Apr 30 2024 - 17:32:23 EST


Quoting Stephen Boyd (2024-04-24 11:11:21)
> Quoting Stephen Boyd (2024-04-22 16:23:58)
> > diff --git a/drivers/base/test/platform_kunit.c b/drivers/base/test/platform_kunit.c
[...]
> > +
> > + /*
> > + * Wait for the driver to probe (or at least flush out of the deferred
> > + * workqueue)
> > + */
> > + wait_for_device_probe();
>
> Should this be removed? I was thinking that this isn't a pure wrapper
> around platform_driver_register() because it has this wait call. Maybe
> it's better to have some other kunit API that can wait for a specific
> device to probe and timeout if it doesn't happen in that amount of time.
> That API would use the bus notifiers and look for
> BUS_NOTIFY_BOUND_DRIVER. Or maybe that function could setup a completion
> that the test can wait on.

I have an implementation that does this that I'll send.