davids5 commented on a change in pull request #3161:
URL: https://github.com/apache/incubator-nuttx/pull/3161#discussion_r601395959



##########
File path: arch/arm/src/stm32f7/stm32_adc.c
##########
@@ -735,7 +735,7 @@ static int adc_timinit(FAR struct stm32_dev_s *priv)
    *   position.
    */
 
-  ainfo("Initializing timers extsel = 0x%08x\n", priv->extsel);
+  ainfo("Initializing timers extsel = 0x%08lx\n", priv->extsel);

Review comment:
       @gustavonihei  I asked a question on the usage in the low level code 
[here]( 
https://github.com/apache/incubator-nuttx/pull/3169#issuecomment-806025894)
   
   Would mind responding there as well. I am just looking for clarity on 
establishing the pattern and applying it consistently. 
   
   Is it as simple as all format specifiers should be replaced everywhere?
   




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to