public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.
Date: Tue, 8 Apr 2008 15:25:36 +0200	[thread overview]
Message-ID: <200804081525.36771.sr@denx.de> (raw)
In-Reply-To: <1207660051.5826.47.camel@gentoo-jocke.transmode.se>

On Tuesday 08 April 2008, Joakim Tjernlund wrote:
> > > I first did a new symbol for this but changed it to use _start as  I
> > > didn't want to introduce yet another symbol. I would hope it is
> > > possible to rework 4xx to move the _start symbol to were it belongs?
> > > Not having the _start symbol where it should be could bite you some
> > > other day too.
> >
> > I think you got this wrong. _start *is* where it should be, where the
> > execution of the code begins. Where you set the breakpoint in gdb.
>
> Ah, right. I *assumed* that _start was always the first symbol in the
> text segment too. On 4xx it isn't for some reason, but it should be
> possible to move it first.
>
> Sidnote: Does execution really begin at _start for 4xx? I still looks
> like it begin at _start_440 and eventually end up at _start

On 4xx execution always starts at 0xfffffffc (last lword in 32bit address 
space). This location holds a jump to _start for 405 PPC's and to _start_440 
for 440 PPC's. 440 PPC's need some extended initialization (TLB setup etc) 
and later jump to the 4xx common _start.

> > The "new symbol" you mention corresponds to CFG_MONITOR_BASE, but note
> > that it has no fixed address as it depends where you place your image
> > in flash.
>
> Yes, but the address is not important here, it is the difference
> _end - _start. I guess we could define a linker symbol too
> that calculates _end - _start for us and then just do
>   len = _uboot_reloc_size + EXC_OFF_SYS_RESET;
>
> or define a new symbol that is initialised to CFG_MONITOR_BASE or let
> the linker place it at the beginning of .text, hopefully there is
> already a name reserved for the symbol although I don't know of such a
> name. I either case I think one needs to add that symbol to the GOT list
> in start.S

I'm an linker script dyslexic. So no idea if we can handle this solely in the 
linker script or if we need a new common symbol in the PPC start.S's.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  reply	other threads:[~2008-04-08 13:25 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-28 14:41 [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation Joakim Tjernlund
2008-03-28 23:33 ` Kim Phillips
2008-03-28 23:47   ` Joakim Tjernlund
2008-03-29  0:01     ` Kim Phillips
2008-03-29  0:04       ` Joakim Tjernlund
2008-04-08  8:58     ` Stefan Roese
2008-04-08  9:31       ` Joakim Tjernlund
2008-04-08 10:06         ` Stefan Roese
2008-04-08 10:50           ` Joakim Tjernlund
2008-04-08 11:58             ` Wolfgang Denk
2008-04-08 13:07               ` Joakim Tjernlund
2008-04-08 13:25                 ` Stefan Roese [this message]
2008-04-08 14:04                   ` Joakim Tjernlund
2008-04-08 19:52                     ` Stefan Roese
2008-04-08 20:30                       ` Joakim Tjernlund
2008-04-08 20:41                     ` Wolfgang Denk
2008-04-08 20:52                       ` Joakim Tjernlund
2008-04-08 21:09                         ` Wolfgang Denk
2008-04-08 21:51                           ` Joakim Tjernlund
2008-04-08 22:17                             ` David Hawkins
2008-04-09  9:40                             ` Stefan Roese
2008-04-09 10:44                               ` Wolfgang Denk
2008-04-09 11:14                                 ` Joakim Tjernlund
2008-04-09 11:45                                   ` Stefan Roese
2008-04-09 12:36                                   ` Wolfgang Denk
2008-04-08 11:52         ` Wolfgang Denk
  -- strict thread matches above, loose matches on Subject: below --
2008-02-02 19:44 Joakim Tjernlund

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200804081525.36771.sr@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox