fix(mcpwm): fix division by zero error in prescale

Kainarx · Kainarx · commit 0c95b8c267ed · 2025-09-30T12:21:33.000+08:00
Closes #17665
diff --git a/components/esp_driver_mcpwm/src/mcpwm_com.c b/components/esp_driver_mcpwm/src/mcpwm_com.c
@@ -248,12 +248,12 @@ esp_err_t mcpwm_set_prescale(mcpwm_group_t *group, uint32_t expect_module_resolu
         }
         module_prescale = fit_module_prescale;
         group_prescale = fit_group_prescale;
+        ESP_RETURN_ON_FALSE(group_prescale > 0 && group_prescale <= MCPWM_LL_MAX_GROUP_PRESCALE, ESP_ERR_INVALID_STATE, TAG,
+                            "set group prescale failed, group clock cannot match the resolution");
         group_resolution_hz = periph_src_clk_hz / group_prescale;
     }
 
     ESP_LOGD(TAG, "group (%d) calc prescale:%"PRIu32", module calc prescale:%"PRIu32"", group_id, group_prescale, module_prescale);
-    ESP_RETURN_ON_FALSE(group_prescale > 0 && group_prescale <= MCPWM_LL_MAX_GROUP_PRESCALE, ESP_ERR_INVALID_STATE, TAG,
-                        "set group prescale failed, group clock cannot match the resolution");
 
     // check if we need to update the group prescale, group prescale is shared by all mcpwm modules
     bool prescale_conflict = false;

Original file line number	Diff line number	Diff line change
`@@ -248,12 +248,12 @@ esp_err_t mcpwm_set_prescale(mcpwm_group_t *group, uint32_t expect_module_resolu`
`248`	`248`	`}`
`249`	`249`	`module_prescale = fit_module_prescale;`
`250`	`250`	`group_prescale = fit_group_prescale;`
	`251`	`+ ESP_RETURN_ON_FALSE(group_prescale > 0 && group_prescale <= MCPWM_LL_MAX_GROUP_PRESCALE, ESP_ERR_INVALID_STATE, TAG,`
	`252`	`+ "set group prescale failed, group clock cannot match the resolution");`
`251`	`253`	`group_resolution_hz = periph_src_clk_hz / group_prescale;`
`252`	`254`	`}`
`253`	`255`
`254`	`256`	`ESP_LOGD(TAG, "group (%d) calc prescale:%"PRIu32", module calc prescale:%"PRIu32"", group_id, group_prescale, module_prescale);`
`255`		`- ESP_RETURN_ON_FALSE(group_prescale > 0 && group_prescale <= MCPWM_LL_MAX_GROUP_PRESCALE, ESP_ERR_INVALID_STATE, TAG,`
`256`		`- "set group prescale failed, group clock cannot match the resolution");`
`257`	`257`
`258`	`258`	`// check if we need to update the group prescale, group prescale is shared by all mcpwm modules`
`259`	`259`	`bool prescale_conflict = false;`