From: john stultz Currently register_clocksource() is a bit unconventional in that it's currently void, which has to be a first for kernel register_xxx() routines. Additionally, register_clocksource() _can_ fail, and essentially every caller of it has an int return type anyways, so it doesn't make much sense to lie about it. The other issue is that the clocksource documentation is wrong, the example won't compile, defines everything as non-static, and the initcall is never flagged as __init. Also fixes a "teh" typo in a comment. Signed-off-by: Paul Mundt Signed-off-by: John Stultz Signed-off-by: Andrew Morton --- arch/i386/kernel/tsc.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff -puN arch/i386/kernel/tsc.c~x86-blacklist-tsc-from-systems-where-it-is-known-to-be-bad-crs arch/i386/kernel/tsc.c --- 25/arch/i386/kernel/tsc.c~x86-blacklist-tsc-from-systems-where-it-is-known-to-be-bad-crs Fri Feb 3 15:35:31 2006 +++ 25-akpm/arch/i386/kernel/tsc.c Fri Feb 3 15:35:31 2006 @@ -419,7 +419,7 @@ static int __init init_tsc_clocksource(v /* lower the rating if we already know its unstable: */ if (check_tsc_unstable()) clocksource_tsc.rating = 50; - register_clocksource(&clocksource_tsc); + return register_clocksource(&clocksource_tsc); } return 0; _