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

backport: variants from 3.0.0 + fixes #9381

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

SuGlider
Copy link
Collaborator

@SuGlider SuGlider commented Mar 16, 2024

Description of Change

Backports changes from 3.0.0 to variants and fixes the "unphone8" wrong board definition.
Only changes boards that were already in 2.0.14. No new board is added.

RGB_LED is now fixed for all boards with NEO LED.
The same for BUILTIN_LED. Both will work fine with digitalWrite() and neo_pixel_write()
Applied fixes done by the board makers in 3.0.0 based on PR #8726

Tests scenarios

Quick random testing with some boards.

Related links

Closes #9037

@SuGlider SuGlider added this to the 2.0.15 milestone Mar 16, 2024
@SuGlider SuGlider self-assigned this Mar 16, 2024
@SuGlider SuGlider linked an issue Mar 16, 2024 that may be closed by this pull request
1 task
@me-no-dev
Copy link
Member

overall looks OK, but hard to really find if there is something wrong :)

Copy link
Member

@P-R-O-C-H-Y P-R-O-C-H-Y left a comment

Choose a reason for hiding this comment

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

Looks good, lets run all boards test after merging this to catch if there is any issue :)

@P-R-O-C-H-Y
Copy link
Member

Also we may include board fixes from PR: Heltec ESP32 Series boards update #9367

Copy link
Collaborator

@lucasssvaz lucasssvaz left a comment

Choose a reason for hiding this comment

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

LGTM

@SuGlider
Copy link
Collaborator Author

Also we may include board fixes from PR: Heltec ESP32 Series boards update #9367

From #9367, just variants/heltec_wireless_stick_v3/pins_arduino.h can be backported.
Done now in f89d902

@me-no-dev me-no-dev merged commit 651a14c into espressif:release/v2.x Mar 18, 2024
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Wrong PIN for RGB LED on board lolin_s3_mini
4 participants