Hi Alexey, On 1 February 2018 at 10:33, Alexey Brodkin <[email protected]> wrote: > > 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 <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Tom Rini <[email protected]> > --- > common/board_f.c | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-)
I think it is sometimes useful to print BSS. I have had problems with it overlapping (e.g.) the DT. Can you just fix the output? Regards, Simon _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
