fix(esp_hw_support): fix esp32p4 CPU frequency switching timing

This commit is contained in:
wuzhenghui
2024-09-25 20:55:39 +08:00
parent 5b1796b284
commit a82938bb7e

View File

@ -205,15 +205,18 @@ static void rtc_clk_cpu_freq_to_8m(void)
*/ */
static void rtc_clk_cpu_freq_to_cpll_mhz(int cpu_freq_mhz, hal_utils_clk_div_t *div) static void rtc_clk_cpu_freq_to_cpll_mhz(int cpu_freq_mhz, hal_utils_clk_div_t *div)
{ {
// CPLL -> CPU_CLK -> MEM_CLK -> SYS_CLK -> APB_CLK /**
// Constraint: MEM_CLK <= 200MHz, APB_CLK <= 100MHz * Constraint: MEM_CLK <= 200MHz, APB_CLK <= 100MHz
// This implies that when clock source is CPLL, * This implies that when clock source is CPLL,
// If cpu_divider < 2, mem_divider must be larger or equal to 2 * If cpu_divider < 2, mem_divider must be larger or equal to 2
// If cpu_divider < 2, mem_divider = 2, sys_divider < 2, apb_divider must be larger or equal to 2 * If cpu_divider < 2, mem_divider = 2, sys_divider < 2, apb_divider must be larger or equal to 2
// Current available configurations: *
// 360 - 360 - 180 - 180 - 90 * Current available configurations:
// 360 - 180 - 180 - 180 - 90 * CPLL -> CPU_CLK -> MEM_CLK -> SYS_CLK -> APB_CLK
// 360 - 90 - 90 - 90 - 90 * 360 div1 360 div2 180 div1 180 div2 90
* 360 div2 180 div1 180 div1 180 div2 90
* 360 div4 90 div1 90 div1 90 div1 90
*/
uint32_t mem_divider = 1; uint32_t mem_divider = 1;
uint32_t sys_divider = 1; // We are not going to change this uint32_t sys_divider = 1; // We are not going to change this
uint32_t apb_divider = 1; uint32_t apb_divider = 1;
@ -237,16 +240,38 @@ static void rtc_clk_cpu_freq_to_cpll_mhz(int cpu_freq_mhz, hal_utils_clk_div_t *
// To avoid such case, we will strictly do abort here. // To avoid such case, we will strictly do abort here.
abort(); abort();
} }
// Update bit does not control CPU clock sel mux. Therefore, there may be a middle state during the switch (CPU rises)
// Since this is upscaling, we need to configure the frequency division coefficient before switching the clock source. // If it's upscaling, the divider of MEM/SYS/APB needs to be increased, to avoid illegal intermediate states,
// Otherwise, an intermediate state will occur, in the intermediate state, the frequency of APB/MEM does not meet the // the clock divider should be updated in the order from the APB_CLK to CPU_CLK.
// timing requirements. If there are periperals/CPU access that depend on these two clocks at this moment, some exception // And if it's downscaling, the divider of MEM/SYS/APB needs to be decreased, the clock divider should be updated
// might occur. // in the order from the CPU_CLK to APB_CLK.
clk_ll_cpu_set_divider(div->integer, div->numerator, div->denominator); // Otherwise, an intermediate state will occur, in the intermediate state, the frequency of APB/MEM does not meet
clk_ll_mem_set_divider(mem_divider); // the timing requirements. If there are periperals/CPU access that depend on these two clocks at this moment, some
clk_ll_sys_set_divider(sys_divider); // exception might occur.
clk_ll_apb_set_divider(apb_divider); if (cpu_freq_mhz >= esp_rom_get_cpu_ticks_per_us()) {
clk_ll_bus_update(); // Frequency Upscaling
clk_ll_apb_set_divider(apb_divider);
clk_ll_bus_update();
clk_ll_sys_set_divider(sys_divider);
clk_ll_bus_update();
clk_ll_mem_set_divider(mem_divider);
clk_ll_bus_update();
clk_ll_cpu_set_divider(div->integer, div->numerator, div->denominator);
clk_ll_bus_update();
} else {
// Frequency Downscaling
clk_ll_cpu_set_divider(div->integer, div->numerator, div->denominator);
clk_ll_bus_update();
clk_ll_mem_set_divider(mem_divider);
clk_ll_bus_update();
clk_ll_sys_set_divider(sys_divider);
clk_ll_bus_update();
clk_ll_apb_set_divider(apb_divider);
clk_ll_bus_update();
}
// Update bit does not control CPU clock sel mux, the clock source needs to be switched at
// last to avoid intermediate states.
clk_ll_cpu_set_src(SOC_CPU_CLK_SRC_PLL); clk_ll_cpu_set_src(SOC_CPU_CLK_SRC_PLL);
esp_rom_set_cpu_ticks_per_us(cpu_freq_mhz); esp_rom_set_cpu_ticks_per_us(cpu_freq_mhz);
} }