On Mon, Jul 18, 2022 at 11:10 PM Weiwei Li <liweiwei@iscas.ac.cn> wrote:
>
> This patchset tries to improve the U/S/H related check:
> * add check for the supported privilege modes conbinations: only supporting M,
> S mode is not a suggested combination
> * add check for "H extension depends on I extension"
> * add check for "H extension implicitly requires S mode"
> * add check for csrs only existed when U mode is supported
> * fix and simplify the checks in hmode/hmode32
>
> Any suggestions are welcome.
>
> v3:
> * Fix the issues suggested by Andrew Jones in commit messages
>
> v2:
> * update patch 3 to make the newlines start at the same position as above line
> * update patch 5 to add check for "H extension implicitly requires S mode"
>
> Weiwei Li (6):
> target/riscv: Add check for supported privilege mode combinations
> target/riscv: H extension depends on I extension
> target/riscv: Fix checkpatch warning may triggered in csr_ops table
> target/riscv: Add check for csrs existed with U extension
> target/riscv: Fix checks in hmode/hmode32
> target/riscv: Simplify the check in hmode to resue the check in
> riscv_csrrw_check
Thanks!
Applied to riscv-to-apply.next
Alistair
>
> target/riscv/cpu.c | 17 ++
> target/riscv/csr.c | 490 ++++++++++++++++++++++++---------------------
> 2 files changed, 278 insertions(+), 229 deletions(-)
>
> --
> 2.17.1
>
>