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

 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

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
>
>