On 11/1/23 18:09, Daniel Henrique Barboza wrote:
Hi,
This is a follow-up of:
"[PATCH v5 00/11] riscv: OpenSBI boot test and cleanups"
Patches were based on top of riscv-to-apply.next [1] + the series above.
The recent FDT changes made in hw/riscv (all machines are now using the
FDT via MachineState::fdt) allowed for most of the cleanups made here.
Patches 9 and 10 were based on a suggestion made by Phil a few weeks ago.
I decided to go for it.
[1] https://github.com/alistair23/qemu/tree/riscv-to-apply.next
Daniel Henrique Barboza (10):
hw/riscv/spike.c: simplify create_fdt()
hw/riscv/virt.c: simplify create_fdt()
hw/riscv/sifive_u.c: simplify create_fdt()
hw/riscv/virt.c: remove 'is_32_bit' param from
create_fdt_socket_cpus()
hw/riscv: use MachineState::fdt in riscv_socket_fdt_write_id()
hw/riscv: use ms->fdt in riscv_socket_fdt_write_distance_matrix()
hw/riscv: simplify riscv_load_fdt()
hw/riscv/virt.c: calculate socket count once in create_fdt_imsic()
hw/riscv/virt.c: rename MachineState 'mc' pointers to 'ms'
hw/riscv/spike.c: rename MachineState 'mc' pointers to' ms'
Patch 7 likely needs rework (problem predating your series).
Meanwhile for patches 1-6 & 8-10:
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Thanks for this cleanup!