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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to