Skip to content

STEVAL-MKSBOX1V1 SDIO broken under STMduino STM32SD 1.3.0; works under 1.2.3 #60

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
gemelko opened this issue Jul 5, 2023 · 3 comments · Fixed by #61
Closed

STEVAL-MKSBOX1V1 SDIO broken under STMduino STM32SD 1.3.0; works under 1.2.3 #60

gemelko opened this issue Jul 5, 2023 · 3 comments · Fixed by #61
Labels
bug 🐛 Something isn't working
Milestone

Comments

@gemelko
Copy link

gemelko commented Jul 5, 2023

A new project that I created using the STM32duino STM32SD library would not access any SD card on the sensortile box (STEVAL-MKSBOX1V1). I also had a previous project which used to work however was now failing.

So then tried several of the included examples (listfiles, CardInfo, etc) and I found that none of them worked either. It does one of two things: hangs for a long time then fails on the SD.begin() call, or in the case of CardInfo, it sometimes passes the card.init() call but then fails the fatFs.init() call and reports that the card is not formatted. I tried this with several different SD cards, formatted to either FAT16 or FAT32: all had the same problem.

I realized that I had upgraded to library version 1.3.0 recently, and so I removed 1.3.0 and installed 1.2.3. Now all of the examples and my code work properly for all cards. I did not dig into the library or causes any further.

Glenn

@fpistm fpistm added the bug label Jul 5, 2023
@fpistm fpistm added this to the 1.3.1 milestone Jul 5, 2023
fpistm added a commit to fpistm/STM32SD that referenced this issue Jul 5, 2023
Wrong endif preprocessor placement when introducing new
SD pins management.

Fixes stm32duino#60

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm
Copy link
Member

fpistm commented Jul 5, 2023

Hi @gemelko
Thanks for the report.
I've made a fix in #61.

@gemelko
Copy link
Author

gemelko commented Jul 5, 2023

I can confirm that this works properly. Thanks!

@fpistm
Copy link
Member

fpistm commented Jul 5, 2023

I can confirm that this works properly. Thanks!

Anyway, I've found an other issue that I will fix for other series due to my mistakes 😩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants