[GitHub] [incubator-nuttx] pkarashchenko commented on a change in pull request #5192: arch/risc-v: Merge rv32im and rv64gc into common

[GitBox]

pkarashchenko commented on a change in pull request #5192:
URL: https://github.com/apache/incubator-nuttx/pull/5192#discussion_r780529480



##########
File path: arch/risc-v/src/common/riscv_swint.c
##########
@@ -455,7 +455,7 @@ int riscv_swint(int irq, void *context, void *arg)
 
           rtcb->flags         |= TCB_FLAG_SYSCALL;
 #else
-          svcerr("ERROR: Bad SYS call: %" PRId64 "\n", regs[REG_A0]);
+          svcerr("ERROR: Bad SYS call: %" PRIdPTR "\n", regs[REG_A0]);

Review comment:
       Maybe can switch to `PRIxREG` here.
   Also rework `static void riscv_registerdump(const uint64_t *regs)` to use 
`uintptr_t`.




-- 
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