diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-11-23 02:57:22 -0500 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-12-01 08:21:58 -0500 |
commit | c6826ad8a49f1230dcbc45dac26b54d492b13280 (patch) | |
tree | 40508a256ffb34242576c7c803b8e248715fa6ca /scripts/mod/modpost.c | |
parent | d2665ca8e3310d045b62397590ab54dc485e035e (diff) |
modpost: merge module iterations
Probably, this is just a matter of the order of error/warning
messages. Merge the two for-loops.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'scripts/mod/modpost.c')
-rw-r--r-- | scripts/mod/modpost.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index eedc2742dc8e..963a0b69b9a3 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c | |||
@@ -2496,12 +2496,6 @@ int main(int argc, char **argv) | |||
2496 | if (files_source) | 2496 | if (files_source) |
2497 | read_symbols_from_files(files_source); | 2497 | read_symbols_from_files(files_source); |
2498 | 2498 | ||
2499 | for (mod = modules; mod; mod = mod->next) { | ||
2500 | if (mod->skip) | ||
2501 | continue; | ||
2502 | check_exports(mod); | ||
2503 | } | ||
2504 | |||
2505 | err = 0; | 2499 | err = 0; |
2506 | 2500 | ||
2507 | for (mod = modules; mod; mod = mod->next) { | 2501 | for (mod = modules; mod; mod = mod->next) { |
@@ -2513,6 +2507,7 @@ int main(int argc, char **argv) | |||
2513 | buf.pos = 0; | 2507 | buf.pos = 0; |
2514 | 2508 | ||
2515 | err |= check_modname_len(mod); | 2509 | err |= check_modname_len(mod); |
2510 | check_exports(mod); | ||
2516 | add_header(&buf, mod); | 2511 | add_header(&buf, mod); |
2517 | add_intree_flag(&buf, !external_module); | 2512 | add_intree_flag(&buf, !external_module); |
2518 | add_retpoline(&buf); | 2513 | add_retpoline(&buf); |