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 2022/01/14 13:18:49 UTC

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5228: Support both rv32/rv64 on qemu-rv

no1wudi commented on a change in pull request #5228:
URL: https://github.com/apache/incubator-nuttx/pull/5228#discussion_r784835433



##########
File path: arch/risc-v/Kconfig
##########
@@ -113,8 +113,6 @@ config ARCH_CHIP_RV32M1
 config ARCH_CHIP_QEMU_RV32
 	bool "QEMU RV32"
 	select ARCH_RV32
-	select ARCH_RV_ISA_M
-	select ARCH_RV_ISA_A

Review comment:
       Now these options moved to `qemu-rv`'s Kconfig to provide more flexiblity.
   ```
   config ARCH_CHIP_QEMU_RV_ISA_M
       bool "Standard Extension for Integer Multiplication and Division"
       default y
       select ARCH_RV_ISA_M
   
   config ARCH_CHIP_QEMU_RV_ISA_A
       bool "Standard Extension for Atomic Instructions"
       default y
       select ARCH_RV_ISA_A
   
   config ARCH_CHIP_QEMU_RV_ISA_C
       bool "Standard Extension for Compressed Instructions"
       default y
       select ARCH_RV_ISA_C
   ```




-- 
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