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/08 03:15:35 UTC

[GitHub] [incubator-nuttx] no1wudi commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

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



##########
File path: arch/risc-v/include/common/irq.h
##########
@@ -121,7 +121,13 @@
   #define INT_XCPT_REGS     33
 #endif
 
-#define INT_XCPT_SIZE       (8 * INT_XCPT_REGS)
+#ifdef CONFIG_ARCH_RV32
+#define INT_REG_SIZE        4
+#else
+#define INT_REG_SIZE        8
+#endif

Review comment:
       Can't do this since uintptr_t is invalid in assembly.




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