On Thu, 23 Jan 2020 16:07:19 +0100 (CET) BALATON Zoltan <bala...@eik.bme.hu> wrote:
> On Thu, 23 Jan 2020, Igor Mammedov wrote: > > memory_region_allocate_system_memory() API is going away, so > > replace it with memdev allocated MemoryRegion. The later is > > initialized by generic code, so board only needs to opt in > > to memdev scheme by providing > > MachineClass::default_ram_id > > and using MachineState::ram instead of manually initializing > > RAM memory region. > > > > Signed-off-by: Igor Mammedov <imamm...@redhat.com> > > Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > > Tested-by: Philippe Mathieu-Daudé <phi...@redhat.com> > > Acked-by: Richard Henderson <richard.hender...@linaro.org> > > --- > > hw/alpha/alpha_sys.h | 2 +- > > hw/alpha/dp264.c | 3 ++- > > hw/alpha/typhoon.c | 8 ++------ > > 3 files changed, 5 insertions(+), 8 deletions(-) > > This still has a : instead of / in the patch title. Maybe should be > alpha/dp264: ... Probably does not worth a respin but you could correct it > if there will be another version for some other reason or when applying I plan to respin series, so it will be fixed up in v4 Thanks > it. > > Regrards, > BALATON Zoltan