From: Peter Zijlstra <peterz@infradead.org>
Date: Fri, 2 May 2025 14:12:05 +0000 (+0200)
Subject: modpost: Use for() loop
X-Git-Tag: v6.16-rc1~10^2~27
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=6b91ff002c67b2502009f99115c4b7fe5b7b8248;p=thirdparty%2Flinux.git

modpost: Use for() loop

Slight cleanup by using a for() loop instead of while(). This makes it
clearer what is the iteration and what is the actual work done.

Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---

diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index be89921d60b61..2d1c059bf6cf9 100644
--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -1595,12 +1595,10 @@ static void read_symbols(const char *modname)
 			license = get_next_modinfo(&info, "license", license);
 		}
 
-		namespace = get_modinfo(&info, "import_ns");
-		while (namespace) {
+		for (namespace = get_modinfo(&info, "import_ns");
+		     namespace;
+		     namespace = get_next_modinfo(&info, "import_ns", namespace))
 			add_namespace(&mod->imported_namespaces, namespace);
-			namespace = get_next_modinfo(&info, "import_ns",
-						     namespace);
-		}
 
 		if (!get_modinfo(&info, "description"))
 			warn("missing MODULE_DESCRIPTION() in %s\n", modname);