If the kernel build process is creating files automatically, the least it can do is create them in a properly formatted manner. Sure, it's a minor issue, but being consistent is nice.
Unified · Split
| @@ -1853,14 +1853,14 @@ static void add_header(struct buffer *b, struct module *mod) · 1852 unmodified lines | |||
| 1853 | buf_printf(b, "\n"); | 1853 | buf_printf(b, "\n"); |
| 1854 | buf_printf(b, "struct module __this_module\n"); | 1854 | buf_printf(b, "struct module __this_module\n"); |
| 1855 | buf_printf(b, "__attribute__((section(\".gnu.linkonce.this_module\"))) = {\n"); | 1855 | buf_printf(b, "__attribute__((section(\".gnu.linkonce.this_module\"))) = {\n"); |
| 1856 | buf_printf(b, " .name = KBUILD_MODNAME,\n"); | 1856 | buf_printf(b, "\t.name = KBUILD_MODNAME,\n"); |
| 1857 | if (mod->has_init) | 1857 | if (mod->has_init) |
| 1858 | buf_printf(b, " .init = init_module,\n"); | 1858 | buf_printf(b, "\t.init = init_module,\n"); |
| 1859 | if (mod->has_cleanup) | 1859 | if (mod->has_cleanup) |
| 1860 | buf_printf(b, "#ifdef CONFIG_MODULE_UNLOAD\n" | 1860 | buf_printf(b, "#ifdef CONFIG_MODULE_UNLOAD\n" |
| 1861 | " .exit = cleanup_module,\n" | 1861 | "\t.exit = cleanup_module,\n" |
| 1862 | "#endif\n"); | 1862 | "#endif\n"); |
| 1863 | buf_printf(b, " .arch = MODULE_ARCH_INIT,\n"); | 1863 | buf_printf(b, "\t.arch = MODULE_ARCH_INIT,\n"); |
| 1864 | buf_printf(b, "};\n"); | 1864 | buf_printf(b, "};\n"); |
| 1865 | } | 1865 | } |
| 1866 | 1866 | ||