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

IDF release/v5.1 #8893

Closed
wants to merge 2 commits into from
Closed

IDF release/v5.1 #8893

wants to merge 2 commits into from

Conversation

me-no-dev
Copy link
Member

esp-idf: release/v5.1 5c61c89308
tinyusb: master 0f187b4d1

@Magnatto
Copy link

Magnatto commented Nov 22, 2023

Hello, on arduino-esp32/cores/esp32/esp32-hal-rmt.c on lines 516 and 541:
rmt_rx_channel_config_t type no longer contains intr_priority, removing them solved for me when using the function rmtInit.
image
This was the error:
image

@me-no-dev
Copy link
Member Author

@Magnatto are you using Arduino as component? Because you must have not updated your IDF to the latest release/v5.1 branch. The flag exists now and without it being set there, it would cause RMT not to start

@Magnatto
Copy link

Magnatto commented Nov 22, 2023

@Magnatto are you using Arduino as component? Because you must have not updated your IDF to the latest release/v5.1 branch. The flag exists now and without it being set there, it would cause RMT not to start

Pardon me @me-no-dev , you are correct, indeed i'm using arduino as component, however i'm using esp-idf v5.1.1, it didn't have the interrupt priority but 5.1.2 now has it.

@me-no-dev me-no-dev closed this Nov 23, 2023
@me-no-dev me-no-dev deleted the idf-release/v5.1 branch November 23, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants