LuchianMihai commented on PR #15861:
URL: https://github.com/apache/nuttx/pull/15861#issuecomment-2667954267

   Hi!
   Regarding the use of
   ```c
   #if BOARD_NGPIOINT > 0
   #endif
   ```
   It's a personal preference, but,
   I suggest the to use something like
   ```c
   #ifdef BOARD_NGPIOINT
   #endif
   ```
   If one or more gpio is needed, use
   ```c
   #define BOARD_NGPIOINT number
   ```
   else do not define BOARD_NGPIOINT at all.  
   
   Current implementation assumes that BOARD_NGPIOINT should always be defined 
and numeric value,  
   Which arguably may introduce undefined behavior.
   
   But, it's just my suggestion.


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

Reply via email to