Skip to content

Backport fix ESP8266 DMA off-by-one to 0.15#5139

Merged
netmindz merged 1 commit intowled:0_15_xfrom
willmmiles:0_15_esp8266-dma-fix
Dec 1, 2025
Merged

Backport fix ESP8266 DMA off-by-one to 0.15#5139
netmindz merged 1 commit intowled:0_15_xfrom
willmmiles:0_15_esp8266-dma-fix

Conversation

@willmmiles
Copy link
Member

Shim in Makuna/NeoPixelBus#894 until approved by upstream.

Backport of #5138

Shim in Makuna/NeoPixelBus#894 until approved by
upstream.  Fixes wled#4906 and wled#5136.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 1, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@DedeHai DedeHai self-requested a review December 1, 2025 05:10
Copy link
Collaborator

@DedeHai DedeHai left a comment

Choose a reason for hiding this comment

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

confirmed: fixes the 52LED & output 3 issue

@netmindz netmindz merged commit 95849e9 into wled:0_15_x Dec 1, 2025
20 checks passed
willmmiles added a commit to willmmiles/WLED that referenced this pull request Dec 2, 2025
Includes bonus fix for ESP32 DMA driver, too!
Replaces wled#5139.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants