On 01/19/2011 05:42 AM, Stephen Rothwell wrote:
Presumably caused by commit b64781e8e2b0f480affb170460cbfb3b11fa9e5d
("param: add null statement to compiled-in module params"). I wonder how
many more build errors this change may provoke. Please try to find them
and fix them before adding this change, thanks.
It can provoke if I'm not mistaken, one more error. So two in total.
grep -r -E '^MODULE_.*\(.*\) *[^;]* *$' .
Yields a lot of false positives (enhancements welcome) that sifts down
to these two:
./net/dsa/dsa.c:MODULE_AUTHOR("Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx>")
./drivers/net/arm/ks8695net.c:MODULE_AUTHOR("Simtec Electronics")
I'll send patches for these two, so Rusty can apply them on
top of his next branch and be done with it.
I used the version of the rr tree from next-20110118 for today.
Couldn't find any more violators in the next tree.
There shouldn't be many: any file that is built as modules with
allmod will result in compile error even without this patch, the two above
must already be broken with allmodconfig, presumably they are
used in archs that do not enable loadable modules, so they don't
see the problem.
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
