tone generation with ledcSetup and ledcWrite instable under fw 2.0 (was fine with 1.0.6)

[bongoo1]

Board

esp32dev

Device Description

piezzo beeper connected to gpio33

Hardware Configuration

nothing relevant

Version

v2.0.0

IDE Name

platformIO

Operating System

win10

Flash frequency

default

PSRAM enabled

no

Upload speed

115200

Description

i generate short tones when pressing a key on the keyboard connected using the commands ledcSetup and ledcWrite to do pwm on the pin, connected to the piezzo beeper.
this works fine under arduino framework 1.0.6.
after switching to framework 2, tone generation got instable.
only about every second time, i hear a beep, although the code that should beep is executed every time.
when going back to 1.0.6, this works without any issues.

Sketch

ledcSetup(1,BEEP[beepRptr].freq,8);
        ledcWrite(1,128);

Debug Message

no error message, just no output signal (beep)

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

• I confirm I have checked existing issues, online documentation and Troubleshooting guide.

[P-R-O-C-H-Y]

Hi @bongoo1,
Can you provide complete Sketch to reproduce the issue? Or at least the part with the keyboard.

[bongoo1]

here's the sketch as it actually looks like after i tried a few things more.

with platformio.ini containing
platform = espressif32
it builds with framework 1.0.6
then it works fine.

with platformio.ini containing

platform = https://github.com/platformio/platform-espressif32.git#feature/arduino-upstream
platform_packages =	platformio/framework-arduinoespressif32 @ https://github.com/espressif/arduino-esp32.git

instead, there is not beep audible about half of the time.

with the code below, call
beepShort750()
to add a beep to the beep sequence queue.
the
beepDo()
is then periodically called from within the loop to play the beep sequence from the queue.
this allows me to also configure a melody of a few seconds without blocking the execution of other code.

typedef struct beepType {
  int freq;    //0: off, 1: full, all others: frequency
  int length;  //length in ms
} beepType;

#define beepPin 33

#define beepSize 128
beepType BEEP[beepSize] = {0};
int beepWptr = 0;
int beepRptr = 0;

void setupBuzzer() {

    ledcSetup(1,1000,8); //channel 1, 1khz, 0..255
    ledcAttachPin(beepPin,1); //prepare pwm output, gpio33 from channel 1
    ledcWrite(1,0); 
}

void beepAdd(int freq, int length) {
//add beep to queue

  if ((beepWptr + 1) == beepRptr) {
    //beep queue full
  } else {
    BEEP[beepWptr].freq = freq;
    BEEP[beepWptr].length = length;
    beepWptr++;
    if (beepWptr == beepSize) {
      beepWptr = 0;
    }
  }
}

void beepDo() {
// beep if required
  
  static long unsigned int endBeep = 0;
  if (millis() > endBeep) { //beep phase ended
    if (beepRptr != beepWptr) { //beep command in queue
      if (BEEP[beepRptr].freq == 0) {
        ledcWrite(1,0); //turn off
        Serial.printf("B:off, ");
      } else if (BEEP[beepRptr].freq == 1) {
        ledcWriteTone(1,100);
        ledcWrite(1,254); //turn fully on (nominal freq of beeper)
        Serial.printf("B:max, ");
      } else {
        ledcWrite(1,128);
        ledcWriteTone(1,BEEP[beepRptr].freq);
        Serial.printf("B:%d, ",BEEP[beepRptr].freq);
      }
      endBeep = millis() + BEEP[beepRptr].length;
      Serial.printf("(%u/%u)\n",millis(),endBeep);
      beepRptr++;
      if (beepRptr == beepSize) {
        beepRptr = 0;
      }
    } else { //no new beep data, stop beeper
      ledcWrite(1,0); //turn off
    }
  }
}


void beepShort750() {

  beepAdd(750,100);
  beepAdd(0,0);
}

[cjcr]

I can confirm this issue. Is ok with 1.0.6.