[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 2/2] hw/arm: add Arm SBSA reference machine,
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v5 2/2] hw/arm: add Arm SBSA reference machine, devices part |
Date: |
Tue, 22 Jan 2019 11:49:44 +0000 |
On Fri, 7 Dec 2018 at 09:08, Hongbo Zhang <address@hidden> wrote:
>
> Following the previous patch, this patch adds peripheral devices to the
> newly introduced SBSA-ref machine.
>
> Signed-off-by: Hongbo Zhang <address@hidden>
> ---
> hw/arm/sbsa-ref.c | 421
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 421 insertions(+)
Most of this code looks good, thanks.
> +
> +static void sbsa_ref_machine_done(Notifier *notifier, void *data)
> +{
> + VirtMachineState *vms = container_of(notifier, VirtMachineState,
> + machine_done);
> + ARMCPU *cpu = ARM_CPU(first_cpu);
> + struct arm_boot_info *info = &vms->bootinfo;
> + AddressSpace *as = arm_boot_address_space(cpu, info);
> +
> + if (arm_load_dtb(info->dtb_start, info, info->dtb_limit, as) < 0) {
> + exit(1);
> + }
> +}
I still don't understand why we need this -- it doesn't seem
to be doing anything different from the default behaviour that
boot.c implements if skip_dtb_autoload is false.
thanks
-- PMM
- Re: [Qemu-devel] [PATCH v5 2/2] hw/arm: add Arm SBSA reference machine, devices part,
Peter Maydell <=