Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

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

wants to merge 1 commit into from

Conversation

mattytrentini
Copy link
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
Copy link
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
Copy link
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
Copy link
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