lkml.org 
[lkml]   [2006]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] define setup_arch() in header file
Ben Dooks <ben-linux@fluff.org> wrote:
>
> When running sparse over an ARM build of 2.6.16-rc5, I came
> across this error, which is due to setup_arch() being used
> be init/main.c, but not being defined in any headers.
>
> This patch adds setup_arch() definition to include/linux/init.h
>
> The warning is:
> arch/arm/kernel/setup.c:730:13: warning: symbol 'setup_arch' was not declared. Should it be static?
>
> ...
>
> --- linux-2.6.16-rc5/include/linux/init.h 2006-02-28 09:05:02.000000000 +0000
> +++ linux-2.6.16-rc5-fixes/include/linux/init.h 2006-03-05 20:39:21.000000000 +0000
> @@ -69,6 +69,10 @@ extern initcall_t __security_initcall_st
>
> /* Defined in init/main.c */
> extern char saved_command_line[];
> +
> +/* used by init/main.c */
> +extern void setup_arch(char **);

There are already declarations of setup_arch in include/asm-ppc and
include/asm-powerpc. Different declarations.

Plus there's an unneeded-with-this-patch declaration in init/main.c.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-03-06 13:12    [W:0.070 / U:0.596 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site