Re: [Patch, microblaze]: Add a small Linux BSP

[Michael Eager <eager at eagerm dot com>]

On 07/25/13 19:13, David Holsgrove wrote:
> > -----Original Message-----
> > From: Michael Eager [mailto:eager@eagerm.com]
> > Sent: Friday, 26 July 2013 11:43 am
> > To: newlib@sourceware.org; vinschen@redhat.com; David Holsgrove
> > Subject: Re: [Patch, microblaze]: Add a small Linux BSP
> >
> > On 07/23/13 02:11, Corinna Vinschen wrote:
> > > On Jul 23 17:16, David Holsgrove wrote:
> > > > On 23 July 2013 17:10, Corinna Vinschen <vinschen@redhat.com> wrote:
> > > > > On Jul 23 16:25, David Holsgrove wrote:
> > > > > > From: "David Holsgrove" <...>
> > > > > >
> > > > > > libgloss/ChangeLog
> > > > > >
> > > > > > 2013-07-23  Edgar E. Iglesias <...>
> > > > > >
> > > > > >     * microblaze/Makefile.in: Add LINUX_BSP.
> > > > > >     * microblaze/elf-gloss-linux.ld: New file.
> > > > > >     * microblaze/elf-gloss-linux.specs: Likewise.
> > > > > >     * microblaze/linux-crt0.S: Likewise.
> > > > > >     * microblaze/linux-inbyte.c: Likewise.
> > > > > >     * microblaze/linux-outbyte.c: Likewise.
> > > > > >     * microblaze/linux-syscalls-wrap.c: Likewise.
> > > > > >     * microblaze/linux-syscalls.S: Likewise.
> > > > > >     * microblaze/linux-syscalls.h: Likewise.
> > > > >
> > > > > Please apply.
> > > >
> > > > Thanks Corinna,
> > > >
> > > > I don't have write perms on newlib - can I request them, or can you
> > > > apply the patch for me?
> > >
> > > Everything applied.
> >
> > This patch breaks breaks existing mb-gcc compilations.
> >
> > Please revert the patch until it can be fixed.
> Hi Michael,
>
> Can you provide details regarding the breakage?
>
> This should be a separate library and not linked in by default - what sort
> of compilation errors are you seeing?

libgloss/sbrk.o is being linked into executables instead of
libgloss/microblaze/sbrk.o, which results in an unresolved reference to _end.


--
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077