Wolfgang Denk <w...@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 <w...@denx.de>
> Cc: Murray Jensen <murray.jen...@csiro.au>
> Cc: Joakim Tjernlund <joakim.tjernl...@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
>

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to