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

CSE7766 does not work on latest esp8266 core #3991

Closed
TD-er opened this issue Apr 2, 2022 · 4 comments · Fixed by #4501
Closed

CSE7766 does not work on latest esp8266 core #3991

TD-er opened this issue Apr 2, 2022 · 4 comments · Fixed by #4501
Labels
Category: Plugin Related to supported sensors Type: Bug Considered a bug

Comments

@TD-er
Copy link
Member

TD-er commented Apr 2, 2022

The code for this plugin has hardly changed the last year, but now on the latest core build, it no longer seems to work.

This plugin does some strange things with the Serial0 port, so that's probably what's wrong here.

@TD-er TD-er added Type: Bug Considered a bug Category: Plugin Related to supported sensors labels Apr 2, 2022
@TD-er
Copy link
Member Author

TD-er commented Apr 25, 2022

See also this forum thread.
These changes are also required to make this plugin work on ESP32.

@tonhuisman
Copy link
Contributor

This is most likely resolved by #4328

@TD-er
Copy link
Member Author

TD-er commented Nov 9, 2022

Not sure, as it needs some different fixes for ESP32.
More likely this should be combined with this one:
#4293

Related newer forum topic: https://www.letscontrolit.com/forum/viewtopic.php?t=9337

@tonhuisman
Copy link
Contributor

Removed the strange things with the Serial0 port part from the plugin in #4501 that might solve this issue. Tested that PR with hard_Sonoff_POW ESP8266 build, and works as intended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Plugin Related to supported sensors Type: Bug Considered a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants