You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/12/16 20:44:10 UTC
[GitHub] [incubator-nuttx] davids5 commented on pull request #5012: Revert "stm32h7 sdmmc: set SDMMC_CK pin to high speed (50 MHz) mode."
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org