On Thu, Jan 23, 2020 at 12:37:44PM +0100, 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. > > PS: > remove no longer needed MCIMX7Sabre > > Signed-off-by: Igor Mammedov <imamm...@redhat.com> > --- > CC: drjo...@redhat.com > CC: andrew.smir...@gmail.com > CC: peter.mayd...@linaro.org > CC: qemu-...@nongnu.org > --- > hw/arm/mcimx7d-sabre.c | 25 +++++++++---------------- > 1 file changed, 9 insertions(+), 16 deletions(-) >
I don't know anything about mcimx7d-sabre, but I promised Igor to put another pair of eyes on his changes. Looks fine to me. Reviewed-by: Andrew Jones <drjo...@redhat.com>
