lba.gitea/WLED
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Reduce CPU load during interrupt handler

Browse Source 
See https://github.com/esp8266/Arduino/pull/7057#issuecomment-591632232
This commit is contained in:
Jason2866 2020-02-27 09:52:48 +01:00 committed by GitHub
parent a776b8ac31
commit 9eb646085e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
wled00/src/dependencies/arduino/core_esp8266_waveform.cpp
View File

@ -224,8 +224,8 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {
endPin = 32 - __builtin_clz(waveformEnabled); endPin = 32 - __builtin_clz(waveformEnabled);
} }
if (waveformEnabled) {
bool done = false; bool done = false;
if (waveformEnabled) {
do { do {
nextEventCycles = microsecondsToClockCycles(MAXIRQUS); nextEventCycles = microsecondsToClockCycles(MAXIRQUS);
for (int i = startPin; i <= endPin; i++) { for (int i = startPin; i <= endPin; i++) {
@ -257,7 +257,13 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {
// Check for toggles // Check for toggles
int32_t cyclesToGo = wave->nextServiceCycle - now; int32_t cyclesToGo = wave->nextServiceCycle - now;
if (cyclesToGo < 0) { if (cyclesToGo < 0) {
cyclesToGo = -((-cyclesToGo) % (wave->nextTimeHighCycles + wave->nextTimeLowCycles)); // See #7057
// The following is a no-op unless we have overshot by an entire waveform cycle.
// As modulus is an expensive operation, this code is removed for now:
// cyclesToGo = -((-cyclesToGo) % (wave->nextTimeHighCycles + wave->nextTimeLowCycles));
//
// Alternative version with lower CPU impact:
// while (-cyclesToGo > wave->nextTimeHighCycles + wave->nextTimeLowCycles) { cyclesToGo += wave->nextTimeHighCycles + wave->nextTimeLowCycles)};
waveformState ^= mask; waveformState ^= mask;
if (waveformState & mask) { if (waveformState & mask) {
if (i == 16) { if (i == 16) {