[U-Boot] [PATCH 01/22] dm: core: Set return value first in lists_bind_fdt()

Simon Glass sjg at chromium.org
Wed Jan 18 06:50:30 CET 2017


Adjust the order to make it clear that *devp is set to NULL by default.

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

 drivers/core/lists.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/core/lists.c b/drivers/core/lists.c
index 23b6ba78d3b..72c55e205f9 100644
--- a/drivers/core/lists.c
+++ b/drivers/core/lists.c
@@ -140,10 +140,10 @@ int lists_bind_fdt(struct udevice *parent, const void *blob, int offset,
 	int result = 0;
 	int ret = 0;
 
-	name = fdt_get_name(blob, offset, NULL);
-	dm_dbg("bind node %s\n", name);
 	if (devp)
 		*devp = NULL;
+	name = fdt_get_name(blob, offset, NULL);
+	dm_dbg("bind node %s\n", name);
 
 	compat_list = fdt_getprop(blob, offset, "compatible", &compat_length);
 	if (!compat_list) {
-- 
2.11.0.483.g087da7b7c-goog



More information about the U-Boot mailing list