提交 00a461cc 编写于 作者: J Johan Hovold 提交者: Stephen Boyd

clk: ti: fix OF child-node lookup

Fix child-node lookup which by using the wrong OF helper was searching
the whole tree depth-first, something which could end up matching an
unrelated node.

Also fix the related node-reference leaks.

Fixes: 5b385a45 ("clk: ti: add support for clkctrl aliases")
Signed-off-by: NJohan Hovold <johan@kernel.org>
Acked-by: NTero Kristo <t-kristo@ti.com>
Signed-off-by: NStephen Boyd <sboyd@kernel.org>
上级 5b394b2d
...@@ -129,7 +129,7 @@ int ti_clk_setup_ll_ops(struct ti_clk_ll_ops *ops) ...@@ -129,7 +129,7 @@ int ti_clk_setup_ll_ops(struct ti_clk_ll_ops *ops)
void __init ti_dt_clocks_register(struct ti_dt_clk oclks[]) void __init ti_dt_clocks_register(struct ti_dt_clk oclks[])
{ {
struct ti_dt_clk *c; struct ti_dt_clk *c;
struct device_node *node; struct device_node *node, *parent;
struct clk *clk; struct clk *clk;
struct of_phandle_args clkspec; struct of_phandle_args clkspec;
char buf[64]; char buf[64];
...@@ -164,8 +164,12 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[]) ...@@ -164,8 +164,12 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[])
continue; continue;
node = of_find_node_by_name(NULL, buf); node = of_find_node_by_name(NULL, buf);
if (num_args) if (num_args) {
node = of_find_node_by_name(node, "clk"); parent = node;
node = of_get_child_by_name(parent, "clk");
of_node_put(parent);
}
clkspec.np = node; clkspec.np = node;
clkspec.args_count = num_args; clkspec.args_count = num_args;
for (i = 0; i < num_args; i++) { for (i = 0; i < num_args; i++) {
...@@ -173,11 +177,12 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[]) ...@@ -173,11 +177,12 @@ void __init ti_dt_clocks_register(struct ti_dt_clk oclks[])
if (ret) { if (ret) {
pr_warn("Bad tag in %s at %d: %s\n", pr_warn("Bad tag in %s at %d: %s\n",
c->node_name, i, tags[i]); c->node_name, i, tags[i]);
of_node_put(node);
return; return;
} }
} }
clk = of_clk_get_from_provider(&clkspec); clk = of_clk_get_from_provider(&clkspec);
of_node_put(node);
if (!IS_ERR(clk)) { if (!IS_ERR(clk)) {
c->lk.clk = clk; c->lk.clk = clk;
clkdev_add(&c->lk); clkdev_add(&c->lk);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册