Update esp32_ledc_mcu.cpp to ensure ledc_timer is initialised to a known state (ledc_timer = false)

[rob-deutsch]

Need to set this uninitialised piece of memory, otherwise ledc_timer_config() might try to call ledc_timer_del() and that will fail.

[askuric]

Thanks for this!
This seems to be a API change actually going from the espressif 5.1 to 5.2
Introducing the deconfigure might actually break the code for people with v3.0.x of the arduino-esp32 which depends on espressif 5.1.4. So we will have to handle this with care 😄

ledc_timer_config_t definition

• in 5.1.4 - https://github.com/espressif/esp-idf/blob/a9424f8a356c1eb9f38cca506ff27cda6dc0eb89/components/driver/ledc/include/driver/ledc.h#L56
• in 5.2+ - https://github.com/espressif/esp-idf/blob/adf5319639b7e66e4f80868b14fee1acb73672c7/components/esp_driver_ledc/include/driver/ledc.h#L56

Could you just confirm which version of espressif and ardunio-esp32 are you using?

[rob-deutsch]

Oh good pickup. I'm actually using pioarduino because espressif/arduino-esp32 doesn't seem to support platformio any more.

Maybe the simplest way forward is to calloc/memset ledc_timer_config_t to 0 based on its size?