In the message string " %s[%d]\t[0x%llx-0x%llx], 0x%08llx bytes flags: "
a comma is missing before flags.

To avoid increasing the code size replace '0x%' by '%#'.

Printing the size with leading zeros but not the addresses does not really
make sense. Remove the leading zeros from the size output.

Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
---
v2:
        Remove leading zeros from size field.
---
 lib/lmb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/lmb.c b/lib/lmb.c
index 8b306e4de8e..48682f98b3a 100644
--- a/lib/lmb.c
+++ b/lib/lmb.c
@@ -90,7 +90,7 @@ static void lmb_dump_region(struct alist *lmb_rgn_lst, char 
*name)
        enum lmb_flags flags;
        int i;
 
-       printf(" %s.count = 0x%x\n", name, lmb_rgn_lst->count);
+       printf(" %s.count = %#x\n", name, lmb_rgn_lst->count);
 
        for (i = 0; i < lmb_rgn_lst->count; i++) {
                base = rgn[i].base;
@@ -98,7 +98,7 @@ static void lmb_dump_region(struct alist *lmb_rgn_lst, char 
*name)
                end = base + size - 1;
                flags = rgn[i].flags;
 
-               printf(" %s[%d]\t[0x%llx-0x%llx], 0x%08llx bytes flags: ",
+               printf(" %s[%d]\t[%#llx-%#llx], %#llx bytes, flags: ",
                       name, i, base, end, size);
                lmb_print_region_flags(flags);
        }
-- 
2.45.2

Reply via email to