You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/12/23 08:15:57 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new issue #2593: Merge rv32im and rv64gc into one

xiaoxiang781216 opened a new issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593


   32bit and 64bit riscv share the same specification, it doesn't make sense to support two most identical arch by two code base.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] btashton commented on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
btashton commented on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-761623153


   @xiaoxiang781216 I am fine with this change.  `nr5m100` was RTL was never released publically and Ken never responded, and the GAP8 while real, I think is also unused and we already heard back from the contributor that they were fine.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] btashton commented on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
btashton commented on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-750005340


   We should be aware that some cores do have some ISA extensions (like GAP8) I'm not sure how much that will matter.
   The rv32e has different a register count and some other limitations, but I don't know of any publicly available chips using this yet.
   https://github.com/lowRISC/ibex has a "micro" mode that use rv32e so that could probably used for a test on an FPGA.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-761623485


   @btashton Ok, I will found time to merge 32bit and 64bit into one.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] btashton edited a comment on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-761623153


   @xiaoxiang781216 I am fine with this change.  `nr5m100` RTL was never released publicly and Ken never responded. While the GAP8 is real, I think is also unused and we already heard back from the contributor that they were fine, it an early PULP design, that I think they are replacing with a new chip.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-750012170


   Yes, but these variants are minor and can be controlled by Kconfig. Since the spec is controlled by a mature committee who are considering the compatibility seriously, it expect that the potential conflict between 32bit and 64bit should be very small. On the other hand, it's better to support the experimental/predated implementation by the duplicated arch code.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] btashton edited a comment on issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
btashton edited a comment on issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593#issuecomment-761623153


   @xiaoxiang781216 I am fine with this change.  `nr5m100` RTL was never released publicly and Ken never responded, and the GAP8 while real, I think is also unused and we already heard back from the contributor that they were fine.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] xiaoxiang781216 closed issue #2593: Merge rv32im and rv64gc into one

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 closed issue #2593:
URL: https://github.com/apache/incubator-nuttx/issues/2593


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org