Core3 updates - #925
Open
sblantipodi wants to merge 20 commits into
Open
Conversation
# Conflicts: # library.json # library.properties
- bit-bang CSR/GPIO - channel constructor overload
- bit-bang CSR/GPIO - channel constructor overload
- bit-bang CSR/GPIO - channel constructor overload
- bit-bang CSR/GPIO - channel constructor overload
Replace deprecated `gpio_hal_iomux_func_sel()` with `gpio_iomux_out()` for ESP-IDF 5.5+. - ESP-IDF 5.5.0 removed `gpio_hal_iomux_func_sel()`. - Use `gpio_iomux_out()` for ESP-IDF 5.5+. - Maintain backward compatibility with `#ifdef` for older versions. - Fixes compilation error with pioarduino platform (ESP-IDF 5.5.1). Tested with: - pioarduino platform (ESP-IDF 5.5.1, Arduino Core 3.3.4). - ESP32-S3 boards (Lolin S3 Mini, Seeed XIAO ESP32S3). Resolves: Makuna#895
config.mem_block_symbols = 192 in NeoEsp32RmtXMethod.h gives the RMT ping-pong half-buffer only ~96 symbols (~120us at WS2812x 800Kbps). NimBLE and WiFi event handlers run at higher interrupt priority than the RMT refill ISR and routinely preempt for longer, leaving the data line idle past the ~300us LED latch threshold and producing ghost pixels at 8-pixel intervals. On chips with DMA-capable RMT (ESP32-S3, ESP32-P4), enable flags.with_dma and request a generous DMA buffer; DMA decouples the buffer from RMT channel memory and from ISR refill timing entirely. On chips without DMA support (ESP32, S2, C3, C6, H2, C5), request the maximum mem_block_symbols a single TX channel can borrow from its group (SOC_RMT_TX_CANDIDATES_PER_GROUP * SOC_RMT_MEM_WORDS_PER_CHANNEL), giving the ISR the most slack each chip can provide. Also removes NEOPIXELBUS_RMT_INT_FLAGS, a leftover macro from the legacy IDF v3/v4 driver that is unused by this IDF v5 channel API. Fixes Makuna#921 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contained in this PR:
This PR includes cherry‑picked commits from: