[U-Boot] [PATCH] MPC8260: Fix compile problems with "hymod" board

Commit 9d8fbd1 "powerpc, 8xx: Fixup all 8xx u-boot.lds scripts" broke building of the MPC8260 based "hymod" board. Fix this.
Signed-off-by: Wolfgang Denk wd@denx.de Cc: Murray Jensen Murray.Jensen@csiro.au Cc: Joakim Tjernlund Joakim.Tjernlund@transmode.se --- board/hymod/u-boot.lds | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-)
Joakim - how comes that a patch that is declared to affect 8xx boards only also modifies (and breaks!) a 82xx board??
Murray - can you please verify that the fix actually works on your hardware?
diff --git a/board/hymod/u-boot.lds b/board/hymod/u-boot.lds index 1efa8b3..1309f20 100644 --- a/board/hymod/u-boot.lds +++ b/board/hymod/u-boot.lds @@ -87,9 +87,10 @@ SECTIONS PROVIDE (erotext = .); .reloc : { - *(.got) _GOT2_TABLE_ = .; - *(.got2) + KEEP(*(.got2)) + KEEP(*(.got)) + PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4); _FIXUP_TABLE_ = .; *(.fixup) }

Wolfgang Denk wd@denx.de wrote on 2011/05/01 16:35:06:
Commit 9d8fbd1 "powerpc, 8xx: Fixup all 8xx u-boot.lds scripts" broke building of the MPC8260 based "hymod" board. Fix this.
Signed-off-by: Wolfgang Denk wd@denx.de Cc: Murray Jensen Murray.Jensen@csiro.au Cc: Joakim Tjernlund Joakim.Tjernlund@transmode.se
board/hymod/u-boot.lds | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-)
Joakim - how comes that a patch that is declared to affect 8xx boards only also modifies (and breaks!) a 82xx board??
The change was done with grep/simple script so it must have "missed" this. Anyway, your patch looks good.
Jocke
Murray - can you please verify that the fix actually works on your hardware?
diff --git a/board/hymod/u-boot.lds b/board/hymod/u-boot.lds index 1efa8b3..1309f20 100644 --- a/board/hymod/u-boot.lds +++ b/board/hymod/u-boot.lds @@ -87,9 +87,10 @@ SECTIONS PROVIDE (erotext = .); .reloc : {
- *(.got) _GOT2_TABLE_ = .;
- *(.got2)
- KEEP(*(.got2))
- KEEP(*(.got))
- PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4); _FIXUP_TABLE_ = .; *(.fixup) }
-- 1.7.4.4

Dear Wolfgang Denk,
In message 1304260506-1456-1-git-send-email-wd@denx.de you wrote:
Commit 9d8fbd1 "powerpc, 8xx: Fixup all 8xx u-boot.lds scripts" broke building of the MPC8260 based "hymod" board. Fix this.
Signed-off-by: Wolfgang Denk wd@denx.de Cc: Murray Jensen Murray.Jensen@csiro.au Cc: Joakim Tjernlund Joakim.Tjernlund@transmode.se
board/hymod/u-boot.lds | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-)
applied, thanks.
Best regards,
Wolfgang Denk
participants (2)
-
Joakim Tjernlund
-
Wolfgang Denk