diff options
author | Rajendra Nayak <rnayak@codeaurora.org> | 2018-03-23 04:26:15 -0400 |
---|---|---|
committer | Stephen Boyd <sboyd@kernel.org> | 2018-06-01 14:14:52 -0400 |
commit | 31543ebb8aff5efb53464d65221b3a1f7951e99c (patch) | |
tree | bc94b36363847dab154fcae40d09b140f9e25229 | |
parent | fb55bea1fe436b2b00a8bcc0bad69c76409270e8 (diff) |
clk: qcom: Register the gdscs before the clocks
We have atleast some instances of ALWAYS_ON gdscs, which need to
be turned ON *before* some clocks within the gdsc domain marked
with a CLK_IS_CRITICAL can be turned ON.
To facilitate this sequence, register the GDCSs (and hence handle
the ALWAYS_ON gdscs) before we register clocks (and handle the
clocks marked as CLK_IS_CRITICAL)
Signed-off-by: Rajendra Nayak <rnayak@codeaurora.org>
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Tested-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
-rw-r--r-- | drivers/clk/qcom/common.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/drivers/clk/qcom/common.c b/drivers/clk/qcom/common.c index b8064a336d46..39ce64c2783b 100644 --- a/drivers/clk/qcom/common.c +++ b/drivers/clk/qcom/common.c | |||
@@ -228,22 +228,6 @@ int qcom_cc_really_probe(struct platform_device *pdev, | |||
228 | if (!cc) | 228 | if (!cc) |
229 | return -ENOMEM; | 229 | return -ENOMEM; |
230 | 230 | ||
231 | cc->rclks = rclks; | ||
232 | cc->num_rclks = num_clks; | ||
233 | |||
234 | for (i = 0; i < num_clks; i++) { | ||
235 | if (!rclks[i]) | ||
236 | continue; | ||
237 | |||
238 | ret = devm_clk_register_regmap(dev, rclks[i]); | ||
239 | if (ret) | ||
240 | return ret; | ||
241 | } | ||
242 | |||
243 | ret = devm_of_clk_add_hw_provider(dev, qcom_cc_clk_hw_get, cc); | ||
244 | if (ret) | ||
245 | return ret; | ||
246 | |||
247 | reset = &cc->reset; | 231 | reset = &cc->reset; |
248 | reset->rcdev.of_node = dev->of_node; | 232 | reset->rcdev.of_node = dev->of_node; |
249 | reset->rcdev.ops = &qcom_reset_ops; | 233 | reset->rcdev.ops = &qcom_reset_ops; |
@@ -272,6 +256,22 @@ int qcom_cc_really_probe(struct platform_device *pdev, | |||
272 | return ret; | 256 | return ret; |
273 | } | 257 | } |
274 | 258 | ||
259 | cc->rclks = rclks; | ||
260 | cc->num_rclks = num_clks; | ||
261 | |||
262 | for (i = 0; i < num_clks; i++) { | ||
263 | if (!rclks[i]) | ||
264 | continue; | ||
265 | |||
266 | ret = devm_clk_register_regmap(dev, rclks[i]); | ||
267 | if (ret) | ||
268 | return ret; | ||
269 | } | ||
270 | |||
271 | ret = devm_of_clk_add_hw_provider(dev, qcom_cc_clk_hw_get, cc); | ||
272 | if (ret) | ||
273 | return ret; | ||
274 | |||
275 | return 0; | 275 | return 0; |
276 | } | 276 | } |
277 | EXPORT_SYMBOL_GPL(qcom_cc_really_probe); | 277 | EXPORT_SYMBOL_GPL(qcom_cc_really_probe); |