TimJTi commented on code in PR #7702:
URL: https://github.com/apache/nuttx/pull/7702#discussion_r1038782007
##########
arch/arm/src/sama5/sam_adc.c:
##########
@@ -865,7 +935,7 @@ static void sam_adc_endconversion(void *arg)
int ret;
DEBUGASSERT(priv != NULL);
- ainfo("pending=%08x\n", priv->pending);
+ ainfo("pending=%08lx\n", priv->pending);
Review Comment:
Thanks! Because the SAMA5s are 32 bit devices (and I have only ever written
serious code for 32 bit devices) the %08lx is technically correct I think? But
I understand that anything that makes code clearer to any coder is a good
thing. I will adopt this and VERY much appreciate the clear explanation :)
--
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
