[U-Boot] [PATCH 26/34] binman: Correct symbol calculation with non-zero image base
sjg at google.com
Tue Oct 15 14:07:00 UTC 2019
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.
More information about the U-Boot