Re: [REGRESSION] "bind" a device to a driver doesn't not workanymore

From: Greg KH
Date: Sun Oct 18 2009 - 04:02:55 EST


On Sun, Oct 18, 2009 at 12:51:27AM -0700, Dmitry Torokhov wrote:
> On Mon, Oct 12, 2009 at 08:17:41PM -0700, Dmitry Torokhov wrote:
> > On Mon, Oct 12, 2009 at 11:54:17AM -0700, Greg KH wrote:
> > > On Mon, Oct 12, 2009 at 11:33:13AM -0700, Dmitry Torokhov wrote:
> > > > On Mon, Oct 12, 2009 at 10:35:51AM -0700, Greg KH wrote:
> > > > > On Mon, Oct 12, 2009 at 08:48:46AM -0700, Dmitry Torokhov wrote:
> > > > > > > So at least, unbind should fail as well as bind.
> > > > > > >
> > > > > >
> > > > > > That would be Greg's domain s it is driver core decision whether to
> > > > > > allow unbinding platform devices registered with
> > > > > > platform_driver_probe().
> > > > >
> > > > > No, I do not see why that should not be allowed.
> > > > >
> > > >
> > > > Because once you did unbind the device you are stuck (unless the driver
> > > > is compiled as a module, but then you could just unload the module
> > > > instead of unbinding). Disallowing unbind would allow discarding not
> > > > only __devinit but __devexit sections when driver is built in which
> > > > would make ebedded people happy[^Hier].
> > >
> > > Yeah, good point.
> > >
> > > Does anyone do bind/unbind with platform devices today?
> > >
> > > If not, a patch changing this would be welcome.
> > >
> >
> > How about this one?
> >
>
> *ping* We need to resolve this, otherwise everyone who likes to rebind
> i8042 in couse of STR/STD will be in trouble.

Sorry, am in Tokyo right now.

I like this patch, and will queue it up, but it's too late for .32, it
will have to wait for .33.

I'll hope to get my patch queue flushed out after I return in about 10
days.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/