summaryrefslogtreecommitdiffstats
path: root/target/arm/cpu.h
diff options
context:
space:
mode:
authorPeter Maydell2019-02-05 17:52:42 +0100
committerPeter Maydell2019-02-05 17:52:42 +0100
commit5614ca800e05dc07e4045b7738351058538c6079 (patch)
tree198267c0c2417ad1376a17391aa606affa5fdc14 /target/arm/cpu.h
parenthw/arm/boot: Clarify why arm_setup_firmware_boot() doesn't set env->boot_info (diff)
downloadqemu-5614ca800e05dc07e4045b7738351058538c6079.tar.gz
qemu-5614ca800e05dc07e4045b7738351058538c6079.tar.xz
qemu-5614ca800e05dc07e4045b7738351058538c6079.zip
hw/arm/boot: Support DTB autoload for firmware-only boots
The arm_boot_info struct has a skip_dtb_autoload flag: if this is set to true by the board code then arm_load_kernel() will not load the DTB itself, but will leave this for the board code to do itself later. However, the check for this is done in a code path which is only executed for the case where we load a kernel image file. If we're taking the "boot via firmware" code path then the flag isn't honoured and the DTB is never loaded. We didn't notice this because the only real user of "boot via firmware" that cares about the DTB is the virt board (for UEFI boot), and that always wants skip_dtb_autoload anyway. But the SBSA reference board model we're planning to add will want the flag to behave correctly. Now we've refactored the arm_load_kernel() function, the fix is simple: drop the early 'return' so we fall into the same "load the DTB" code the boot-direct-kernel path uses. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Igor Mammedov <imammedo@redhat.com> Message-id: 20190131112240.8395-6-peter.maydell@linaro.org
Diffstat (limited to 'target/arm/cpu.h')
0 files changed, 0 insertions, 0 deletions