[PATCH v2 4/9] soc: qcom: geni-se: Remove OPP rate reset from resource deactivation
From: Praveen Talari
Date: Thu Aug 27 2026 - 14:05:16 EST
Remove the dev_pm_opp_set_rate(se->dev, 0) call from
geni_se_resources_deactivate().
OPP resource management should be controlled by the consumer driver.
Additionally, clocks are already disabled by geni_se_clks_off(), making
the OPP rate reset unnecessary during resource deactivation.
Signed-off-by: Praveen Talari <praveen.talari@xxxxxxxxxxxxxxxx>
---
drivers/soc/qcom/qcom-geni-se.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c
index 04f58a6f26ff..d4f0544d73c4 100644
--- a/drivers/soc/qcom/qcom-geni-se.c
+++ b/drivers/soc/qcom/qcom-geni-se.c
@@ -1039,8 +1039,8 @@ EXPORT_SYMBOL_GPL(geni_icc_disable);
* geni_se_resources_deactivate() - Deactivate GENI SE device resources
* @se: Pointer to the geni_se structure
*
- * Deactivates device resources for power saving: OPP rate to 0, pin control
- * to sleep state, turns off clocks, and disables interconnect. Skips ACPI devices.
+ * Deactivates device resources for power saving: pin control to sleep
+ * state, turns off clocks, and disables interconnect. Skips ACPI devices.
*
* Return: 0 on success, negative error code on failure
*/
@@ -1051,9 +1051,6 @@ int geni_se_resources_deactivate(struct geni_se *se)
if (has_acpi_companion(se->dev))
return 0;
- if (se->has_opp)
- dev_pm_opp_set_rate(se->dev, 0);
-
ret = pinctrl_pm_select_sleep_state(se->dev);
if (ret)
return ret;
--
2.34.1