Hello Albert,
Albert ARIBAUD wrote:
Le 12/10/2010 13:31, Heiko Schocher a écrit :
Signed-off-by: Wolfgang Denkwd@denx.de Signed-off-by: Heiko Schocherhs@denx.de
I assume this is [ELF_RELOC], not [NEW_RELOC], right? I'm asking because using the same tag for all "elf_reloc" branch related patches makes it easier to find them all back.
Ups, you are right, fix this.
diff --git a/arch/arm/cpu/arm1136/u-boot.lds b/arch/arm/cpu/arm1136/u-boot.lds
- __dynsym_start = .;
- .dynsym : { *(.dynsym) }
- __got_start = .; . = ALIGN(4); .got : { *(.got) }
Do you need to keep .got in the linker file? I think it can be removed.
Hmm.. think you are right, I try this, thanks.
You could also add
/DISCARD/ : { *(.dynstr*) } /DISCARD/ : { *(.dynamic*) } /DISCARD/ : { *(.plt*) } /DISCARD/ : { *(.interp*) } /DISCARD/ : { *(.gnu*) }
At the end of the sections statement.
Ok.
diff --git a/include/configs/qong.h b/include/configs/qong.h
+#define CONFIG_RELOC_FIXUP_WORKS
I believe this is now common to all ARMs, hence unneeded in config files.
Hmm.. instead we should remove in arch/arm/include/asm/config.h
24 #if defined(CONFIG_SYS_ARM_WITHOUT_RELOC) 25 /* Relocation to SDRAM works on all ARM boards */ 26 #define CONFIG_RELOC_FIXUP_WORKS 27 #endif
this #if defined(CONFIG_SYS_ARM_WITHOUT_RELOC) ... #endif
And remove for all arm board, which now define CONFIG_RELOC_FIXUP_WORKS this in the config file. If you are OK with that, I can make such a patch.
bye, Heiko