no1wudi commented on a change in pull request #5192: URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r780722927
########## File path: arch/risc-v/src/litex/litex_irq_dispatch.c ########## @@ -114,13 +114,13 @@ void *litex_dispatch_irq(uint32_t vector, uint32_t *regs) #endif /* If a context switch occurred while processing the interrupt then - * g_current_regs may have change value. If we return any value different + * CURRENT_REGS may have change value. If we return any value different * from the input regs, then the lower level will know that a context * switch occurred during interrupt processing. */ - regs = (uint32_t *)g_current_regs; - g_current_regs = NULL; + regs = (uintptr_t *)CURRENT_REGS; Review comment: We should keep this cast to avoid warning "assignment discards 'volatile' qualifier from pointer target type [-Wdiscarded-qualifiers]". -- 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