On Mon, Mar 05, 2018 at 08:27:24AM -0300, Philippe Mathieu-Daudé wrote: > It eases code review, unit is explicit. > > Patch generated using: > > $ git grep -E '(1024|2048|4096|8192|(<<|>>).?(10|20|30))' hw/ include/hw/ > > and modified manually. > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > Reviewed-by: Gerd Hoffmann <kra...@redhat.com> > --- > hw/display/xenfb.c | 2 +- > diff --git a/hw/display/xenfb.c b/hw/display/xenfb.c > index f5afcc0358..1ae660519a 100644 > --- a/hw/display/xenfb.c > +++ b/hw/display/xenfb.c > @@ -889,7 +889,7 @@ static int fb_initialise(struct XenDevice *xendev) > return rc; > > fb_page = fb->c.page; > - rc = xenfb_configure_fb(fb, videoram * 1024 * 1024U, > + rc = xenfb_configure_fb(fb, videoram * M_BYTE, > fb_page->width, fb_page->height, fb_page->depth, > fb_page->mem_length, 0, fb_page->line_length); > if (rc != 0)
Acked-by: Anthony PERARD <anthony.per...@citrix.com> Thanks, -- Anthony PERARD