Off-by-one in 100% duty overshoot correction.

dok-net · dok-net · commit 4b7964f22787 · 2020-05-08T19:18:50.000+02:00
diff --git a/cores/esp8266/core_esp8266_waveform.cpp b/cores/esp8266/core_esp8266_waveform.cpp
@@ -341,7 +341,7 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {
             const bool endOfPeriod = wave.nextPeriodCcy == wave.endDutyCcy;
             // active configuration and forward 100% duty
             if (!idleCcys) {
-              wave.nextPeriodCcy += fwdPeriodCcys + wave.periodCcys;
+              wave.nextPeriodCcy += fwdPeriodCcys;
               wave.endDutyCcy = wave.nextPeriodCcy;
               nextEdgeCcy = wave.nextPeriodCcy;
             }

Original file line number	Diff line number	Diff line change
`@@ -341,7 +341,7 @@ static ICACHE_RAM_ATTR void timer1Interrupt() {`
`341`	`341`	`const bool endOfPeriod = wave.nextPeriodCcy == wave.endDutyCcy;`
`342`	`342`	`// active configuration and forward 100% duty`
`343`	`343`	`if (!idleCcys) {`
`344`		`- wave.nextPeriodCcy += fwdPeriodCcys + wave.periodCcys;`
	`344`	`+ wave.nextPeriodCcy += fwdPeriodCcys;`
`345`	`345`	`wave.endDutyCcy = wave.nextPeriodCcy;`
`346`	`346`	`nextEdgeCcy = wave.nextPeriodCcy;`
`347`	`347`	`}`