ESP32 mcpwm_get_frequency: division by zero about arduino-foc HOT 3 CLOSED

Ah, nice!
So the mcpwm_init() now sets the pwm frequency?
This would remove the need for the fancy calculation of prescalers and the periods.

I'll look into it and if it works it will simplify the code quiet a bit!

from arduino-foc.

Hey @jkirsons

I've tested the mcpwm_init() frequency setting feature and the good news is that it works.
The bad news is that they used a fixed prescale and therefore lose a lot of precision for higher frequencies.
For example if you set 30kHz, mcpwm_init() will set 33kHz
If you set 40kHz, it will set 45kHz

This is not a huge deal but for now I'd leave our implementatio as is, regarding the frequency setting but hwever we'll add the line as you suggested it to avoid the zero division errors:
pwm_config.frequency = 2*pwm_frequency;

from arduino-foc.

Thanks! It may save others some time figuring out what is wrong.

mcpwm looks like it changes a bit in IDF 4 versions, so there may be further changes when new ESP32 Arduino versions are released.

from arduino-foc.