@dpgeorge It seems that all your remarks are fixed.
I see failed checks, but I don't understand what is wrong.
I open Code formatters #7829

If you click at the 'details' link of the fails checks, you'll see what is wrong. For code formatting, the lines preceded with a + show how it should look like, and the line starting with - show how the code looks at t he moment.
At the commit messages the period at the end of the subject line is missing. So e.g. instead of:
esp32\machine_pwm.c: Add support for all PWM timers and channels
it shall be
esp32\machine_pwm.c: Add support for all PWM timers and channels.

@robert-hh Thank you

As for me, it is an horrible code formatting style:

        if (self->mode == LEDC_LOW_SPEED_MODE) {
            gpio_matrix_out(self->pin, LEDC_LS_SIG_OUT0_IDX + self->channel, false, true);
        } else {
            #if LEDC_SPEED_MODE_MAX > 1
            #if CONFIG_IDF_TARGET_ESP32
            gpio_matrix_out(self->pin, LEDC_HS_SIG_OUT0_IDX + self->channel, false, true);
            #else
            #error Add supported CONFIG_IDF_TARGET_ESP32_xxx
            #endif
            #endif
        }

I'd prefer like this:

        if (self->mode == LEDC_LOW_SPEED_MODE) {
            gpio_matrix_out(self->pin, LEDC_LS_SIG_OUT0_IDX + self->channel, false, true);
        } else {
            #if LEDC_SPEED_MODE_MAX > 1
                #if CONFIG_IDF_TARGET_ESP32
                    gpio_matrix_out(self->pin, LEDC_HS_SIG_OUT0_IDX + self->channel, false, true);
                #else
                    #error Add supported CONFIG_IDF_TARGET_ESP32_xxx
                #endif
            #endif
        }

:(

Nevertheless. Could someone check the performance of the PR on ESP32
GENERIC_S2 and GENERIC_C3 boards?
Thanks.

Tested GENERIC on TinyPICO, GENERIC_S2 on FeatherS2, and GENERIC_C3 on ESP32-C3-DevKitC. PWM works as expected.

Squashed and merged in 52636fa and 71111cf

Thank you very much for the work on this!