[U-Boot] [PATCH 26/34] binman: Correct symbol calculation with non-zero image base

[Stephen Warren]

On 10/15/19 8:07 AM, Simon Glass wrote:
> Hi Stephen,
> 
> On Mon, 14 Oct 2019 at 09:49, Stephen Warren <swarren at wwwdotorg.org> wrote:
>>
>> On 9/26/19 6:38 PM, sjg at google.com wrote:
>>> At present binman adds the image base address to the symbol value before
>>> it writes it to the binary. This is not correct since the symbol value
>>> itself (e.g. image position) has no relationship to the image base.
>>>
>>> Fix this and update the tests to cover this case.
>>>
>>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>> ---
>>>
>>>    tools/binman/elf.py                      | 4 +---
>>>    tools/binman/test/u_boot_binman_syms.lds | 2 +-
>>>    2 files changed, 2 insertions(+), 4 deletions(-)
>>>
>>> Applied to u-boot-dm, thanks!
>>
>> This seems to have only just been pushed. This patch breaks boot on
>> Jetson TK1; u-boot-dm.git master hangs in SPL or before the main U-Boot
>> prints anything, whereas after reverting this patch solves the issue.
>>
>> With this patch applied, all I get is:
>>
>> U-Boot SPL 2019.10-00490-g4f035abcde98 (Oct 14 2019 - 09:48:30 -0600)
>> Trying to boot from RAM
> 
> Yes, just pushed as I had to wait for u-boot-dm/testing to pass, and
> it has some flaky runs and then I went on holiday.
> 
> This is unfortunate. It looks like we were missing test coverage. I'll
> see if I can look at it later in the week, but for now I think I might
> drop this patch.

Thanks. The latest push to u-boot-dm/master solves/removes this issue.