Re: [PATCH v5 0/3] riscv_load_fdt() semantics change

[Alistair Francis]

On Thu, Feb 2, 2023 at 3:12 AM Daniel Henrique Barboza
<dbarboza@ventanamicro.com> wrote:
>
> Hi,
>
> This version contains changed proposed by Bin in the v4 review.
>
> Patches are based on riscv-to-apply.next.
>
> Changes from v4:
> - patch 1:
>   - remove the fdt_pack() related lines instead of letting patch 2 do it
> - patch 2:
>   - make riscv_compute_fdt_addr() return an uint64_t
> - patch 3:
>   - do not change riscv_compute_fdt_addr() return type to hwaddr
>   - fix comment typo
>   - use a ternary instead of if/else in dram_end
> - v4 link: https://lists.gnu.org/archive/html/qemu-devel/2023-01/msg05828.html
>
> Daniel Henrique Barboza (3):
>   hw/riscv/boot.c: calculate fdt size after fdt_pack()
>   hw/riscv: split fdt address calculation from fdt load
>   hw/riscv: change riscv_compute_fdt_addr() semantics

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  hw/riscv/boot.c            | 53 +++++++++++++++++++++++++++++++-------
>  hw/riscv/microchip_pfsoc.c |  7 +++--
>  hw/riscv/sifive_u.c        |  8 +++---
>  hw/riscv/spike.c           |  7 ++---
>  hw/riscv/virt.c            |  8 +++---
>  include/hw/riscv/boot.h    |  4 ++-
>  6 files changed, 65 insertions(+), 22 deletions(-)
>
> --
> 2.39.1
>
>