xiaoxiang781216 commented on a change in pull request #5761: URL: https://github.com/apache/incubator-nuttx/pull/5761#discussion_r833050660
########## File path: arch/arm/src/armv7-a/arm_schedulesigaction.c ########## @@ -326,13 +329,14 @@ void up_schedule_sigaction(struct tcb_s *tcb, sig_deliver_t sigdeliver) * been delivered. */ - CURRENT_REGS = - (FAR void *)STACK_ALIGN_DOWN((uint32_t)CURRENT_REGS - - (uint32_t)XCPTCONTEXT_SIZE); + CURRENT_REGS = (FAR void *) Review comment: FAR is still used in arm, it's better to remove all FAR in one patch. -- 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