On Sun, Oct 20, 2019 at 3:58 PM Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > IEC binary prefixes ease code review: the unit is explicit. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > hw/arm/xilinx_zynq.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c > index c14774e542..3a0fa5b23f 100644 > --- a/hw/arm/xilinx_zynq.c > +++ b/hw/arm/xilinx_zynq.c > @@ -16,6 +16,7 @@ > */ > > #include "qemu/osdep.h" > +#include "qemu/units.h" > #include "qapi/error.h" > #include "cpu.h" > #include "hw/sysbus.h" > @@ -194,7 +195,7 @@ static void zynq_init(MachineState *machine) > memory_region_add_subregion(address_space_mem, 0, ext_ram); > > /* 256K of on-chip memory */ > - memory_region_init_ram(ocm_ram, NULL, "zynq.ocm_ram", 256 << 10, > + memory_region_init_ram(ocm_ram, NULL, "zynq.ocm_ram", 256 * KiB, > &error_fatal); > memory_region_add_subregion(address_space_mem, 0xFFFC0000, ocm_ram); > > -- > 2.21.0 > >