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

ESP-IDF built-in driver doesn't work right #7

Closed
bbulkow opened this issue Jun 30, 2020 · 3 comments
Closed

ESP-IDF built-in driver doesn't work right #7

bbulkow opened this issue Jun 30, 2020 · 3 comments

Comments

@bbulkow
Copy link
Owner

bbulkow commented Jun 30, 2020

In release versions 4.1, 4.2, and master, the system driver is enabled by default, because the code doesn't crash.

However, it also doesn't work. The colors are terribly wrong, so it seems the conversion going into ESP-IDF must be bad, or the bit rate is bad, or something.

First test will be to see if the build-in interrupt handler at least is working under the later releases just so something works, then to go after the actual flaw causing the driver to not work.

@plebed
Copy link

plebed commented Jul 16, 2020

Maybe the problem is caused by espressif/esp-idf#5524

@bbulkow
Copy link
Owner Author

bbulkow commented Jul 20, 2020

Thanks for the pointer. This repo is now a fork of https://github.com/samguyer/FastLED , and thus all the code has changed, so needs to be retested. I don't think Sam's code was correct for the built in driver - I sent him a pull request.

As a reminder, the FastLED irq driver is standard, and seems to work very well.

I will keep an eye on 5524, thanks.

@bbulkow bbulkow changed the title ESP-IDF driver doesn't work right ESP-IDF built-in driver doesn't work right Jul 20, 2020
@bbulkow
Copy link
Owner Author

bbulkow commented Sep 15, 2020

Seems to work now.

@bbulkow bbulkow closed this as completed Sep 15, 2020
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

No branches or pull requests

2 participants