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.

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

ESP32S3_DEVKITC LED Strip Sample - RGB Pin # Wrong in Overlay #81893

Open
burke-david opened this issue Nov 25, 2024 · 3 comments · May be fixed by #82002
Open

ESP32S3_DEVKITC LED Strip Sample - RGB Pin # Wrong in Overlay #81893

burke-david opened this issue Nov 25, 2024 · 3 comments · May be fixed by #82002
Assignees
@sylvioalves
Labels
bug The issue is a bug, or the PR is fixing a bug platform: ESP32 Espressif ESP32 priority: low Low impact/importance bug

Comments

@burke-david
Copy link

Describe the bug

In running the led_strip sample project on the Espressif ESP32S3_DEVKITC platform, I noticed that the defined pin for the RGB LED in the esp32s3_devkitc_procpu.overlay file should be GPIO38, not GPIO39.
So instead of

pinmux = <I2S0_O_SD_GPIO39>;

Line 42 should read

pinmux = <I2S0_O_SD_GPIO38>;

To Reproduce

Expected behavior

Impact

The example will not function properly without the pin being changed to GPIO38 on the esp32s3_devkitc boards.

Logs and console output

Environment (please complete the following information):

  • OS: (e.g. Linux, MacOS, Windows)
  • Toolchain (e.g Zephyr SDK, ...)
  • Commit SHA or Version used

Discovered using the latest commit as of yesterday b1def71

Additional context
@burke-david burke-david added the bug The issue is a bug, or the PR is fixing a bug label Nov 25, 2024
@github-actions GitHub Actions
Copy link

Hi @burke-david! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

@kartben
Copy link
Collaborator

kartben commented Nov 25, 2024

Thanks @burke-david, do you want to send a pull request to get this fixed?

@kartben kartben added platform: ESP32 Espressif ESP32 priority: low Low impact/importance bug labels Nov 25, 2024
@burke-david burke-david linked a pull request Nov 26, 2024 that will close this issue
Open
@burke-david
Copy link
Author

@kartben, yes, thank you. Should have done that in the first place. PR submitted referencing this issue.

@kartben kartben linked a pull request Nov 26, 2024 that will close this issue
samples:drivers:led:led_strip: esp32s3_devkitc uses GPIO38, not GPIO39 for RGB led. #82002
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sylvioalves sylvioalves

Labels
bug The issue is a bug, or the PR is fixing a bug platform: ESP32 Espressif ESP32 priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
3 participants
@kartben @sylvioalves @burke-david