diff options
| author | Bryan O'Donoghue <bryan.odonoghue@linaro.org> | 2025-01-17 13:54:07 +0000 |
|---|---|---|
| committer | Bjorn Andersson <andersson@kernel.org> | 2025-02-07 23:20:55 -0600 |
| commit | 0e6dfde439df0bb977cddd3cf7fff150a084a9bf (patch) | |
| tree | ca0dae9fd67b641e54af42dec157400f98cb368a /drivers/clk/qcom | |
| parent | 52b10b591f83dc6d9a1d6c2dc89433470a787ecd (diff) | |
clk: qcom: gdsc: Release pm subdomains in reverse add order
gdsc_unregister() should release subdomains in the reverse order to the
order in which those subdomains were added.
I've made this patch a standalone patch because it facilitates a subsequent
fix to stable.
Fixes: 1b771839de05 ("clk: qcom: gdsc: enable optional power domain support")
Cc: stable@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Link: https://lore.kernel.org/r/20250117-b4-linux-next-24-11-18-clock-multiple-power-domains-v10-1-13f2bb656dad@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Diffstat (limited to 'drivers/clk/qcom')
| -rw-r--r-- | drivers/clk/qcom/gdsc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/qcom/gdsc.c b/drivers/clk/qcom/gdsc.c index fa5fe4c2a2ee..bc1b1e37bf42 100644 --- a/drivers/clk/qcom/gdsc.c +++ b/drivers/clk/qcom/gdsc.c @@ -571,7 +571,7 @@ void gdsc_unregister(struct gdsc_desc *desc) size_t num = desc->num; /* Remove subdomains */ - for (i = 0; i < num; i++) { + for (i = num - 1; i >= 0; i--) { if (!scs[i]) continue; if (scs[i]->parent) |