Re: [PATCH 1/2] target/riscv: Remove redundant check in pmp_is_locked

2023-07-02 Thread Alistair Francis
On Wed, Jun 28, 2023 at 11:34 PM Ruibo Lu wrote: > > the check of top PMP is redundant and will not influence the return > value, so consider remove it > > Signed-off-by: Ruibo Lu Reviewed-by: Alistair Francis Alistair > --- > target/riscv/pmp.c | 5 - > 1 file changed, 5 deletions(-) >

[PATCH 1/2] target/riscv: Remove redundant check in pmp_is_locked

2023-06-28 Thread Ruibo Lu
the check of top PMP is redundant and will not influence the return value, so consider remove it Signed-off-by: Ruibo Lu --- target/riscv/pmp.c | 5 - 1 file changed, 5 deletions(-) diff --git a/target/riscv/pmp.c b/target/riscv/pmp.c index 9d8db493e6..1a9279ba88 100644 --- a/target/riscv/p

Re: [PATCH 1/2] target/riscv: Remove redundant check in pmp_is_locked

2023-06-28 Thread Weiwei Li
On 2023/6/28 18:36, Ruibo Lu wrote: the check of top PMP is redundant and will not influence the return value, so consider remove it Signed-off-by: Ruibo Lu --- Reviewed-by: Weiwei Li Weiwei Li target/riscv/pmp.c | 5 - 1 file changed, 5 deletions(-) diff --git a/target/riscv/pmp.