Commit 395717ee authored by Abhishek Sahu's avatar Abhishek Sahu Committed by Stephen Boyd

clk: qcom: ipq4019: Add the cpu clock frequency change notifier

The current driver code gives the crash or gets hang while switching
the CPU frequency some time. The APSS CPU Clock divider is not glitch
free so it the APPS clock need to be switched for stable clock during
the change.

This patch adds the frequency change notifier for APSS CPU clock. It
changes the parent of this clock to stable PLL FEPLL500 for
PRE_RATE_CHANGE event. This event will be generated before actual
clock set operations. The clock set operation will again change its
corresponding parent by getting the same from frequency table.
Signed-off-by: default avatarAbhishek Sahu <absahu@codeaurora.org>
[sboyd@codeaurora.org: Indent less in probe]
Signed-off-by: default avatarStephen Boyd <sboyd@codeaurora.org>
parent 86c654d4
...@@ -22,6 +22,7 @@ ...@@ -22,6 +22,7 @@
#include <linux/reset-controller.h> #include <linux/reset-controller.h>
#include <linux/math64.h> #include <linux/math64.h>
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/clk.h>
#include <dt-bindings/clock/qcom,gcc-ipq4019.h> #include <dt-bindings/clock/qcom,gcc-ipq4019.h>
...@@ -165,6 +166,12 @@ static struct parent_map gcc_xo_ddr_500_200_map[] = { ...@@ -165,6 +166,12 @@ static struct parent_map gcc_xo_ddr_500_200_map[] = {
{ P_DDRPLLAPSS, 1 }, { P_DDRPLLAPSS, 1 },
}; };
/*
* Contains index for safe clock during APSS freq change.
* fepll500 is being used as safe clock so initialize it
* with its index in parents list gcc_xo_ddr_500_200.
*/
static const int gcc_ipq4019_cpu_safe_parent = 2;
static const char * const gcc_xo_ddr_500_200[] = { static const char * const gcc_xo_ddr_500_200[] = {
"xo", "xo",
"fepll200", "fepll200",
...@@ -1735,13 +1742,44 @@ static const struct of_device_id gcc_ipq4019_match_table[] = { ...@@ -1735,13 +1742,44 @@ static const struct of_device_id gcc_ipq4019_match_table[] = {
}; };
MODULE_DEVICE_TABLE(of, gcc_ipq4019_match_table); MODULE_DEVICE_TABLE(of, gcc_ipq4019_match_table);
static int
gcc_ipq4019_cpu_clk_notifier_fn(struct notifier_block *nb,
unsigned long action, void *data)
{
int err = 0;
if (action == PRE_RATE_CHANGE)
err = clk_rcg2_ops.set_parent(&apps_clk_src.clkr.hw,
gcc_ipq4019_cpu_safe_parent);
return notifier_from_errno(err);
}
static struct notifier_block gcc_ipq4019_cpu_clk_notifier = {
.notifier_call = gcc_ipq4019_cpu_clk_notifier_fn,
};
static int gcc_ipq4019_probe(struct platform_device *pdev) static int gcc_ipq4019_probe(struct platform_device *pdev)
{ {
return qcom_cc_probe(pdev, &gcc_ipq4019_desc); int err;
err = qcom_cc_probe(pdev, &gcc_ipq4019_desc);
if (err)
return err;
return clk_notifier_register(apps_clk_src.clkr.hw.clk,
&gcc_ipq4019_cpu_clk_notifier);
}
static int gcc_ipq4019_remove(struct platform_device *pdev)
{
return clk_notifier_unregister(apps_clk_src.clkr.hw.clk,
&gcc_ipq4019_cpu_clk_notifier);
} }
static struct platform_driver gcc_ipq4019_driver = { static struct platform_driver gcc_ipq4019_driver = {
.probe = gcc_ipq4019_probe, .probe = gcc_ipq4019_probe,
.remove = gcc_ipq4019_remove,
.driver = { .driver = {
.name = "qcom,gcc-ipq4019", .name = "qcom,gcc-ipq4019",
.of_match_table = gcc_ipq4019_match_table, .of_match_table = gcc_ipq4019_match_table,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment