[PATCH v7 6/7] clk: hi6220: Clock driver support for Hisilicon hi6220 SoC
From: Brent Wang <hidden>
Date: 2015-05-23 00:45:36
Also in:
linux-clk, lkml
Hello Stephen, 2015-05-23 3:17 GMT+08:00 Stephen Boyd [off-list ref]:
On 05/22/15 11:57, Brent Wang wrote:quoted
Hello Stephen, 2015-05-23 2:41 GMT+08:00 Stephen Boyd [off-list ref]:quoted
On 05/22, Stephen Boyd wrote:quoted
On 05/22/15 11:30, Brent Wang wrote:quoted
Hello Stephen, 2015-05-22 13:20 GMT+08:00 Bintian [off-list ref]:quoted
quoted
Is pl011 the uart device? Does it have a node in DT somewhere? If it does, then we could put the assigned-parents properties in that node so that when the pl011 probes the uart1 clock has its parent set to clk_150m. See the "Assigned clock parents and rates" section of Documentation/devicetree/bindings/clock/clock-bindings.txt.I will verify this.Currently the "assigned-clock*" doesn't work for pl011 UART device node, maybe we will do some fix for its driver later or other modules.Why doesn't it work?Does this patch help?Yes, it works! I also tested adding "of_clk_set_defaults" to "pl011_probe()", and "assigned-clock*" also works. So you will submit another patch to fix this problem and I can revove the "clk_set_parent" from my patch, right ?How about you take it as part of your series? Or if you're planning on splitting out the clk patch to go through clk-tree I can apply it before applying the clock driver patch.
It's better you take it and I split my patch :)
quoted hunk ↗ jump to hunk
----8<----- From: Stephen Boyd <redacted> Subject: [PATCH] amba: Support clk parents and rates assigned in DT Add the call to of_clk_set_defaults() into the amba probe path so that devices on the amba bus can use the assigned rates and parents feature of the common clock framework. Signed-off-by: Stephen Boyd <redacted> --- drivers/amba/bus.c | 5 +++++ 1 file changed, 5 insertions(+)diff --git a/drivers/amba/bus.c b/drivers/amba/bus.c index f0099360039e..350ed93d4281 100644 --- a/drivers/amba/bus.c +++ b/drivers/amba/bus.c@@ -19,6 +19,7 @@ #include <linux/amba/bus.h> #include <linux/sizes.h> #include <linux/limits.h> +#include <linux/clk/clk-conf.h> #include <asm/irq.h>@@ -237,6 +238,10 @@ static int amba_probe(struct device *dev) int ret; do { + ret = of_clk_set_defaults(dev->of_node, false); + if (ret < 0) + break; + ret = dev_pm_domain_attach(dev, true); if (ret == -EPROBE_DEFER) break;
Tested-by: Bintian Wang <redacted> Thanks, Bintian
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project