[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 35/86] arm:xilinx_zynq: use memdev for RAM
From: |
Alistair Francis |
Subject: |
Re: [PATCH v2 35/86] arm:xilinx_zynq: use memdev for RAM |
Date: |
Thu, 16 Jan 2020 10:20:20 +1000 |
On Thu, Jan 16, 2020 at 1:29 AM Igor Mammedov <address@hidden> 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: Alistair Francis <address@hidden>
Alistair
> ---
> CC: address@hidden
> CC: address@hidden
> CC: address@hidden
> CC: address@hidden
> ---
> hw/arm/xilinx_zynq.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c
> index df950fc..0ef9688 100644
> --- a/hw/arm/xilinx_zynq.c
> +++ b/hw/arm/xilinx_zynq.c
> @@ -160,7 +160,6 @@ static void zynq_init(MachineState *machine)
> {
> ARMCPU *cpu;
> MemoryRegion *address_space_mem = get_system_memory();
> - MemoryRegion *ext_ram = g_new(MemoryRegion, 1);
> MemoryRegion *ocm_ram = g_new(MemoryRegion, 1);
> DeviceState *dev;
> SysBusDevice *busdev;
> @@ -190,9 +189,7 @@ static void zynq_init(MachineState *machine)
> object_property_set_bool(OBJECT(cpu), true, "realized", &error_fatal);
>
> /* DDR remapped to address zero. */
> - memory_region_allocate_system_memory(ext_ram, NULL, "zynq.ext_ram",
> - machine->ram_size);
> - memory_region_add_subregion(address_space_mem, 0, ext_ram);
> + memory_region_add_subregion(address_space_mem, 0, machine->ram);
>
> /* 256K of on-chip memory */
> memory_region_init_ram(ocm_ram, NULL, "zynq.ocm_ram", 256 * KiB,
> @@ -318,6 +315,7 @@ static void zynq_machine_init(MachineClass *mc)
> mc->no_sdcard = 1;
> mc->ignore_memory_transaction_failures = true;
> mc->default_cpu_type = ARM_CPU_TYPE_NAME("cortex-a9");
> + mc->default_ram_id = "zynq.ext_ram";
> }
>
> DEFINE_MACHINE("xilinx-zynq-a9", zynq_machine_init)
> --
> 2.7.4
>
>
- [PATCH v2 30/86] arm:sbsa-ref: use memdev for RAM, (continued)
- [PATCH v2 30/86] arm:sbsa-ref: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 32/86] arm:vexpress: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 36/86] arm:xlnx-versal-virt: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 33/86] arm:virt: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 34/86] arm:xilinx_zynq: drop RAM size fixup, Igor Mammedov, 2020/01/15
- [PATCH v2 35/86] arm:xilinx_zynq: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 42/86] hw/hppa/machine: Restrict the total memory size to 3GB, Igor Mammedov, 2020/01/15
- [PATCH v2 39/86] null-machine: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 37/86] arm:xlnx-zcu102: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 40/86] cris:axis_dev88: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 43/86] hw/hppa/machine: Map the PDC memory region with higher priority, Igor Mammedov, 2020/01/15
- [PATCH v2 45/86] x86:microvm: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 44/86] hppa: use memdev for RAM, Igor Mammedov, 2020/01/15