From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter Tyser <ptyser@xes-inc.com>
Date: Tue, 15 Jun 2010 15:39:01 -0500
Subject: [U-Boot] [PATCH] examples/standalone: Remove relocation compile
 flags for PowerPC
In-Reply-To: <20100615202813.A1816153659@gemini.denx.de>
References: <1276572325-12642-1-git-send-email-ptyser@xes-inc.com>
	<4C17CE2F.9050208@freescale.com> <4C17CEEC.8000409@freescale.com>
	<1276630601.32134.1501.camel@petert> <4C17D851.4010307@freescale.com>
	<1276632905.32134.1535.camel@petert>
	<20100615202813.A1816153659@gemini.denx.de>
Message-ID: <1276634341.32134.1541.camel@petert>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

<snip>

> >  all:   $(obj).depend $(OBJS) $(LIB) $(SREC) $(BIN) $(ELF)
> > @@ -88,7 +89,7 @@ $(LIB):       $(obj).depend $(LIBOBJS)
> >  
> >  $(ELF):
> >  $(obj)%:       $(obj)%.o $(LIB)
> > -               $(LD) -g -Ttext $(STANDALONE_LOAD_ADDR) \
> > +               $(LD) -g -Ttext $(STANDALONE_LOAD_ADDR) -sort-common \
> >                         -o $@ -e $(SYM_PREFIX)$(notdir $(<:.o=)) $< $(LIB) \
> >                         -L$(gcclibdir) -lgcc
> 
> I'm not sure if this is needed.

Agreed, in fact, I'm not sure why I even put that in there in the first
place after re-reading ld's man page:)

> > Could you try the above change with your flash_wp test case?  Or make
> > the flash_wp app public?  It should put the first function at the base
> > of the image in theory.
> 
> I think the "timer" code is sufficient to show the problem, and that
> your fix helps.  If Timur confirms it's working for his secret code
> too we should apply this.

Do you want this rolled into the first patch, or sent as a follow-up?

Best,
Peter