Quoting Kuninori Morimoto (2019-03-06 16:20:40)
Hi Stephen
index 14cbf23..96053a9 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -4213,7 +4213,7 @@ struct clk *of_clk_get_by_name(struct device_node *np, const char *name) if (!np) return ERR_PTR(-ENOENT);
return __of_clk_get(np, -1, np->full_name, name);
return __of_clk_get(np, 0, np->full_name, name);}
Yes this is correct. Thanks for debugging and fixing my thinko here. I was thinking that nobody would call of_clk_get_by_name() unless they wanted to find some clk that had a matching name, but it seems that we also allow NULL to be passed as the name to mean the typical "wildcard match" thing that clkdev has done for years. I'll throw this patch on top of the merge commit so the breakage window is small as I'd rather not rewrite the series just for this. Thanks.
Ahh, OK. So, how about this ? it is including both opinion :) I'm not sure which one should be applied, but both can solve my issue.
No worries. I applied your original patch.