Re: [PATCH 4/4] target/riscv: Remove redundant assignment to SXL

2023-06-11 Thread LIU Zhiwei
On 2023/5/29 20:17, Weiwei Li wrote: SXL is initialized as env->misa_mxl which is also the mxl value. So we can just remain it unchanged to keep it read-only. Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- target/riscv/csr.c | 4 1 file changed, 4 deletions(-) diff --git

Re: [PATCH 4/4] target/riscv: Remove redundant assignment to SXL

2023-05-31 Thread Alistair Francis
On Mon, May 29, 2023 at 10:18 PM Weiwei Li wrote: > > SXL is initialized as env->misa_mxl which is also the mxl value. > So we can just remain it unchanged to keep it read-only. > > Signed-off-by: Weiwei Li > Signed-off-by: Junqiang Wang Reviewed-by: Alistair Francis Alistair > --- > target

Re: [PATCH 4/4] target/riscv: Remove redundant assignment to SXL

2023-05-30 Thread Daniel Henrique Barboza
On 5/29/23 09:17, Weiwei Li wrote: SXL is initialized as env->misa_mxl which is also the mxl value. So we can just remain it unchanged to keep it read-only. Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- Reviewed-by: Daniel Henrique Barboza target/riscv/csr.c | 4 1

[PATCH 4/4] target/riscv: Remove redundant assignment to SXL

2023-05-29 Thread Weiwei Li
SXL is initialized as env->misa_mxl which is also the mxl value. So we can just remain it unchanged to keep it read-only. Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- target/riscv/csr.c | 4 1 file changed, 4 deletions(-) diff --git a/target/riscv/csr.c b/target/riscv/csr.c i