<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 14, 2023 at 9:23 AM Hesham Almatary <<a href="mailto:heshamelmatary@gmail.com">heshamelmatary@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 14 Mar 2023 at 14:14, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
><br>
> ---<br>
>  config/rtems-bsps-riscv.ini | 16 +++++++++++-----<br>
>  1 file changed, 11 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/config/rtems-bsps-riscv.ini b/config/rtems-bsps-riscv.ini<br>
> index 804245e..816675f 100644<br>
> --- a/config/rtems-bsps-riscv.ini<br>
> +++ b/config/rtems-bsps-riscv.ini<br>
> @@ -1,6 +1,7 @@<br>
>  #<br>
>  # RTEMS Tools Project (<a href="http://www.rtems.org/" rel="noreferrer" target="_blank">http://www.rtems.org/</a>)<br>
>  # Copyright 2018 embedded brains GmbH<br>
> +# Copyright 2022 OAR Corporation<br>
>  # All rights reserved.<br>
>  #<br>
>  # This file is part of the RTEMS Tools package in 'rtems-bsp-builder'.<br>
> @@ -22,8 +23,13 @@<br>
>  #<br>
>  [riscv]<br>
>  bsps = frdme310arty,<br>
> -       griscv, grv32i, grv32im, grv32imac, grv32imafdc,<br>
> -       rv32iac, rv32i, rv32imac, rv32imafc, rv32imafdc, rv32imafd, rv32im,<br>
> -       rv64imac, rv64imac_medany, rv64imafdc, rv64imafd, rv64imafdc_medany,<br>
> -       rv64imafd_medany<br>
> -exclude-smp = rv32i, rv32im<br>
> +    griscv, grv32i, grv32im, grv32imac, grv32imafdc,<br>
> +    mpfs64imafdc,<br>
> +    noel32im, noel32imafd, noel64imac, noel64imafd, noel64imafdc,<br>
> +    rv32i, rv32iac, rv32im, rv32imac, rv32imafc, rv32imafd, rv32imafdc,<br>
Could we make sure that only variants in multilib are here? For<br>
example, there shouldn't be rv32imafdc. I submitted a similar patch<br>
before here [1], please feel free to use it as a reference as in to<br>
what multilib variants should be used here.<br></blockquote><div><br></div><div>This list is based on what is currently in RTEMS. Kinsey and I started with the</div><div>list reported by "rtems-bsps -a riscv" and reviewed which has SMP as an option.</div><div><br></div><div>Given that you know the riscv much better, I would politely request that you let</div><div>me push this and then you follow up with patches to remove those BSPs from </div><div>rtems and rtems-tools.</div><div><br></div><div>--joel</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
[1] <a href="https://lists.rtems.org/pipermail/devel/2023-February/074455.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2023-February/074455.html</a><br>
<br>
> +    rv64imac, rv64imafd, rv64imafdc<br>
> +<br>
> +exclude-smp = frdme310arty,<br>
> +    grv32i, grv32im,<br>
> +    noel32im,<br>
> +    rv32i, rv32im<br>
> --<br>
> 1.8.3.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
<br>
<br>
<br>
-- <br>
Regards,<br>
Hesham<br>
</blockquote></div></div>