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

bootloop fix for spotpear esp32c3 1.44 lcd board #9997

Merged
merged 1 commit into from
Jan 26, 2025

Conversation

SeanTheITGuy
Copy link

Resolved issue related by @gabriel-gagnon in the original pull request (#9201)

The Spotpear ESP32C3 1.44" LCD board would work with CircuitPython 9.2.1, but since 9.2.2 has gone into boot loop after flashing. Modifying the mpconfigboard.mk to force legacy flash layout returned functionality.

Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix. I am going to try to understand why this made the board work. It makes it match other boards that do work. But the "LEGACY" layout should not make a difference if you don't care about preserving CIRCUITPY. The partition table is replaced when the .bin is flashed.

@dhalbert dhalbert merged commit 65556b4 into adafruit:main Jan 26, 2025
15 checks passed
@SeanTheITGuy
Copy link
Author

Thank you for the fix. I am going to try to understand why this made the board work. It makes it match other boards that do work. But the "LEGACY" layout should not make a difference if you don't care about preserving CIRCUITPY. The partition table is replaced when the .bin is flashed.

Even more mysterious, but possibly a clue to resolving the "why", is why it worked as-was on 9.2.1 but not on 9.2.2+.

@dhalbert
Copy link
Collaborator

Even more mysterious, but possibly a clue to resolving the "why", is why it worked as-was on 9.2.1 but not on 9.2.2+.

Exactly. I hope I can reproduce this on a QT Py C3 or similar and then do a bisect.

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.

2 participants