davids5 commented on pull request #5012: URL: https://github.com/apache/incubator-nuttx/pull/5012#issuecomment-996180290
> @davids5, then we are removing GPIO_SPEED_50MHz,GPIO_PUSHPULL,GPIO_PULLUP from all SDIO lines (not only GPIO_SDMMC*_CK) in stm32h7x3xx_pinmap.h, and renaming all the constants (adding _< number >), correct? It was: > > ``` > #define GPIO_SDMMC2_CK_1 (GPIO_ALT|GPIO_AF11|GPIO_SPEED_50MHz|GPIO_PORTD|GPIO_PIN6) > #define GPIO_SDMMC2_CK_2 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PORTC|GPIO_PIN1) > #define GPIO_SDMMC2_CMD_1 (GPIO_ALT|GPIO_AF11|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PULLUP|GPIO_PORTD|GPIO_PIN7) > #define GPIO_SDMMC2_CMD_2 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTA|GPIO_PIN0) > #define GPIO_SDMMC2_D0 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN14) > #define GPIO_SDMMC2_D1 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN15) > #define GPIO_SDMMC2_D2_1 (GPIO_ALT|GPIO_AF10|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTG|GPIO_PIN11) > #define GPIO_SDMMC2_D2_2 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN3) > #define GPIO_SDMMC2_D3 (GPIO_ALT|GPIO_AF9|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN4) > #define GPIO_SDMMC2_D4 (GPIO_ALT|GPIO_AF10|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN8) > #define GPIO_SDMMC2_D5 (GPIO_ALT|GPIO_AF10|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTB|GPIO_PIN9) > #define GPIO_SDMMC2_D6 (GPIO_ALT|GPIO_AF10|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTC|GPIO_PIN6) > #define GPIO_SDMMC2_D7 (GPIO_ALT|GPIO_AF10|GPIO_SPEED_50MHz|GPIO_PUSHPULL|GPIO_PORTC|GPIO_PIN7) > ``` > > It will be: > > ``` > #define GPIO_SDMMC2_CK_1 (GPIO_ALT|GPIO_AF11|GPIO_PORTD|GPIO_PIN6) > #define GPIO_SDMMC2_CK_2 (GPIO_ALT|GPIO_AF9|GPIO_PORTC|GPIO_PIN1) > #define GPIO_SDMMC2_CMD_1 (GPIO_ALT|GPIO_AF11|GPIO_PORTD|GPIO_PIN7) > #define GPIO_SDMMC2_CMD_2 (GPIO_ALT|GPIO_AF9|GPIO_PORTA|GPIO_PIN0) > #define GPIO_SDMMC2_D0_1 (GPIO_ALT|GPIO_AF9|GPIO_PORTB|GPIO_PIN14) > #define GPIO_SDMMC2_D1_1 (GPIO_ALT|GPIO_AF9|GPIO_PORTB|GPIO_PIN15) > #define GPIO_SDMMC2_D2_1 (GPIO_ALT|GPIO_AF10|GPIO_PORTG|GPIO_PIN11) > #define GPIO_SDMMC2_D2_2 (GPIO_ALT|GPIO_AF9|GPIO_PORTB|GPIO_PIN3) > #define GPIO_SDMMC2_D3_1 (GPIO_ALT|GPIO_AF9|GPIO_PORTB|GPIO_PIN4) > #define GPIO_SDMMC2_D4_1 (GPIO_ALT|GPIO_AF10|GPIO_PORTB|GPIO_PIN8) > #define GPIO_SDMMC2_D5_1 (GPIO_ALT|GPIO_AF10|GPIO_PORTB|GPIO_PIN9) > #define GPIO_SDMMC2_D6_1 (GPIO_ALT|GPIO_AF10|GPIO_PORTC|GPIO_PIN6) > #define GPIO_SDMMC2_D7_1 (GPIO_ALT|GPIO_AF10|GPIO_PORTC|GPIO_PIN7) > ``` > > And similar changes are for all the other interfaces (CAN, I2C etc.). Yes that is the idea. It is a breaking change and has to be documented very well. So that people can get back to working and know what slew rates they had. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
