Re: [Qemu-arm] [Qemu-devel] [PATCH v5 2/2] hw/arm: add Arm SBSA reference machine, devices part

[Peter Maydell]

On Wed, 30 Jan 2019 at 08:23, Igor Mammedov <address@hidden> wrote:
>
> On Tue, 29 Jan 2019 15:26:29 +0000
> Peter Maydell <address@hidden> wrote:
> > Oh, I see now. I think this is a bug in boot.c and we should
> > fix it by making the "this is a firmware boot" code path also
> > do the arm_load_dtb() call. I'll send a patch.
> pls, CC me on it as well.
>
> I'm interested in it from hotplug + reboot perspective,
> where we have a changed/bigger DTB on reboot as it contains new devices 
> (cpus|memory)
>
> loading rom like now doesn't work nice in case of reboot.

OK, but I don't think it will help you there, because all it
does (after a pile of refactoring) is make the firmware-boot
code path go through the if() clause at the bottom of
arm_load_kernel() that calls arm_load_dtb().

thanks
-- PMM