summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHeiko Stuebner <heiko@sntech.de>2017-03-01 22:00:42 +0100
committerStephen Boyd <sboyd@codeaurora.org>2017-03-07 05:54:50 -0800
commitf8ba2d68e54fbca340ad0fce97397291ba9637bc (patch)
tree843c766da16133f7de08910d8307a58a567ce609
parent9b1b23f03abdd25ffde8bbfe5824b89bc0448c28 (diff)
downloadlinux-stable-f8ba2d68e54fbca340ad0fce97397291ba9637bc.tar.gz
linux-stable-f8ba2d68e54fbca340ad0fce97397291ba9637bc.tar.bz2
linux-stable-f8ba2d68e54fbca340ad0fce97397291ba9637bc.zip
clk: rockchip: Make uartpll a child of the gpll on rk3036
The shared uart-pll is on boot a child of the apll that can get changed by cpu frequency scaling. So move it away to the more stable gpll to make sure the uart doesn't break on cpu frequency changes. This turned up during the 4.11 merge-window when commit 6a171b299379 ("serial: 8250_dw: Allow hardware flow control to be used") added general termios enablement making the uart on rk3036 change frequency and thus making it susceptible for the frequency scaling issue. Signed-off-by: Heiko Stuebner <heiko@sntech.de> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
-rw-r--r--drivers/clk/rockchip/clk-rk3036.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/clk/rockchip/clk-rk3036.c b/drivers/clk/rockchip/clk-rk3036.c
index dcde70f4c105..00d4150e33c3 100644
--- a/drivers/clk/rockchip/clk-rk3036.c
+++ b/drivers/clk/rockchip/clk-rk3036.c
@@ -450,6 +450,13 @@ static void __init rk3036_clk_init(struct device_node *np)
return;
}
+ /*
+ * Make uart_pll_clk a child of the gpll, as all other sources are
+ * not that usable / stable.
+ */
+ writel_relaxed(HIWORD_UPDATE(0x2, 0x3, 10),
+ reg_base + RK2928_CLKSEL_CON(13));
+
ctx = rockchip_clk_init(np, reg_base, CLK_NR_CLKS);
if (IS_ERR(ctx)) {
pr_err("%s: rockchip clk init failed\n", __func__);