Skip to content
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

Update sdmmc support #378

Merged
merged 8 commits into from
Aug 28, 2022
Merged

Update sdmmc support #378

merged 8 commits into from
Aug 28, 2022

Conversation

richardeoin
Copy link
Member

Update sdio-host version and improve eMMC support

@richardeoin
Copy link
Member Author

@ostenning There's a few changes to the sdmmc support here, it would be helpful if you could try this out!

@ost-ing
Copy link
Contributor

ost-ing commented Aug 12, 2022

Hi @richardeoin , I've given this branch a fairly quick test with the embedded-sdmmc-rs crate, reading / writing data to the FAT32 formatted SDCard. I've had no problems as of yet.

Is there anything specifically you would like me to test?

@richardeoin
Copy link
Member Author

Thanks @ostenning 👍 I was looking for a sanity check really, so that test with embedded-sdmmc-rs is great

@burrbull
Copy link
Member

@richardeoin Do you plan to update f4xx-hal also?

@richardeoin
Copy link
Member Author

@burrbull No I don't plan to update f4xx-hal. The F4 reference manuals I found don't mention eMMC at all (although it might still be possible..) and I don't have any f4 + eMMC hardware to test on

@richardeoin
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Aug 28, 2022
377: Added reset reason impl r=richardeoin a=diondokter



378: Update sdmmc support r=richardeoin a=richardeoin

Update sdio-host version and improve eMMC support

Co-authored-by: Dion Dokter <diondokter@gmail.com>
Co-authored-by: Richard Meadows <962920+richardeoin@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Aug 28, 2022

This PR was included in a batch that was canceled, it will be automatically retried

@bors bors bot merged commit 3aa95a7 into stm32-rs:master Aug 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants