Re: [PATCH REPOST v3 06/80] alpha:dp264: use memdev for RAM

[Igor Mammedov]

On Thu, 23 Jan 2020 16:07:19 +0100 (CET)
BALATON Zoltan <address@hidden> 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 <address@hidden>
> > Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> > Tested-by: Philippe Mathieu-Daudé <address@hidden>
> > Acked-by: Richard Henderson <address@hidden>
> > ---
> > 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