Skip to content

Make ESP32-S2 4MB boards use no-ota partition table; clean up espressif mpconfigboard.mk #10468

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 3 commits into from
Jul 14, 2025

Conversation

dhalbert
Copy link
Collaborator

@dhalbert dhalbert commented Jul 9, 2025

This completes the multi-step task of converting 4MB Espressif boards to use a single larger firmware partition, instead of having one main partition and a second OTA-update partition of the same size. The increased size of the partition allows us to turn on BLE on 4MB boards, and allow other features.

  • Fixes correct board special pin setup #9553 (last task: do ESP32-S2 boards)
  • Clean up espressif/boards/*/mpconfigboard.mk.
    • remove CIRCUITPY_ESPCAMERA = 0, which no longer needs to be done explicitly on non-PSRAM boards: it is turned off automically on those boards.
    • turn on modules on a few boards now that there is more space.
    • add comments about why some modules are turned off, mostly due to not having enough pins available

dhalbert added 2 commits July 9, 2025 17:36
… add more modules now

- CIRCUITPY_ESPCAMERA = 0 is now automatic for boards without PSRAM, so remove explicit references.
- Restore some modules to boards now that there is more room.
- Add comments noting boards with few pins. More could be turned off on these boards, but it doesn't matter much.
@dhalbert dhalbert marked this pull request as draft July 9, 2025 21:53
@dhalbert dhalbert changed the title Make ESP32-S3 4MB boards use no-ota partition table; clean up espressif mpconfigboard.mk Make ESP32-S2 4MB boards use no-ota partition table; clean up espressif mpconfigboard.mk Jul 9, 2025
tannewt
tannewt previously approved these changes Jul 11, 2025
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Awesome thanks!

@dhalbert dhalbert marked this pull request as ready for review July 14, 2025 19:45
@dhalbert dhalbert requested review from tannewt and FoamyGuy July 14, 2025 20:06
Copy link
Collaborator

@FoamyGuy FoamyGuy 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!

@dhalbert dhalbert merged commit 710eaa7 into adafruit:main Jul 14, 2025
245 checks passed
@dhalbert dhalbert deleted the esp32-s2-no-ota branch July 14, 2025 21:30
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