On Sat, 2 Nov 2024 at 06:41, Heinrich Schuchardt <[email protected]> wrote: > > 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 '%#'. > > Signed-off-by: Heinrich Schuchardt <[email protected]> > --- > lib/lmb.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/lmb.c b/lib/lmb.c > index 00c8888936a..95adf2e93a4 100644 > --- a/lib/lmb.c > +++ b/lib/lmb.c > @@ -97,7 +97,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; > @@ -105,7 +105,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], %#08llx bytes, flags: ", > name, i, base, end, size); > lmb_print_region_flags(flags); > } > -- > 2.45.2 >
Reviewed-by: Ilias Apalodimas <[email protected]>
