A few places pre-declare "int module_init(void);" and "void
module_cleanup(void);".  Other than being obsolete, this is
unneccessary (it's in init.h anyway).

There are still about 100 places which still use the
obsolete-since-2.2 "a function named module_init() magically gets
called": this change frees us up implement that via a macro.