On Wed, Jan 24, 2007 at 05:14:41PM +0300, Vitaly Bordug wrote:
>
> Keeps sysfs from creating link if device is already bound to driver.
> Similar behavior was before the rework;
Which "rework" caused this?
> now without this patch fixed PHY will complain about not been able to
> create file in sysfs.
Which device is being bound to a driver before it is registered? I know
the i2c layer does it, but I haven't had any people reporting issues
there.
> Signed-off-by: Vitaly Bordug <[email protected]>
> ---
>
> drivers/base/dd.c | 8 +++++---
> 1 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/base/dd.c b/drivers/base/dd.c
> index 510e788..6da3d01 100644
> --- a/drivers/base/dd.c
> +++ b/drivers/base/dd.c
> @@ -26,12 +26,12 @@
> #define to_drv(node) container_of(node, struct device_driver, kobj.entry)
>
>
> -static void driver_bound(struct device *dev)
> +static int driver_bound(struct device *dev)
> {
> if (klist_node_attached(&dev->knode_driver)) {
> printk(KERN_WARNING "%s: device %s already bound\n",
> __FUNCTION__, kobject_name(&dev->kobj));
> - return;
> + return 0;
> }
>
> pr_debug("bound device '%s' to driver '%s'\n",
> @@ -42,6 +42,7 @@ static void driver_bound(struct device *dev)
> BUS_NOTIFY_BOUND_DRIVER, dev);
>
> klist_add_tail(&dev->knode_driver, &dev->driver->klist_devices);
> + return 1;
Please return 0 for "good" and -ERRORSOMETHING for a when a problem
happens, like the rest of the kernel.
thanks,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
