Skip to content

/esp32/boards: Add LOLIN_S3_PRO board definition. - #17290

Closed
mattytrentini wants to merge 1 commit into
micropython:masterfrom
mattytrentini:add-lolin-s3-pro
Closed

/esp32/boards: Add LOLIN_S3_PRO board definition.#17290
mattytrentini wants to merge 1 commit into
micropython:masterfrom
mattytrentini:add-lolin-s3-pro

Conversation

@mattytrentini

@mattytrentini mattytrentini commented May 12, 2025

Copy link
Copy Markdown
Contributor

Summary

Added a board definition for the Wemos Lolin S3 Pro.

Testing

I deployed the firmware to my S3 Pro, it appeared to work fine. Tested the filesystem (since a key feature of this board is the 16MB flash) and the GRB (not RGB) LED:

MicroPython v1.25.0-preview.575.g076e07197 on 2025-05-12; LOLIN_S3_PRO with ESP32S3
Type "help()" for more information.
>>>
paste mode; Ctrl-C to cancel, Ctrl-D to finish
=== import os
=== fs_stat = os.statvfs('/')
=== fs_size = fs_stat[0] * fs_stat[2]
=== fs_free = fs_stat[0] * fs_stat[3]
=== print(f"File System Size {fs_size:,} - Free Space {fs_free:,}")
===
File System Size 14,680,064 - Free Space 14,667,776
>>> from machine import Pin
>>> from neopixel import NeoPixel
>>> neo = NeoPixel(Pin.board.RGB_LED, 1)
>>> NeoPixel.ORDER = (0, 1, 2, 3)  # GRB rather than RGB
>>> neo[0] = (40, 0, 0)
>>> neo.write() # Red
>>> neo[0] = (0, 40, 0)
>>> neo.write() # Green
>>> neo[0] = (0, 0, 40)
>>> neo.write() # Blue

Trade-offs and Alternatives

A user could define their own custom board. We could also extend ESP32_GENERIC_S3 to support 16MB models (which may be wroth doing as well).

See #17217 for a request for this board.

@mattytrentini mattytrentini added the board-definition New or updated board definition files. Combine with a port- label. label May 12, 2025
@mattytrentini mattytrentini linked an issue May 12, 2025 that may be closed by this pull request
Request: Port for Lolin ESP32-S3 Pro (16MB flash, 8MB SPRAM) #17217
Open
@mattytrentini mattytrentini mentioned this pull request May 13, 2025
Open
@dpgeorge dpgeorge mentioned this pull request May 13, 2025
Merged
@dpgeorge

dpgeorge commented Jun 16, 2025

Copy link
Copy Markdown
Member

We could also extend ESP32_GENERIC_S3 to support 16MB models

This is now done, see commit 6201e77

Given that, does it still make sense to add this Lolin board definition? If so, please update this PR so it works with the new auto-sizing feature (basically, you don't need to specify the flash size or partitions csv).

@mattytrentini

mattytrentini commented Jun 16, 2025

Copy link
Copy Markdown
Contributor Author

Given that, does it still make sense to add this Lolin board definition?

It doesn't add much on top of ESP32_GENERIC_S3. Really the only benefit is that it's more obvious which firmware is appropriate for Lolin S3 Pro users. Maybe we need to list 'community tested' boards for the ESP32_GENERIC* boards?

In any case, I'm ok with closing this and directing Lolin S3 Pro users to the generic alternative.

@dpgeorge

dpgeorge commented Jun 19, 2025

Copy link
Copy Markdown
Member

Maybe we need to list 'community tested' boards for the ESP32_GENERIC* boards?

Yes, I think that's a good idea. Can easily just list them in the board.md for those generic boards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

board-definition New or updated board definition files. Combine with a port- label. port-esp32

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Request: Port for Lolin ESP32-S3 Pro (16MB flash, 8MB SPRAM)

2 participants

@mattytrentini @dpgeorge