Added peripheral support for Daisy Seed stm32H750 board #10198
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
snkYmkrct
commented
Mar 29, 2025
•
snkYmkrct
commented
- Added the SDRAM controller and set up heap to use the 64 MB external RAM
- Added the rest of the board pin definitions from the pinout
- Did peripheral pins cleanup for the stm32H750 chip
- Added SDMMC peripheral and sdioio module support
- Added pwmio support and started work on audiopwmio
snkYmkrct
changed the title
tannewt
requested changes
Mar 31, 2025
Comment on lines
34
to
68
|
|
||
| // SDRAM -- on FMC controller | ||
|
|
||
| #define CPY_SDRAM_REGION MPU_REGION_NUMBER10 | ||
| #define CPY_SDRAM_REGION_SIZE MPU_REGION_SIZE_64MB | ||
|
|
||
| #define CIRCUITPY_HW_FMC_SDCKE0 | ||
| #define CIRCUITPY_HW_FMC_SDNE0 | ||
|
|
||
| #define CIRCUITPY_HW_SDRAM_SIZE (64 * 1024 * 1024) // 64 MByte | ||
| #define CIRCUITPY_HW_SDRAM_STARTUP_TEST (1) | ||
| #define CIRCUITPY_HW_FMC_SWAP_BANKS (0) | ||
|
|
||
| #define CIRCUITPY_HW_SDRAM_CLOCK_PERIOD 2 | ||
| #define CIRCUITPY_HW_SDRAM_CAS_LATENCY 3 | ||
| #define CIRCUITPY_HW_SDRAM_FREQUENCY_KHZ (100000) // 100 MHz | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TMRD (2) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TXSR (8) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TRAS (5) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TRC (6) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TWR (3) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TRP (2) | ||
| #define CIRCUITPY_HW_SDRAM_TIMING_TRCD (2) | ||
|
|
||
| #define CIRCUITPY_HW_SDRAM_ROW_BITS_NUM 13 | ||
| #define CIRCUITPY_HW_SDRAM_MEM_BUS_WIDTH 32 | ||
| #define CIRCUITPY_HW_SDRAM_REFRESH_CYCLES 8192 | ||
|
|
||
| #define CIRCUITPY_HW_SDRAM_COLUMN_BITS_NUM 9 | ||
| #define CIRCUITPY_HW_SDRAM_INTERN_BANKS_NUM 4 | ||
| #define CIRCUITPY_HW_SDRAM_RPIPE_DELAY 0 | ||
| #define CIRCUITPY_HW_SDRAM_RBURST (1) | ||
| #define CIRCUITPY_HW_SDRAM_WRITE_PROTECTION (0) | ||
|
|
||
| #define CIRCUITPY_HW_SDRAM_AUTOREFRESH_NUM (8) |
There was a problem hiding this comment.
I don't think it makes sense to factor all of this out. Instead, board_init could call sdram_init and just pass in the structs with the config. I'd only add stuff here that is applicable across boards and ports.
There was a problem hiding this comment.
I refactored the SDRAM code and moved the data and sdram_init() call in board.c
Please have a look and let me know if this is a better approach :)
- Added the SDRAM controller and set up heap to use the 64 MB external RAM - Added the rest of the board pin definitions from the pinout - Did peripheral pins cleanup for the stm32H750 chip - Added SDMMC peripheral and sdioio module support - Added pwmio support and started work on audiopwmio
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.