diff options
author | Greg Kroah-Hartman <gregkh@suse.de> | 2009-01-09 17:44:18 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-01-09 17:44:18 -0500 |
commit | cda5e83fdea476dce9c0a9b1152cd6ca46832cc4 (patch) | |
tree | 38095e8b88c8c163cb8b61e8f071d444752fad6b /drivers/base/driver.c | |
parent | 4db8e282f2d1dfa43d51ce2a4817901312c9134d (diff) |
Revert "driver core: move knode_driver into private structure"
This reverts commit 93e746db183b3bdbbda67900f79b5835f9cb388f.
Turns out that device_initialize shouldn't fail silently.
This series needs to be reworked in order to get into proper
shape.
Reported-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/base/driver.c')
-rw-r--r-- | drivers/base/driver.c | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/drivers/base/driver.c b/drivers/base/driver.c index b76cc69f1106..1e2bda780e48 100644 --- a/drivers/base/driver.c +++ b/drivers/base/driver.c | |||
@@ -19,14 +19,7 @@ | |||
19 | static struct device *next_device(struct klist_iter *i) | 19 | static struct device *next_device(struct klist_iter *i) |
20 | { | 20 | { |
21 | struct klist_node *n = klist_next(i); | 21 | struct klist_node *n = klist_next(i); |
22 | struct device *dev = NULL; | 22 | return n ? container_of(n, struct device, knode_driver) : NULL; |
23 | struct device_private *dev_prv; | ||
24 | |||
25 | if (n) { | ||
26 | dev_prv = to_device_private_driver(n); | ||
27 | dev = dev_prv->device; | ||
28 | } | ||
29 | return dev; | ||
30 | } | 23 | } |
31 | 24 | ||
32 | /** | 25 | /** |
@@ -49,7 +42,7 @@ int driver_for_each_device(struct device_driver *drv, struct device *start, | |||
49 | return -EINVAL; | 42 | return -EINVAL; |
50 | 43 | ||
51 | klist_iter_init_node(&drv->p->klist_devices, &i, | 44 | klist_iter_init_node(&drv->p->klist_devices, &i, |
52 | start ? &start->p->knode_driver : NULL); | 45 | start ? &start->knode_driver : NULL); |
53 | while ((dev = next_device(&i)) && !error) | 46 | while ((dev = next_device(&i)) && !error) |
54 | error = fn(dev, data); | 47 | error = fn(dev, data); |
55 | klist_iter_exit(&i); | 48 | klist_iter_exit(&i); |
@@ -83,7 +76,7 @@ struct device *driver_find_device(struct device_driver *drv, | |||
83 | return NULL; | 76 | return NULL; |
84 | 77 | ||
85 | klist_iter_init_node(&drv->p->klist_devices, &i, | 78 | klist_iter_init_node(&drv->p->klist_devices, &i, |
86 | (start ? &start->p->knode_driver : NULL)); | 79 | (start ? &start->knode_driver : NULL)); |
87 | while ((dev = next_device(&i))) | 80 | while ((dev = next_device(&i))) |
88 | if (match(dev, data) && get_device(dev)) | 81 | if (match(dev, data) && get_device(dev)) |
89 | break; | 82 | break; |