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


Reply via email to