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

RMT: Fix RMT TX interrupt level #8813

Merged
merged 1 commit into from
Oct 27, 2023
Merged

RMT: Fix RMT TX interrupt level #8813

merged 1 commit into from
Oct 27, 2023

Conversation

lucasssvaz
Copy link
Collaborator

@lucasssvaz lucasssvaz commented Oct 27, 2023 •
edited
Loading

Description of Change

This PR aims to fix RMT failing to start due to the uninitialized interrupt value. This would cause the interrupt to be initialized with garbage memory values:

E (18193) rmt: rmt_new_tx_channel(210): invalid interrupt priority:-2113922172

Tests scenarios

Tested with ESP32-S3-DevKitC-1 using the BlinkRGB example.

@lucasssvaz lucasssvaz added Type: Bug 🐛 All bugs Area: Peripherals API Relates to peripheral's APIs. labels Oct 27, 2023
@lucasssvaz lucasssvaz self-assigned this Oct 27, 2023
@P-R-O-C-H-Y P-R-O-C-H-Y added this to the 3.0.0 milestone Oct 27, 2023
P-R-O-C-H-Y
P-R-O-C-H-Y approved these changes Oct 27, 2023
View reviewed changes
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.

LGTM!

@P-R-O-C-H-Y P-R-O-C-H-Y added the Status: Pending Merge Pull Request is ready to be merged label Oct 27, 2023
@me-no-dev me-no-dev merged commit 6d9ebea into espressif:master Oct 27, 2023
49 checks passed
@QnBarb
Copy link

QnBarb commented Oct 27, 2023

Do you have a simple RMT example to pulse an infrared led and example for a receiver? I'm just trying to shoot infrared pulses between two mcu's. I've tried the nec_example by espressif, but no luck. I'm using arduino, given that, I have little experience at the moment.
Please point me to a good source if possible. Thank you in advance.

@SuGlider SuGlider mentioned this pull request Nov 7, 2023
Closed
1 task
@P-R-O-C-H-Y P-R-O-C-H-Y mentioned this pull request Nov 13, 2023
Closed
1 task
@lucasssvaz lucasssvaz deleted the bugfix/rmt_tx_intr branch March 19, 2024 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@P-R-O-C-H-Y P-R-O-C-H-Y P-R-O-C-H-Y approved these changes

Assignees

@lucasssvaz lucasssvaz

Labels
Area: Peripherals API Relates to peripheral's APIs. Status: Pending Merge Pull Request is ready to be merged Type: Bug 🐛 All bugs
Projects
None yet
Milestone
3.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@lucasssvaz @QnBarb @P-R-O-C-H-Y @me-no-dev