8000 waveshare_esp32_s3_geek: don't init the wrong board SPI for the display by Neradoc · Pull Request #10162 · adafruit/circuitpython · GitHub
[go: up one dir, main page]

Skip to content

waveshare_esp32_s3_geek: don't init the wrong board SPI for the display #10162

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
wants to merge 19 commits into from

Conversation

Neradoc
Copy link
@Neradoc Neradoc commented Mar 22, 2025

Adds board.SPI for the display SPI bus, and use it in board.c.

Previously the code accidentally instantiated both the SD bus (board.SD_SPI) and overwrote it with the display bus in display_init(), making the SD pins unusable. I can't test it.
Should fix #10161

@RetiredWizard
Copy link

I tested the artifact from this update which does allow the SD card to be mounted now.

@Neradoc Neradoc changed the base branch from main to 9.2.x March 22, 2025 13:30
@Neradoc Neradoc force-pushed the fix-waveshare_esp32_s3_geek branch from c5701db to aaa3637 Compare March 22, 2025 13:30
@Neradoc Neradoc force-pushed the fix-waveshare_esp32_s3_geek branch from aaa3637 to 0f31506 Compare March 22, 2025 13:31
@Neradoc
Copy link
Author
Neradoc commented Mar 22, 2025

Hmmm juggled with git a little bit to rebase on 9.2.x, hope this doesn't break anything...

@Neradoc Neradoc closed this Mar 22, 2025
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.

Problems with accessing SD CARD on ESP32-S3-GEEK board
7 participants
0