From 01b120b6395d5be6fc0fb408df8bf8849c2c47b5 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Thu, 18 May 2017 20:08:51 -0600 Subject: [PATCH] dm: core: Set return value first in lists_bind_fdt() Adjust the order to make it clear that *devp is set to NULL by default. Signed-off-by: Simon Glass --- 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 23b6ba78d3..72c55e205f 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.39.5