diff options
author | Simon Glass <sjg@chromium.org> | 2023-08-21 21:16:52 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2023-08-31 13:16:54 -0400 |
commit | 13123276806f5a2e209bd1ae16c894e0415e520d (patch) | |
tree | 1520f892b8d2fdcf9d7baba742d478a9c0620632 /lib | |
parent | 468e372e9ad4c551d513b2e73c1f5c1cbb2e4097 (diff) | |
download | u-boot-13123276806f5a2e209bd1ae16c894e0415e520d.zip u-boot-13123276806f5a2e209bd1ae16c894e0415e520d.tar.gz u-boot-13123276806f5a2e209bd1ae16c894e0415e520d.tar.bz2 |
initcall: Adjust the failure message and return value
Move the failure message outside the loop, so it is easier to follow the
code. Avoid swallowing the error code - just pass it along.
Drop the initcall-list address from the output. This is confusing since
we show two addresses. Really it is only the function address which is
useful, since it can be looked up in the map, e.g. with:
grep -A1 -B1 serial_init u-boot.map
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/initcall.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/lib/initcall.c b/lib/initcall.c index 81c5d24..0f74cef 100644 --- a/lib/initcall.c +++ b/lib/initcall.c @@ -46,11 +46,13 @@ int initcall_run_list(const init_fnc_t init_sequence[]) } ret = func(); - if (ret) { - printf("initcall sequence %p failed at call %p (err=%d)\n", - init_sequence, (char *)func - reloc_ofs, ret); - return -1; - } + } + + if (ret) { + printf("initcall failed at call %p (err=%dE)\n", + (char *)func - reloc_ofs, ret); + + return ret; } return 0; |