[U-Boot] [PATCH] board_f: Show only .text location in display_text_info()
Alexey Brodkin
Alexey.Brodkin at synopsys.com
Thu Feb 1 17:33:07 UTC 2018
For some reason we used to print not only location of .text
section [before relocation] but also .bss start and end.
Moreover how that was printed was obviously misleading:
--------------------------------->8---------------------------
U-Boot code: 20001000 -> 30008C00 BSS: -> 3000C58E
--------------------------------->8---------------------------
I would interpret it like this:
U-Boot's code is situated in memory starting from 0x20001000
till 0x30008000.. and BTW there's .bss section starting from 0x3000C58E.
Which is wrong. But anyways why bother about .bss here so let's
drop .bss part in that pure debug output.
Note even though this change has logically nothing in common with [1]
they both touch the same function display_text_info() which makes [1]
a prerequisite for this one. Otherwise trivial modifications will be
required in both if they happen to be reordered.
[1] http://patchwork.ozlabs.org/patch/868299/
Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
Cc: Simon Glass <sjg at chromium.org>
Cc: Tom Rini <trini at konsulko.com>
---
common/board_f.c | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/common/board_f.c b/common/board_f.c
index de3907cd11fc..cfb4cfc52133 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -124,15 +124,7 @@ static int init_baud_rate(void)
static int display_text_info(void)
{
#if !defined(CONFIG_SANDBOX) && !defined(CONFIG_EFI_APP)
- ulong bss_start, bss_end, text_base;
-
- bss_start = (ulong)&__bss_start;
- bss_end = (ulong)&__bss_end;
-
- text_base = CONFIG_SYS_TEXT_BASE;
-
- debug("U-Boot code: %08lX -> %08lX BSS: -> %08lX\n",
- text_base, bss_start, bss_end);
+ debug("U-Boot code: %08x\n", CONFIG_SYS_TEXT_BASE);
#endif
return 0;
--
2.14.3
More information about the U-Boot
mailing list