Re: [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.
In message <044901c899ba$83532450$89f96cf0$@Tjernlund@transmode.se> you wrote:
.text : {
- _monitor_base = . ;
There is no guarantee that this is the same as the start address of .text
I am no link script expert, but I cannot see why that would not be the start of the .text segment?
There might be other segments at the start of the image...
See for example cpu/mpc8xx/start.S - while at the moment we really start with .text, it would technically be more approrpriate to change the first 256 bytes (from monitor base to EXC_OFF_SYS_RESET) into .rodata or something like this.
Anyhow, we can go with the other solution instead, define the symbol in start.S instead, if one is needed.
If it cannot be avoided ;-)
Best regards,
Wolfgang Denk
-----Original Message----- From: wd@denx.de [mailto:wd@denx.de] Sent: den 8 april 2008 23:09 To: Joakim Tjernlund Cc: 'Stefan Roese'; u-boot-users@lists.sourceforge.net; 'Kim Phillips' Subject: Re: [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.
In message <044901c899ba$83532450$89f96cf0$@Tjernlund@transmode.se> you wrote:
.text : {
- _monitor_base = . ;
There is no guarantee that this is the same as the start address of .text
I am no link script expert, but I cannot see why that would not be the start of the .text segment?
There might be other segments at the start of the image...
See for example cpu/mpc8xx/start.S - while at the moment we really start with .text, it would technically be more approrpriate to change the first 256 bytes (from monitor base to EXC_OFF_SYS_RESET) into .rodata or something like this.
Ahh, now I see what you mean and it gave me an idea ...
Anyhow, we can go with the other solution instead, define the symbol in start.S instead, if one is needed.
If it cannot be avoided ;-)
... Stefan, maybe you can #define EXC_OFF_SYS_RESET for 405 to something that fits with my relocation patch?
Jocke
participants (2)
-
Joakim Tjernlund -
Wolfgang Denk