Re: [PATCH] blackfin / h8300 build error fix

[Andrew Morton <akpm@linux-foundation.org>]

On Wed, 30 Jul 2008 15:22:38 -0400
Yoshinori Sato <ysato@users.sourceforge.jp> wrote:

> At Wed, 30 Jul 2008 12:11:37 -0400,
> Mike Frysinger wrote:
> > 
> > On Wed, Jul 30, 2008 at 11:58 AM, Yoshinori Sato wrote:
> > > init/main.c:781: undefined reference to `___early_initcall_end'
> > >
> > > Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
> > >
> > > ---
> > >  include/asm-generic/vmlinux.lds.h |    2 +-
> > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> > > index 6d88a92..07b2784 100644
> > > --- a/include/asm-generic/vmlinux.lds.h
> > > +++ b/include/asm-generic/vmlinux.lds.h
> > > @@ -362,7 +362,7 @@
> > >
> > >  #define INITCALLS                                                      \
> > >        *(.initcallearly.init)                                          \
> > > -       __early_initcall_end = .;                                       \
> > > +       VMLINUX_SYMBOL(__early_initcall_end) = .;                       \
> > >        *(.initcall0.init)                                              \
> > >        *(.initcall0s.init)                                             \
> > >        *(.initcall1.init)                                              \
> > > --
> > > 1.5.6.3
> > 
> > nice catch, thanks ... looks like BUG_TABLE, TRACEDATA, and PERCPU are
> > also broken, but blackfin/etc... arent using those which is why we
> > havent noticed yet ...
> > -mike
> 
> Thanks.
> I fixed same.
> 
> Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
> 
>  include/asm-generic/vmlinux.lds.h |   14 +++++++-------
>  1 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> index 6d88a92..04cae96 100644
> --- a/include/asm-generic/vmlinux.lds.h
> +++ b/include/asm-generic/vmlinux.lds.h
> @@ -333,9 +333,9 @@
>  #define BUG_TABLE							\
>  	. = ALIGN(8);							\
>  	__bug_table : AT(ADDR(__bug_table) - LOAD_OFFSET) {		\
> -		__start___bug_table = .;				\
> +		VMLINUX_SYMBOL(__start___bug_table) = .;		\
>  		*(__bug_table)						\
> -		__stop___bug_table = .;					\
> +		VMLINUX_SYMBOL(__stop___bug_table) = .;			\
>  	}

Sorry, but I haven't been following this topic very well and I rather
dislike plucking unchangelogged patches out of the middle of email
discussions.

If there's some patch which I should be applying then please resend it,
with a meaningful title, a description of what it does, etc.

Thanks.