Re: [PATCH 2/4] Driver core: add driver_probe_device

From: Greg KH (greg_at_kroah.com)
Date: 10/29/04

  • Next message: Tom Rini: "[patch 1/8] A different KGDB stub for -mm" 
    Date:	Fri, 29 Oct 2004 13:32:15 -0500
To: Dmitry Torokhov <dtor_core@ameritech.net>

    On Fri, Oct 29, 2004 at 01:24:21PM -0500, Dmitry Torokhov wrote:
    > On Friday 29 October 2004 11:37 am, Greg KH wrote:
    > > On Tue, Oct 12, 2004 at 01:31:36AM -0500, Dmitry Torokhov wrote:
    > > > #### AUTHOR dtor_core@ameritech.net
    > > > #### COMMENT START
    > > > ### Comments for ChangeSet
    > > > Driver core: rename bus_match into driver_probe_device and export
    > > > it so subsystems can bind an individual device to a
    > > > specific driver without getting involved with driver
    > > > core internals.
    > >
    > > Applied, thanks.
    > >
    >
    > Greg,
    >
    > What about "bind_mode" device and driver attributes? If you are not going
    > to apply them then I need to rework driver_probe_device to not call
    > bus->match() function.

    Hm, I'm not going to apply them, but haven't written that email yet,
    sorry.

    Is things now broken with only these 2 patches applied?

    > The reason is that if bind_mode is not in the core
    > then I need to check these attributes in serio's bus match function, but
    > then I will not be able to use driver_probe_device to force binding when
    > user requests it. And if I don't check bind_mode in serio_bus_match then
    > I will have to do all driver/device mathing by hand which I wanted to
    > avoid in the first place.

    Heh, I understand. I like the ideas of your next patches, but just not
    the implementation.

    I really like the "driver" part in the device. But not as a file, let's
    make it a symlink back to the driver that is bound to the device at that
    point in time. This makes it just like the other symlinks in the sysfs
    tree.

    But if we do that, we still don't have a way to implement what you are
    really trying to do (and it breaks your code as you already have a
    driver file.) I'll work on what I propose instead in my next message
    (will be a few hours, have real work to do for a bit, sorry...)

    thanks,

    greg k-h
    -
    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/

  • Next message: Tom Rini: "[patch 1/8] A different KGDB stub for -mm"

    Relevant Pages

    • Re: SysFS, module names and .name
      ... Greg KH wrote: ... I would know which driver name corresponds to ... > changing every single pci driver to enable it. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: small patch: enable pwc usb camera driver
      ... > Greg KH wrote: ... Ick ick ick! ... time to mark the driver broken again... ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [linux-usb-devel] Re: Finding user/kernel pointer bugs [no html]
      ... Greg KH wrote: ... the driver you just refered to does not use the video_usercopy ... However, apart from the "__user" context, there are several drivers ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH 2/4] Driver core: add driver_probe_device
      ... On Friday 29 October 2004 11:37 am, Greg KH wrote: ... What about "bind_mode" device and driver attributes? ... then I need to check these attributes in serio's bus match function, ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: avoiding pci_disable_device()...
      ... > Greg KH wrote: ... > Well, for example, add a new pointer to indicate the driver accessing ... This doesn't work with a driver that is properly using ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)