anchao commented on code in PR #13529:
URL: https://github.com/apache/nuttx/pull/13529#discussion_r1766384473
##########
arch/arm/include/armv7-a/irq.h:
##########
@@ -500,23 +500,15 @@ static inline_function uint32_t up_getsp(void)
noinstrument_function
static inline_function uint32_t *up_current_regs(void)
{
- uint32_t *regs;
- __asm__ __volatile__
- (
- "mrc " "p15, " "0" ", %0, " "c13" ", " "c0" ", " "4" "\n"
- : "=r"(regs)
- );
- return regs;
+ ARM_ISB();
Review Comment:
I just want to add the volatile attribute, it looks like it's already
included in the CP15 operation, let me revert
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
