diff options
author | Sam Ravnborg <sam@uranus.ravnborg.org> | 2008-04-25 14:13:30 -0400 |
---|---|---|
committer | Sam Ravnborg <sam@uranus.ravnborg.org> | 2008-04-25 14:13:30 -0400 |
commit | eed7d2798ce7b78547106d0c3557a88251b413d0 (patch) | |
tree | 2643c5c45f65c2a2743fc4b3f061ad7b2e37467b | |
parent | b69d3987f4360a5e7e9e55465b3cdd0cc204b79e (diff) |
kbuild: error out on missing MODULE_LICENSE
Adrian Bunk suggested a build time check for
missing MODULE_LICENSE annotation in modules.
The build time check is fatal as we really
want this fixed for all modules.
In-tree modules should all have been fixed up by now.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Adrian Bunk <bunk@kernel.org>
-rw-r--r-- | scripts/mod/modpost.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 110cf243fa4e..843f6fa517cc 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c | |||
@@ -1552,6 +1552,10 @@ static void read_symbols(char *modname) | |||
1552 | } | 1552 | } |
1553 | 1553 | ||
1554 | license = get_modinfo(info.modinfo, info.modinfo_len, "license"); | 1554 | license = get_modinfo(info.modinfo, info.modinfo_len, "license"); |
1555 | if (!license && !is_vmlinux(modname)) | ||
1556 | fatal("modpost: missing MODULE_LICENSE() in %s\n" | ||
1557 | "see include/linux/module.h for " | ||
1558 | "more information\n", modname); | ||
1555 | while (license) { | 1559 | while (license) { |
1556 | if (license_is_gpl_compatible(license)) | 1560 | if (license_is_gpl_compatible(license)) |
1557 | mod->gpl_compatible = 1; | 1561 | mod->gpl_compatible = 1; |