fix(ledc): fix ledc_get_freq calculation err due to overflow

Closes #14882
espressif · Dec 26, 2024 · c262ef4 · c262ef4
1 parent 306a587
commit c262ef4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/components/driver/ledc/ledc.c b/components/driver/ledc/ledc.c
@@ -774,7 +774,7 @@ uint32_t ledc_get_freq(ledc_mode_t speed_mode, ledc_timer_t timer_num)
     ledc_hal_get_clock_divider(&(p_ledc_obj[speed_mode]->ledc_hal), timer_num, &clock_divider);
     ledc_hal_get_duty_resolution(&(p_ledc_obj[speed_mode]->ledc_hal), timer_num, &duty_resolution);
     ledc_hal_get_clk_cfg(&(p_ledc_obj[speed_mode]->ledc_hal), timer_num, &clk_cfg);
-    uint32_t precision = (0x1 << duty_resolution);
+    uint64_t precision = (0x1 << duty_resolution);
     uint32_t src_clk_freq = 0;
     esp_clk_tree_src_get_freq_hz((soc_module_clk_t)clk_cfg, LEDC_CLK_SRC_FREQ_PRECISION, &src_clk_freq);
     portEXIT_CRITICAL(&ledc_spinlock);

diff --git a/docs/en/api-reference/peripherals/ledc.rst b/docs/en/api-reference/peripherals/ledc.rst
@@ -291,14 +291,14 @@ The LEDC API provides several ways to change the PWM frequency "on the fly":
 More Control Over PWM
 """""""""""""""""""""
 
-There are several lower level timer-specific functions that can be used to change PWM settings:
+There are several individual timer-specific functions that can be used to change PWM output:
 
 * :cpp:func:`ledc_timer_set`
 * :cpp:func:`ledc_timer_rst`
 * :cpp:func:`ledc_timer_pause`
 * :cpp:func:`ledc_timer_resume`
 
-The first two functions are called "behind the scenes" by :cpp:func:`ledc_channel_config` to provide a startup of a timer after it is configured.
+The first two functions are called "behind the scenes" by :cpp:func:`ledc_timer_config` to provide a startup of a timer after it is configured.
 
 
 Use Interrupts

diff --git a/docs/zh_CN/api-reference/peripherals/ledc.rst b/docs/zh_CN/api-reference/peripherals/ledc.rst
@@ -291,14 +291,14 @@ LED PWM 控制器 API 有多种方式即时改变 PWM 频率：
 控制 PWM 的更多方式
 """""""""""""""""""""
 
-有一些较底层的定时器特定函数可用于更改 PWM 设置：
+有一些较独立的定时器特定函数可用于更改 PWM 输出：
 
 * :cpp:func:`ledc_timer_set`
 * :cpp:func:`ledc_timer_rst`
 * :cpp:func:`ledc_timer_pause`
 * :cpp:func:`ledc_timer_resume`
 
-前两个功能可通过函数 :cpp:func:`ledc_channel_config` 在后台运行，在定时器配置后启动。
+前两个功能可通过函数 :cpp:func:`ledc_timer_config` 在后台运行，在定时器配置后启动。
 
 
 使用中断