From: Masahiro Yamada <yamada.m@jp.panasonic.com>
Date: Mon, 17 Nov 2014 08:19:40 +0000 (+0900)
Subject: dm: core: remove unnecessary return condition in driver lookup
X-Git-Tag: v2025.01-rc5-pxa1908~14236^2~1
X-Git-Url: http://git.dujemihanovic.xyz/img/%22http:/www.sics.se/static/git-logo.png?a=commitdiff_plain;h=f724e0bba2967397545bd48992bba66452d6759d;p=u-boot.git

dm: core: remove unnecessary return condition in driver lookup

The variable "drv" never becomes NULL because ll_entry_start()
always returns a valid pointer even if there are no entries.

The case "n_ents == 0" is covered by the following "for" loop.

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Acked-by: Simon Glass <sjg@chromium.org>
---

diff --git a/drivers/core/lists.c b/drivers/core/lists.c
index 3a1823db02..058f2780a3 100644
--- a/drivers/core/lists.c
+++ b/drivers/core/lists.c
@@ -25,9 +25,6 @@ struct driver *lists_driver_lookup_name(const char *name)
 	const int n_ents = ll_entry_count(struct driver, driver);
 	struct driver *entry;
 
-	if (!drv || !n_ents)
-		return NULL;
-
 	for (entry = drv; entry != drv + n_ents; entry++) {
 		if (!strcmp(name, entry->name))
 			return entry;