[2.0.x] Implement SDIO support for STM32F1 #12782
Conversation
jmz52
changed the title
| dma_set_priority(SDIO_DMA_DEV, SDIO_DMA_CHANNEL, DMA_PRIORITY_VERY_HIGH); | ||
|
|
||
|
|
||
| if (!SDIO_CmdGoIdleState()) { return false; } |
There was a problem hiding this comment.
If the intention is to ignore the result of the first, doesn't the return false here defeat that? (sorry if I misunderstand what is supposed to happen here).
There was a problem hiding this comment.
CMD0 does not wait for SD card's response.
False here means that SDIO hardware failed to send command, which should never be the case when SDIO is properly configured and clocked.
thinkyhead
force-pushed
the
Implement-SDIO-support-for-STM32F1
branch
from
f18e5f4
to
739a652
Compare
|
This method is working for me to read data using SDIO on lerdge boards as well (in the STM32 HAL), so very useful. |
thinkyhead
force-pushed
the
bugfix-2.0.x
branch
2 times, most recently
from
a32ec92
to
a59d3d4
Compare
|
OMG! Thank you so much for this PR @jmz52! This PR made my GTM32Pro board able to read the reprap discount smart display's SD card. There's only one more hurdle for my board (eeprom). Thank you, thank you, thank you, thank you! |
Description
Add support for SDIO interface for STM32F1.
Benefits
Marlin can used onboard SD cards on MKS Robin and various STM32F1 development boards.
Related Issues
#11641
Additional Information
Two defines are needed to enable SDIO interface
#define SDSUPPORT#define SDIO_SUPPORTTo add SDIO support for other HALs three functions should be implemented in these HALs
bool SDIO_Init(void);bool SDIO_ReadBlock(uint32_t block, uint8_t *dst);bool SDIO_WriteBlock(uint32_t block, const uint8_t *src);