[PATCH v2] lib: sbi_hart: Shorten the code to set MPV bit

[Anup Patel]

On Thu, Jul 14, 2022 at 11:00 PM gagachang <vivahavey at gmail.com> wrote:
>
> MPV bit is set when the value of next_virt boolean variable equals
> true. Since the value of next_virt is either 0 or 1, we can set
> MPV bit without if-else logic.
>
> Signed-off-by: Che-Chia Chang <alvinga at andestech.com>
> Reviewed-by: Andrew Jones <ajones at ventanamicro.com>

Looks good to me.

Reviewed-by: Anup Patel <anup at brainfault.org>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

>
> --
> Change since v1:
> - Drop the unnecessary brackets
> ---
>  lib/sbi/sbi_hart.c | 13 +++----------
>  1 file changed, 3 insertions(+), 10 deletions(-)
>
> diff --git a/lib/sbi/sbi_hart.c b/lib/sbi/sbi_hart.c
> index de86fee..67b685a 100644
> --- a/lib/sbi/sbi_hart.c
> +++ b/lib/sbi/sbi_hart.c
> @@ -769,19 +769,12 @@ sbi_hart_switch_mode(unsigned long arg0, unsigned long arg1,
>  #if __riscv_xlen == 32
>         if (misa_extension('H')) {
>                 valH = csr_read(CSR_MSTATUSH);
> -               if (next_virt)
> -                       valH = INSERT_FIELD(valH, MSTATUSH_MPV, 1);
> -               else
> -                       valH = INSERT_FIELD(valH, MSTATUSH_MPV, 0);
> +               valH = INSERT_FIELD(valH, MSTATUSH_MPV, next_virt);
>                 csr_write(CSR_MSTATUSH, valH);
>         }
>  #else
> -       if (misa_extension('H')) {
> -               if (next_virt)
> -                       val = INSERT_FIELD(val, MSTATUS_MPV, 1);
> -               else
> -                       val = INSERT_FIELD(val, MSTATUS_MPV, 0);
> -       }
> +       if (misa_extension('H'))
> +               val = INSERT_FIELD(val, MSTATUS_MPV, next_virt);
>  #endif
>         csr_write(CSR_MSTATUS, val);
>         csr_write(CSR_MEPC, next_addr);
> --
> 2.25.1
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi