Can't image stopWaveform is called but PWM shall not stop.

startWaveform already implied stopping PWM before.

Author: dok-net

cores/esp8266/Tone.cpp

@@ -30,11 +30,6 @@ static void _startTone(uint8_t _pin, uint32_t high, uint32_t low, uint32_t durat
     return;
   }
 
-  // Stop any analogWrites (PWM) because they are a different generator
-  _stopPWM(_pin);
-  // If there's another Tone or startWaveform on this pin
-  // it will be changed on-the-fly (no need to stop it)
-
   pinMode(_pin, OUTPUT);
 
   high = std::max(high, (uint32_t)microsecondsToClockCycles(25));  // new 20KHz maximum tone frequency,

cores/esp8266/core_esp8266_waveform_pwm.cpp

@@ -435,6 +435,9 @@ IRAM_ATTR int stopWaveform_weak(uint8_t pin) {
   if (!timerRunning) {
     return false;
   }
+
+  _stopPWM(pin); // Make sure there's no PWM live here
+
   // If user sends in a pin >16 but <32, this will always point to a 0 bit
   // If they send >=32, then the shift will result in 0 and it will also return false
   uint32_t mask = 1<<pin;

cores/esp8266/core_esp8266_wiring_digital.cpp

@@ -83,7 +83,6 @@ extern void __pinMode(uint8_t pin, uint8_t mode) {
 
 extern void IRAM_ATTR __digitalWrite(uint8_t pin, uint8_t val) {
   stopWaveform(pin); // Disable any Tone or startWaveform on this pin
-  _stopPWM(pin);     // and any analogWrites (PWM)
   if(pin < 16){
     if(val) GPOS = (1 << pin);
     else GPOC = (1 << pin);