hartmannathan commented on a change in pull request #745: [DO NOT MERGE] 
arch/stm32h7: Explicit support for all STM32H7x3xx
URL: https://github.com/apache/incubator-nuttx/pull/745#discussion_r407776951
 
 

 ##########
 File path: arch/arm/include/stm32h7/chip.h
 ##########
 @@ -53,16 +53,40 @@
  *
  *   PART        PACKAGE
  *   ----------- ----------------
+ *   STM32H7x3Ax UFBGA169
 
 Review comment:
   @davids5 - The latest commit I pushed to this PR fixes the TODO IO_CONFIG 
items in the STM32H7 Kconfig and defines STM32H7_NGPIO according to those 
IO_CONFIGs. Please give me your opinion on this.
   
   Also, as I mentioned before, the 100-pin IO_CONFIG_V parts have a gap in the 
port letters, it has GPIOA-GPIOH, but it's missing GPIOF-GPIOG. Should this be 
handled, and what would you recommend?
   
   Thanks

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to