[PATCH 02/11] lib: sbi: Remove 's' and 'u' from misa_string() output

Anup Patel anup at brainfault.org
Fri May 6 21:58:19 PDT 2022


On Wed, May 4, 2022 at 7:12 AM Atish Patra <atishp at atishpatra.org> wrote:
>
> On Fri, Apr 29, 2022 at 8:52 AM Anup Patel <apatel at ventanamicro.com> wrote:
> >
> > Both 's' and 'u' are not treated as ISA extensions since these are
> > privilege modes so let's remove it from misa_string() output.
> >
> > Signed-off-by: Anup Patel <apatel at ventanamicro.com>
> > ---
> >  lib/sbi/riscv_asm.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/lib/sbi/riscv_asm.c b/lib/sbi/riscv_asm.c
> > index 5eab1ed..0ebf41d 100644
> > --- a/lib/sbi/riscv_asm.c
> > +++ b/lib/sbi/riscv_asm.c
> > @@ -53,7 +53,7 @@ int misa_xlen(void)
> >  void misa_string(int xlen, char *out, unsigned int out_sz)
> >  {
> >         unsigned int i, pos = 0;
> > -       const char valid_isa_order[] = "iemafdqclbjtpvnsuhkorwxyzg";
> > +       const char valid_isa_order[] = "iemafdqclbjtpvnhkorwxyzg";
> >
> >         if (!out)
> >                 return;
> > --
> > 2.34.1
> >
>
>
> Reviewed-by: Atish Patra <atishp at rivosinc.com>

Applied this patch to the riscv/opensbi repo

Regards,
Anup

>
> --
> Regards,
> Atish



More information about the opensbi mailing list