no1wudi commented on a change in pull request #5645: URL: https://github.com/apache/incubator-nuttx/pull/5645#discussion_r822252911
########## File path: arch/arm/src/common/arm_internal.h ########## @@ -306,14 +307,10 @@ void arm_boot(void); /* Context switching */ -void arm_copyfullstate(uint32_t *dest, uint32_t *src); -#ifdef CONFIG_ARCH_FPU -void arm_copyarmstate(uint32_t *dest, uint32_t *src); -#endif uint32_t *arm_decodeirq(uint32_t *regs); int arm_saveusercontext(uint32_t *saveregs); void arm_fullcontextrestore(uint32_t *restoreregs) noreturn_function; -void arm_switchcontext(uint32_t *saveregs, uint32_t *restoreregs); +void arm_switchcontext(uint32_t **saveregs, uint32_t *restoreregs); Review comment: Should we add `FAR` to arch specific code ? It's no sense on arm (empty defination). -- 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-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org