On Thu, 2009-09-24 at 00:31 +0200, Wolfgang Denk wrote: > Dear Peter Tyser, > > In message <1253550038-16734-1-git-send-email-pty...@xes-inc.com> you wrote: > > This series attempts to fix relocation to RAM for ppc boards. > > > > I split the patches up pretty liberally, let me know if you'd like > > them organized differently. > > > > I tried to be thorough during the changes (especially #1), let me > > know if I missed anything, there's lots of linker scripts for ppc > > boards:) > > > > Peter Tyser (13): > > ppc: Enable full relocation to RAM > > ppc: Check for compilers that don't support relocation > > ppc: Remove board.c relocation fixups > > ppc: Remove pci config table pointer relocation fixups > > ppc: Remove extable relocation fixups > > ppc: Remove board-specific command table relocation fixups > > tsec: Remove PHY command relocation fixups > > fpga: Remove relocation fixups > > mpl: Remove memory test relocation fixups > > lwmon, lwmon5: Remove sysmon POST relocation fixups > > p3mx: Remove serial relocation fixups > > Conditionally perform common relocation fixups > > ppc: Remove reloc_off field from global_data structure > > All patches applied to "reloc" branch. Thanks for all this work. > > So far the only issue I've seen (I didn't do any actual testing yet, > though) is the missing increment to the XF_VERSION value to indicate > incompatibility with older standalone applications. If you agree, I > woul like to squash this into the topmost commit on the "reloc" > branch?
That sounds good to me. 2 additional comments: 1. Might be nice to move the jumptable to the beginning of the global_data struct when we increment XF_VERSION. 2. Depending on when you merge the reloc branch, it would be nice for people to give the patch in "[PATCH/RFC] arm/microblaze/nios/nios2/sh: Remove relocation fixups" a try. It should reduce the bloat on some other arches and will put pressure on other, non-relocatable arches to fix relocation:) Best, Peter _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot