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

Ben Gardiner
2010-09-22T15:07:56Z

On Wed, Sep 22, 2010 at 10:15 AM, Nick Thompson <nick.thompson@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.