[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5066: arch/arm/cortex-a/r: dump all registers with alias

[GitBox <gi...@apache.org>]

xiaoxiang781216 commented on a change in pull request #5066:
URL: https://github.com/apache/incubator-nuttx/pull/5066#discussion_r774401294



##########
File path: arch/arm/src/armv7-a/arm_assert.c
##########
@@ -114,13 +112,24 @@ static inline void arm_registerdump(FAR volatile uint32_t *regs)
 
   /* Dump the interrupt registers */
 
-  for (reg = REG_R0; reg <= REG_R15; reg += 8)
-    {
-      uint32_t *ptr = (uint32_t *)&regs[reg];
-      _alert("R%d: %08x %08x %08x %08x %08x %08x %08x %08x\n",
-             reg, ptr[0], ptr[1], ptr[2], ptr[3],
-             ptr[4], ptr[5], ptr[6], ptr[7]);
-    }
+  _alert("R0: %08x R1: %08x R2: %08x  R3: %08x\n",
+         regs[REG_R0], regs[REG_R1], regs[REG_R2], regs[REG_R3]);
+  _alert("R4: %08x R5: %08x R6: %08x  "

Review comment:
       need update the old arm too




-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org