-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
BW-LT20 blinks when turned on #8046
Comments
Please, show us what the console says when this happens. Thanks |
00:00:00 CFG: Loaded from flash at F8, Count 56 |
What is URL for 8.1.0? |
Tasmota 8.1.0 is on github release page: https://github.com/arendst/Tasmota/releases/tag/v8.1.0 But please, before that, try latest Tasmota from development branch: http://thehackbox.org/tasmota/tasmota.bin |
I want to go back to the firmware that worked, but when i try i get a Upload buffer miscompare. How can i downgrade Tasmota? |
If you are not flashing directly using serial you have to install a minimal version first. |
Ok, i did the development Program Version | 8.2.0.3(c066b0c-tasmota) Exactly the same issue. I think it blinks when connecting to wifi Flashed another one with 8.1.0, that is working now (does not blink). So i have 2 with different versions, so i can do some testing... |
I put the weblog to 4. And here is from 8.1.0 |
Please try provided firmware. It is build with active GPIO blink fix. |
+1 v8.1.0 (sdk 2.6.3) = Normal but small dip in brightness after about 3sec. |
I have LSC bulbs. Will dig em out the trash can and have a look. |
Haha trash can? You threw them away because of the noise they make? |
I had no real use for them as the wife didn't liked them. Just tested and LSC E14 bulb which runs fine but as it uses an SM2135 PWM controller this is expected to run OK. The LSC E27 filament with two PWM channels indeed goes ON-OFF-ON after four seconds when a restart is performed. I'll check what a core 2.4.2 does... |
And as expected, core 2.4.2 runs just fine. So we need to investigate the massive changes to PWM between core versions. Nice challenge. |
But Arduino Core 2.6.3 and Tasmota v8.1.0 also works fine. So it's not only core related? I will try the build of @Jason2866 this evening. See if that solves the issue. |
With both Core 2.6.1 and 2.6.3 and Tasmota 8.1.0 I see some dimming after four seconds. Not a complete Off but still there is something going on. |
True, but wasn't that because of WiFi interupts and software PWM? Or do you mean that 2.4.2 does not even have that dimming issue? |
@arendst I see this dimming too with MagicHome Led driver. The dimming is caused by 'something' interfering with the PWM interrupt handler and changing slightly timings. The PWM settings are unchanged during this period. I guess it's caused by Wifi connection, but that's only a guess. |
@TimelessNL Can you self compile a version, and uncomment in
Then enable |
Just compiled 8.2.0.3 with core feature_stage AND disabled the file |
Wow, that's strange. Theo, would you be kind enough and uncomment line 277
|
As requested by enabling lines 2027-2028 and enabling the local waveform:
will now try your latest request. |
Thanks, nothing in the logs so the issue is definitely in the PWM interrupt handler. |
Did the change as you requested. Got compile error. Solved the code as follows:
but it still does an ON-OFF-ON after a restart (in addition to the planned OFF-ON as a result of the reset off PCM's at start of restart but that has nothing to do with the ON-OFF-ON issue). |
I'll compare our/your waveform.cpp with the core one... |
So the only active difference is below. The current core one (with slight dim):
The current tasmota with ON-OFF-ON:
Was there any reason to get rid of variable now in wave->nextServiceCycles? |
In fact #7851 seems to be culprit |
While experimenting with the core code I noticed it also does ON-OFF-ON with low dimmer values below 10% so... |
I created #7851 to remove ongoing flickering due to phase changes in PWM. I'm don't understand why it would create this On-Off-On effect. Is it happening during wifi connection? |
Yes, around that time. |
I was able to reproduce on MagicHome but the effect depends a lot on the color levels. For some levels it's only a slight change, for others it almost turns the light off. My test is now: I tried with current code, and two other variants from Arduino Core, but they all have the same visual effect I will get my scope in the following days and try to understand what is happening. Edit: I can also reproduce with a single PWM on a Wemos D1 mini. I'm wondering whether Wifi fast connect made it worse. |
I think I understand what's happening. Current core: the PWM interrupt handler tries to keep each PWM level constant, and sacrifices phase between channels, which leads to slight flickering in the long run. Current Tasmota with patch esp8266/Arduino#7057: interrupt handlers tries to keep phase between channels and sacrifies PWM levels. Under normal circumstances it works very well, but under high interrupt pressure, as it happens at wifi connection, it gives bad results. I will try to detect if the interrupt pressure is too high and revert to sacrificing phases during those periods. Stay tuned. |
This should fix it. On my MagicHome the flickering during wifi connection is minimal (slight change in colors). |
Fix PWM flickering during wifi connection (#8046)
@arendst @s-hadinger I am working on PR esp8266/Arduino#7022 , and I am sure I have made great progress recently. My own testing can only go as far as checking the audio, watching the built-in LED, listening to servos, and taking in-detail measurements by oscilloscope. |
@TimelessNL Changing the Arduino PWM implementation in #8246 and setting |
PROBLEM DESCRIPTION
I just upgraded from 8.1 to 8.2 on all my devices, approx 20, but on my BW-LT20,When turned on by switch, it turns off after 2-3 seconds, and turns back on again, annoying
REQUESTED INFORMATION
Make sure your have performed every step and checked the applicable boxes before submitting your issue. Thank you!
Backlog Template; Module; GPIO 255
:Backlog Rule1; Rule2; Rule3
:Status 0
:(Please use
weblog 4
for more debug information)TO REPRODUCE
Steps to reproduce the behavior:
EXPECTED BEHAVIOUR
A clear and concise description of what you expected to happen.
SCREENSHOTS
If applicable, add screenshots to help explain your problem.
ADDITIONAL CONTEXT
Add any other context about the problem here.
(Please, remember to close the issue when the problem has been addressed)
The text was updated successfully, but these errors were encountered: