Skip to content

Metro RP2350 CP 10.x /sd #3061

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

Merged
merged 1 commit into from
Jun 13, 2025
Merged

Metro RP2350 CP 10.x /sd #3061

merged 1 commit into from
Jun 13, 2025

Conversation

mikeysklar
Copy link
Collaborator

@mikeysklar mikeysklar commented Jun 13, 2025

Demo code for CircuitPython 10.x SD card list and write. The 9.x examples are not compatible.

Addresses CircuitPython Issue # 10419

Addresses Forum Issue: SD Card on Metro RP2350 Setup

CircuitPython 10.x introduces automounting of /sd. The Metro RP2350 no detects and automounts a /sd card using SDIO. No initialization is required in the example code. Example code for 9.x and 10.x will need to be in place for some time to prevent mass hysteria.

Tested on both 10.x read / write scripts with:

+------------------------+-------------------------+
| Component              | Details                 |
+------------------------+-------------------------+
| Board                  | Metro RP2350            |
| microSD Card           | SanDisk Ultra A1 16 GB  |
| CircuitPython Version  | 9.2.8                    |
| CircuitPython Version  | 10.0.0-alpha6            |
+------------------------+-------------------------+

Demo code for CircuitPython 10.x SD card list and write. The 9.x examples are not compatible.
@mikeysklar mikeysklar merged commit 736bedf into main Jun 13, 2025
8 checks passed
@FoamyGuy
Copy link
Collaborator

I think it would be good to use try/except so that we can support both 9.2.x and 10.x with the same code for now. I submitted #3062 just now with try/except before I saw that this PR was already submitted

@mikeysklar
Copy link
Collaborator Author

Yeah, I was thinking about doing a single file update with logic to support both as well. My only reluctance was the aesthetic. The new automount code is so clean and simple to drop into projects while the legacy pin setup and additional try / except lose that elegance.

Anyway...Your call. I didn't mean to stomp on your PR.

@FoamyGuy
Copy link
Collaborator

No worries, I appreciate you finding, reporting, and fixing it :)

@mikeysklar
Copy link
Collaborator Author

Your PR looks good. I will delete this branch.

@mikeysklar mikeysklar deleted the metro-rp2350-CP10x-SD branch June 13, 2025 17:03
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