Re: [PATCH v4 20/80] arm/mps2-tz: use memdev for RAM

[Andrew Jones]

On Thu, Jan 23, 2020 at 03:37:11PM +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:
>  while at it add check for user supplied RAM size and error
>  out if it mismatches board expected value.
> 
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> v2:
>   * fix format string causing build failure on 32-bit host
>     (Philippe Mathieu-Daudé <address@hidden>)
> v4:
>   * move default_ram_size to mps2tz_class_init()
>     (Andrew Jones <address@hidden>)
> 
> CC: address@hidden
> CC: address@hidden
> CC: address@hidden
> 
> mps2-tz
> ---
>  hw/arm/mps2-tz.c | 15 +++++++++++----
>  1 file changed, 11 insertions(+), 4 deletions(-)
>

Reviewed-by: Andrew Jones <address@hidden>