diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2022-04-24 21:07:47 +0200 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2022-05-07 20:16:59 +0200 |
| commit | 594ade3eef3f2d458902ced2cb2614dfae8558de (patch) | |
| tree | 139c7e9282167892f57a7e73e6bd0f6a3f298b54 /scripts | |
| parent | modpost: do not write out any file when error occurred (diff) | |
| download | linux-594ade3eef3f2d458902ced2cb2614dfae8558de.tar.xz linux-594ade3eef3f2d458902ced2cb2614dfae8558de.zip | |
modpost: remove stale comment about sym_add_exported()
The description,
it may have already been added without a
CRC, in this case just update the CRC
... is no longer valid.
In the old days, this function was used to update the CRC as well.
Commit 040fcc819a2e ("kbuild: improved modversioning support for
external modules") started to use a separate function (sym_update_crc)
for updating the CRC.
The first part, "Add an exported symbol" is correct, but it is too
obvious from the function name. Drop this comment entirely.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/mod/modpost.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index f0d48f65fb33..c7cfeeb088f7 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -387,10 +387,6 @@ static void sym_update_namespace(const char *symname, const char *namespace) s->namespace = namespace[0] ? NOFAIL(strdup(namespace)) : NULL; } -/** - * Add an exported symbol - it may have already been added without a - * CRC, in this case just update the CRC - **/ static struct symbol *sym_add_exported(const char *name, struct module *mod, enum export export) { |