diff options
| author | Simon Glass <sjg@chromium.org> | 2015-04-24 22:33:07 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2015-04-28 16:49:03 -0600 |
| commit | f66529f998e59acbd64ccce3adfce8eedfa52da8 (patch) | |
| tree | 417083ee2a5060354f7c7b729c75cc9c655e4ca3 /drivers/core | |
| parent | 4f60166c909f40da6aee14b810806dfa9f553bbc (diff) | |
| download | talos-obmc-uboot-f66529f998e59acbd64ccce3adfce8eedfa52da8.tar.gz talos-obmc-uboot-f66529f998e59acbd64ccce3adfce8eedfa52da8.zip | |
dm: core: Correct bug introduced in uclass_first/next_device()
These functions now rely on uclass_find_first/next_device() and assume that
they will either return failure (-ve error code) or a device. In fact,
coming to the end of a list is not considered failure and they return 0
in that case.
The logic to deal with this was replaced in commit acb9ca2a with just using
uclass_get_device_tail(). Add back the missing logic. This bug was
caught by unit tests but since they were broken for other reasons at the
time, this was not noticed.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/core')
| -rw-r--r-- | drivers/core/uclass.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/core/uclass.c b/drivers/core/uclass.c index 04e939d6c1..7de817324b 100644 --- a/drivers/core/uclass.c +++ b/drivers/core/uclass.c @@ -277,6 +277,7 @@ int uclass_get_device_tail(struct udevice *dev, int ret, if (ret) return ret; + assert(dev); ret = device_probe(dev); if (ret) return ret; @@ -342,6 +343,8 @@ int uclass_first_device(enum uclass_id id, struct udevice **devp) *devp = NULL; ret = uclass_find_first_device(id, &dev); + if (!dev) + return 0; return uclass_get_device_tail(dev, ret, devp); } @@ -352,6 +355,8 @@ int uclass_next_device(struct udevice **devp) *devp = NULL; ret = uclass_find_next_device(&dev); + if (!dev) + return 0; return uclass_get_device_tail(dev, ret, devp); } |
