Messages in this thread | | | From | Dmitry Torokhov <> | Subject | Re: [PATCH 2.6.10-rc1 0/4] driver-model: manual device attach | Date | Mon, 8 Nov 2004 02:23:07 -0500 |
| |
On Friday 05 November 2004 01:32 am, Tejun Heo wrote: > > Do we really need 2 or even 3 files ("attach", "detach" and "rescan")? > > Given that you really can't (at least not yet) do all there operations > > for all buses from the core that woudl require 3 per-bus callbacks. > > I think reserving special values such as "none" or "detach" and "rescan" > > shoudl work just fine and also willallow extending supported operations > > on per-bus basis. For example serio bus supports "reconnect" option which > > tries to re-initialize device if something happened to it. It does not > > want to do rescan as that would generate new input devices while it is > > much more convenient to re-use old ones. > > How about making the command format "CMD ARGS" rather than > "{CMD|DRIVERNAME}" i.e. >
Hi guys,
What do you think about following set of patches. It adds drvctl default device attribute (write only) that controls device/driver binding. It expects commands in form of "CMD [DRIVER_NAME] [ARG ARG...]" so I think it will be very easy to adapt it to Tejun's per-device parameters.
I adjusted serio bus to the new form of commands so now it accepts: "detach", "attach <driver>", "rescan", "reconnect"
And I added PCI bus drvctl that understands "detach", "attach <driver>", and "rescan".
As we add drvctl methods to the rest of the buses we can review what can be pulled into the core and what has to stay bus-specific.
Plus there is the bind_mode patch that allows switching between automatic and manual binding on per-device/per-driver base.
-- Dmitry - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |