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