mbox series

[v2,0/2] target/riscv/cpu: fix sifive_u 32/64bits boot in riscv-to-apply.next

Message ID 20230113175230.473975-1-dbarboza@ventanamicro.com (mailing list archive)
Headers show
Series target/riscv/cpu: fix sifive_u 32/64bits boot in riscv-to-apply.next | expand

Message

Daniel Henrique Barboza Jan. 13, 2023, 5:52 p.m. UTC
Hi,

In this version I fixed the commit message typos pointed by Bin. I've
also added some notes about the code repetition the fix is introducing
in the cpu_init() functions.

The patches are based on riscv-to-apply.next at c1e76da3e668
("target/riscv/cpu.c: Fix elen check").

Changes from v1:
- patch 1:
  - fixed commit message typos
v1 review: https://lists.gnu.org/archive/html/qemu-devel/2023-01/msg02035.html

Daniel Henrique Barboza (2):
  target/riscv/cpu: set cpu->cfg in register_cpu_props()
  target/riscv/cpu.c: do not skip misa logic in riscv_cpu_realize()

 target/riscv/cpu.c | 439 +++++++++++++++++++++++++--------------------
 target/riscv/cpu.h |   4 +
 2 files changed, 249 insertions(+), 194 deletions(-)

Comments

Alistair Francis Jan. 16, 2023, 3:09 a.m. UTC | #1
On Sat, Jan 14, 2023 at 3:55 AM Daniel Henrique Barboza
<dbarboza@ventanamicro.com> wrote:
>
> Hi,
>
> In this version I fixed the commit message typos pointed by Bin. I've
> also added some notes about the code repetition the fix is introducing
> in the cpu_init() functions.
>
> The patches are based on riscv-to-apply.next at c1e76da3e668
> ("target/riscv/cpu.c: Fix elen check").
>
> Changes from v1:
> - patch 1:
>   - fixed commit message typos
> v1 review: https://lists.gnu.org/archive/html/qemu-devel/2023-01/msg02035.html
>
> Daniel Henrique Barboza (2):
>   target/riscv/cpu: set cpu->cfg in register_cpu_props()
>   target/riscv/cpu.c: do not skip misa logic in riscv_cpu_realize()

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  target/riscv/cpu.c | 439 +++++++++++++++++++++++++--------------------
>  target/riscv/cpu.h |   4 +
>  2 files changed, 249 insertions(+), 194 deletions(-)
>
> --
> 2.39.0
>
>