Hello Sven, On Wed, May 05, 2021 at 05:04:55PM -0400, Sven Van Asbroeck wrote: > On Wed, May 5, 2021 at 4:29 PM Uwe Kleine-König > <u.kleine-koenig@xxxxxxxxxxxxxx> wrote: > > > > A driver without .probe() callback could never bind to a device because > > anybus_bus_probe() returned an error for such a driver. So refuse to > > register such a useless driver. > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > > Looks good to me. For the series: > Reviewed-by: Sven Van Asbroeck <TheSven73@xxxxxxxxx> > > Uwe, I'm glad you're submitting patches for fieldbus. Hope you find it > useful. This code is in staging mainly because only a single > company/developer is using it. So if you are actively using this, or > planning to use it, we can work towards taking this out of staging, if > you like. It's not useful for me. My motivation is to make struct bus_type::remove return void and this driver was just one of the offenders that theoretically could return a non-zero value. So sorry, I have no motivation to support getting this out of staging. BTW: I just noticed there are occurences of anybuss and anybus in the driver. Is only one of them right? Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
Attachment:
signature.asc
Description: PGP signature
