[U-Boot] [PATCH] [NEXT] da830: fixup ARM relocation support

Ben Gardiner bengardiner at nanometrics.ca
Wed Sep 22 17:07:56 CEST 2010


On Wed, Sep 22, 2010 at 10:15 AM, Nick Thompson <nick.thompson at ge.com> wrote:
> On 22/09/10 14:43, Ben Gardiner wrote:
>> What about removing "#define CONFIG_SKIP_RELOCATE_UBOOT" as in commit
>> ab86f72c354f9b2572340f72b74ca0a258c451bd ?
>
> Hmmm. It wouldn't hurt I guess. The UBL copies the code to the correct
> address though doesn't it? The copy is not executed and so the code is
> redundant - or did I miss something?

Yeah, good point. UBL does copy the code to the correct address -- but
I also remember that I needed to remove that define in my testing of
Heiko's patches on the da850.

I'll get around to testing -next again soon and I'll try with
CONFIG_SKIP_RELOCATE_UBOOT defined.

Best Regards,
Ben Gardiner

---
Nanometrics Inc.
http://www.nanometrics.ca


More information about the U-Boot mailing list