[PATCH 07/11] dm: core: Fix a few incorrect comments on first/next functions

Simon Glass sjg at chromium.org
Thu Aug 19 05:40:29 CEST 2021


Tighten up these comments to make the behaviour clearer.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

 include/dm/device.h | 4 ++--
 include/dm/uclass.h | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/dm/device.h b/include/dm/device.h
index 0a9718a5b81..ef6241bca8b 100644
--- a/include/dm/device.h
+++ b/include/dm/device.h
@@ -738,7 +738,7 @@ int device_find_next_child(struct udevice **devp);
  *
  * @parent:	Parent device to search
  * @uclass_id:	Uclass to look for
- * @devp:	Returns device found, if any
+ * @devp:	Returns device found, if any, else NULL
  * @return 0 if found, else -ENODEV
  */
 int device_find_first_inactive_child(const struct udevice *parent,
@@ -750,7 +750,7 @@ int device_find_first_inactive_child(const struct udevice *parent,
  *
  * @parent: Parent device to search
  * @uclass_id:	Uclass to look for
- * @devp: Returns first child device in that uclass, if any
+ * @devp: Returns first child device in that uclass, if any, else NULL
  * @return 0 if found, else -ENODEV
  */
 int device_find_first_child_by_uclass(const struct udevice *parent,
diff --git a/include/dm/uclass.h b/include/dm/uclass.h
index da0c1bfadb1..15e5f9ef5bc 100644
--- a/include/dm/uclass.h
+++ b/include/dm/uclass.h
@@ -354,7 +354,7 @@ int uclass_next_device(struct udevice **devp);
  * The device returned is probed if necessary, and ready for use
  *
  * @devp: On entry, pointer to device to lookup. On exit, returns pointer
- * to the next device in the uclass if no error occurred, or -ENODEV if
+ * to the next device in the uclass if no error occurred, or NULL if
  * there is no next device.
  * @return 0 if found, -ENODEV if not found, other -ve on error
  */
-- 
2.33.0.rc1.237.g0d66db33f3-goog



More information about the U-Boot mailing list